Re: [Qemu-devel] [PATCH] configure: fix breakage of --extra-{cflags, ldflags}

[Nathan Froyd <froydnj@codesourcery.com>]

On Thu, Jul 23, 2009 at 04:40:41AM -0700, Nathan Froyd wrote:
> Commit e3fc14c3a1a77a1cda669a3e16f1f6b82577e4ec broke the use of
> --extra-{cflags,ldflags} in the case where those flags were necessary
> for bits of configure to succeed.  For instance, if you did:
>
>   configure [...] --extra-cflags=-I/path/to/zlib/headers \
>     --extra-ldflags=-L/path/to/zlib/libraries
>
> prior to the above commit, the paths would go into CFLAGS and LDFLAGS
> for use by the rest of the configure tests.

I spoke too soon; it turns out that the use of CFLAGS in the zlib
configure test is a local patch we have to QEMU.  So this is not an
issue for upstream QEMU at the moment.

It does raise the question of how configure is supposed to find things
in non-standard locations, but since none of the other configure tests
seem to care about non-standard compilation scenarios (e.g. using
--extra-* to specify the location of the SASL libraries won't
work)...this is probably not worth addressing unless people feel it's
worthwhile for the sake of Truth and Beauty.

Sorry about the noise.

-Nathan