[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] configure: replace \n with space in optarg
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] configure: replace \n with space in optarg |
Date: |
Mon, 4 Aug 2014 09:50:20 +0100 |
On 4 August 2014 08:12, Hu Tao <address@hidden> wrote:
> When optarg happens to contain \n like:
>
> ../configure --target-list='i386-softmmu
> x86_64-softmmu'
>
> make will fail with message:
>
> config-host.mak:45: *** missing separator. Stop.
Why would you put newlines in the option string? This is
likely to break for many configure arguments, not just this
one...
I'm a bit dubious about taking this change unless you can
point to existing practice (eg do autoconf configure scripts
clean up whitespace like this?)
thanks
-- PMM