[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] configure: fix libsmartcard_nss compile check
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH] configure: fix libsmartcard_nss compile check |
Date: |
Mon, 09 Jul 2012 15:29:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.5) Gecko/20120601 Thunderbird/10.0.5 |
On 07/09/12 14:06, Alexander Graf wrote:
> When just calling ./configure on my box, I always ran into the same issue:
>
> In file included from /usr/include/nss3/pkcs11t.h:1780,
> from /usr/include/nss3/keythi.h:41,
> from /usr/include/nss3/keyt.h:41,
> from /usr/include/nss3/pk11pub.h:43,
> from libcacard/vcard_emul_nss.c:21:
> /usr/include/nss3/pkcs11n.h:365:26: error: "__GNUC_MINOR" is not defined
>
> This is a bug in nss3. But why didn't configure bail out of the feature
> if it doesn't even compile? Because this really is just a warning, not an
> error. But configure builds its test program without -Werror, while we do
> build our sources with -Werror by default.
>
> Force the check to also use -Werror. This fixes the default build for me
> again.
Good idea, but shouldn't we do that
(a) for all compile tests and
(b) only with --enable-werror
?
cheers,
Gerd