[Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Peter Maydell]

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 <peter.maydell@linaro.org>
---
If I'd noticed when I was adding the banner to config.log that we already
printed this in config-host.mak I'd have done it this way to start with...

 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
+printf "# Configured with:" >> config.log
+printf " '%s'" "$0" "$@" >> config.log
+echo >> config.log
 echo "#" >> config.log
 
 compile_object() {
-- 
1.7.1

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Stefan Hajnoczi]

On Wed, Mar 7, 2012 at 12:16 PM, Peter Maydell <peter.maydell@linaro.org> 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 <peter.maydell@linaro.org>
> ---
> If I'd noticed when I was adding the banner to config.log that we already
> printed this in config-host.mak I'd have done it this way to start with...
>
>  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
> +printf "# Configured with:" >> config.log
> +printf " '%s'" "$0" "$@" >> config.log

You use an interesting feature of printf(1)- it applies the (short)
format string as long as there are arguments.  I checked that this is
in the specification:
http://pubs.opengroup.org/onlinepubs/9699919799/utilities/printf.html

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Stefan

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Eric Blake]

[-- Attachment #1: Type: text/plain, Size: 1362 bytes --]

On 03/07/2012 05:40 AM, Stefan Hajnoczi wrote:
> On Wed, Mar 7, 2012 at 12:16 PM, Peter Maydell <peter.maydell@linaro.org> 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 <peter.maydell@linaro.org>
>> ---
>> If I'd noticed when I was adding the banner to config.log that we already
>> printed this in config-host.mak I'd have done it this way to start with...
>>
>>  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 argument.

-- 
Eric Blake   eblake@redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 620 bytes --]

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Peter Maydell]

On 7 March 2012 17:06, Eric Blake <eblake@redhat.com> wrote:
>>>  # 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 argument.

If you look at the whole change, rather than just the
fragment Stefan quoted to comment on:

 # Print a helpful header at the top of config.log
 echo "# QEMU configure log $(date)" >> config.log
-echo "# produced by $0 $*" >> config.log
+printf "# Configured with:" >> config.log
+printf " '%s'" "$0" "$@" >> config.log
+echo >> config.log
 echo "#" >> config.log

...you'll see that it does exactly what you suggest.

-- PMM

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Eric Blake]

[-- Attachment #1: Type: text/plain, Size: 830 bytes --]

On 03/07/2012 10:08 AM, Peter Maydell wrote:
> If you look at the whole change, rather than just the
> fragment Stefan quoted to comment on:
> 
>  # Print a helpful header at the top of config.log
>  echo "# QEMU configure log $(date)" >> config.log
> -echo "# produced by $0 $*" >> config.log
> +printf "# Configured with:" >> config.log
> +printf " '%s'" "$0" "$@" >> config.log
> +echo >> config.log
>  echo "#" >> config.log
> 
> ...you'll see that it does exactly what you suggest.

Good to hear!  Serves me right for not paying attention to the thread
until my personal keyword of 'opengroup' appeared mid-stream, at which
point I'd already deleted your original message to see the full context.

-- 
Eric Blake   eblake@redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 620 bytes --]

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Peter Maydell]

On 7 March 2012 12:40, Stefan Hajnoczi <stefanha@gmail.com> wrote:
>> +printf "# Configured with:" >> config.log
>> +printf " '%s'" "$0" "$@" >> config.log
>
> You use an interesting feature of printf(1)- it applies the (short)
> format string as long as there are arguments.  I checked that this is
> in the specification:
> http://pubs.opengroup.org/onlinepubs/9699919799/utilities/printf.html

Yes, I had to look up the spec to figure out why it worked too :-)

-- PMM

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Avi Kivity]

On 03/07/2012 02: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.
>  
>  # Print a helpful header at the top of config.log
>  echo "# QEMU configure log $(date)" >> config.log
> -echo "# produced by $0 $*" >> config.log
> +printf "# Configured with:" >> config.log
> +printf " '%s'" "$0" "$@" >> config.log
> +echo >> config.log
>  echo "#" >> config.log
>  

This still breaks if an argument has a single quote.  bash's printf's %q
deals with this correctly, but not sure how portable it is.

-- 
error compiling committee.c: too many arguments to function

Re: [Qemu-devel] [PATCH] configure: Quote the configure args printed in config.log

[Peter Maydell]

On 7 March 2012 17:14, Avi Kivity <avi@redhat.com> wrote:
> This still breaks if an argument has a single quote.  bash's printf's %q
> deals with this correctly, but not sure how portable it is.

True, but we've lived with it for the code that writes to config-host.mak
since 2008 (commit fd69fe2b)...

-- PMM