On Thu, Sep 17, 2009 at 04:44:11PM +0200, Jim Meyering wrote:
All looks good, though this seems like it *is* a functional change, since it removes the -h and -p options below.
Ah yes, you're right. Removing these is OK I think because no one would ever have used them, and they are not used by the appliance init script.
Two questions:
First, note the definition of GUESTFWD_ADDR above Why the commented-out one below (in src/guestfs.c):
//#define GUESTFWD_ADDR "10.0.2.4"
This is just commented out because it isn't used, and gcc now gives a warning when we define but don't use a macro.
Second, I saw this in the context:
/* Make sure SIGPIPE doesn't kill us. */ memset (&sa, 0, sizeof sa); sa.sa_handler = SIG_IGN;
Have you considered alternate ways of ignoring SIGPIPE? Ignoringn SIGPIPE across the board can cause problems (albeit subtle) in unsuspecting child processes.
I didn't know there were alternate ways to ignore SIGPIPE. Can you give some examples of how it should be done? (Possibly in a new thread -- seems a separate issue from the patch at hand). Rich. -- Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones Read my programming blog: http://rwmj.wordpress.com Fedora now supports 75 OCaml packages (the OPEN alternative to F#) http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora