Andrew Black wrote:
Greetings all.

Attached is a patch that should allow the exec utility to execute 'complex' targets. These targets are executable names with an associated set of parameters. If a parameter to a target is %x, the options specified as the value of the -x command line switch are inserted at this position in the parameter string for the child process.


This looks good. Just one thing:

[...]
@@ -228,25 +349,26 @@
    @see process_results
 */
 static void
-run_target (char* target, char** childargv)
+run_target (char* target, char** argv)
[...]
-    printf ("%-18.18s ", target_name);
-    fflush (stdout);
+//     printf ("%-18.18s ", target_name);
+//     fflush (stdout);

I assume you didn't really mean to comment these out. Could you
please resend the patch?

Thanks
Martin

Reply via email to