Macro run: fixed incorrect escaping of arguments

crystal-lang · Apr 10, 2017 · bd09177 · bd09177
1 parent b7e048b
commit bd09177
Showing 1 changed file with 3 additions and 10 deletions.
diff --git a/src/compiler/crystal/macros/macros.cr b/src/compiler/crystal/macros/macros.cr
@@ -91,16 +91,9 @@ class Crystal::Program
     compiled_macro_run = @compiled_macros_cache[filename] ||= macro_compile(filename)
     compiled_file = compiled_macro_run.filename
 
-    command = String.build do |str|
-      str << compiled_file.inspect
-      args.each do |arg|
-        str << " "
-        str << arg.inspect
-      end
-    end
-
-    result = `#{command}`
-    {$?.success?, result}
+    io = IO::Memory.new
+    Process.run(compiled_file, args: args, shell: true, output: io)
+    {$?.success?, io.to_s}
   end
 
   record RequireWithTimestamp, filename : String, epoch : Int64 do