From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:33210) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S5KKc-0002en-F7 for qemu-devel@nongnu.org; Wed, 07 Mar 2012 12:07:19 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S5KKW-0008Ac-5g for qemu-devel@nongnu.org; Wed, 07 Mar 2012 12:07:18 -0500 Message-ID: <4F57959C.5090408@redhat.com> Date: Wed, 07 Mar 2012 10:06:36 -0700 From: Eric Blake MIME-Version: 1.0 References: <1331122589-15717-1-git-send-email-peter.maydell@linaro.org> In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="------------enigEAA529FC88DCA04FFCF6930B" Subject: Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: qemu-trivial@nongnu.org, Peter Maydell , qemu-devel@nongnu.org This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enigEAA529FC88DCA04FFCF6930B Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 03/07/2012 05:40 AM, Stefan Hajnoczi wrote: > On Wed, Mar 7, 2012 at 12:16 PM, Peter Maydell wrote: >> Use the same mechanism we use for printing the configure command >> line to config-host.mak to print it to config.log. This fixes a >> bug where the config.log version didn't quote arguments with spaces. >> >> Signed-off-by: Peter Maydell >> --- >> If I'd noticed when I was adding the banner to config.log that we alre= ady >> printed this in config-host.mak I'd have done it this way to start wit= h... >> >> configure | 4 +++- >> 1 files changed, 3 insertions(+), 1 deletions(-) >> >> diff --git a/configure b/configure >> index a5eb832..1b63436 100755 >> --- a/configure >> +++ b/configure >> @@ -22,7 +22,9 @@ rm -f config.log >> >> # Print a helpful header at the top of config.log >> echo "# QEMU configure log $(date)" >> config.log >> -echo "# produced by $0 $*" >> config.log This left a trailing newline, >> +printf "# Configured with:" >> config.log >> +printf " '%s'" "$0" "$@" >> config.log but this does not. You probably want to add another echo afterwards, so that the next line of the log doesn't look like another configure argumen= t. --=20 Eric Blake eblake@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org --------------enigEAA529FC88DCA04FFCF6930B Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iQEcBAEBCAAGBQJPV5WcAAoJEKeha0olJ0NqUcEH+gKgNXBL0vcuRQWvMlnb/i2W GkKUDHrCFccdxDs0Vk3DSs6XYyAcqKmbOBKWKPM/nmWpyJ+RanWQrEl1u/CLQjhj fg+H3rD1WODcMWuTWN3LWrlU1SODnxg/dGMXiqVgIrFQqw0I3YjL6yX3aYr1PE1A GjjpjodqyPL0RBqnULBwT3IOVCNksmhyUASPJqXmuIxPPgxeK4p6Mag1prYjPugX uQSij1ZZSPVB4AvzTutad54+caOM9/pShueys1wZh8Dhk5HiOd2GAqDBbrim6YM+ ET4xFBnaCdoHKYB+3vz5wHjdkqAZGzfSPyIvfFm5DNVBwvjxaqoRnxGIK2xMRN0= =F15A -----END PGP SIGNATURE----- --------------enigEAA529FC88DCA04FFCF6930B--