Remove guestfs___print_timestamped_argv.
authorRichard W.M. Jones <rjones@redhat.com>
Thu, 18 Aug 2011 16:41:09 +0000 (17:41 +0100)
committerRichard W.M. Jones <rjones@redhat.com>
Thu, 18 Aug 2011 17:02:56 +0000 (18:02 +0100)
commit635af5be04265f845186b40e9a9fe7b102ad6909
tree500912c27eadeab0fd27a3f090462d3d6b7114cb
parent70c033998e0e721dc4f9eb2a20348098b259752c
Remove guestfs___print_timestamped_argv.

This function was used to print the qemu and
febootstrap-supermin-helper command lines.

Unfortunately in the qemu case it was used incorrectly: it called the
internal debug function (ie. event API callback) from the forked qemu
subprocess, which meant that higher level event callbacks might have
been invoked from the child process.

To fix this, convert the qemu case into a new function called
print_qemu_command line which just prints the command line directly to
stderr.  This is called after stderr has been redirected into the pipe
to the main process.  Thus the qemu command line will be marshalled
into the event API along with other qemu and appliance output.

After fixing this, only one use of guestfs___print_timestamped_argv
remained, for printing the febootstrap-supermin-helper command line.
This is converted to a local function print_febootstrap_command_line.

Also print_febootstrap_command_line is now called before we fork
febootstrap-supermin-helper, so that messages no longer overlap.
src/appliance.c
src/guestfs-internal.h
src/launch.c