[PATCH] fix build with --enable-developer-warnings and gcc3.4

Nils Goroll slink at schokola.de
Mon Jan 20 19:37:09 CET 2014


Hi Federico,

On 01/20/14 12:13 PM, Federico Schwindt wrote:

> I was looking at your diff I don't understand the
> -Wno-missing-field-initializers vs -Wno-extra. If the compiler doesn't support
> the former why add the -Wno-extra instead of just not adding -Wno-missing... ?

Hm, your question seems to contain the answer ;)
If the compiler does not support -Wno-missing-field-initializers, we can't use 
-Wno-missing-field-initializers, so we have to use -Wno-extra

> The linker check seems wrong to me. The config.log output for this part is:
>
> configure:16930: checking whether C compiler accepts -fstack-protector
> configure:16949: gcc -std=gnu99 -c -g -O2 -pthread -Wall -Werror
> -Wno-error=unused-result  -fstack-protector  conftest.c >&5
> configure:16949: $? = 0
> configure:16957: result: yes
> configure:16960: checking whether the linker accepts -fstack-protector
> configure:16979: gcc -std=gnu99 -o conftest -g -O2 -pthread -Wall -Werror
> -Wno-error=unused-result    -fstack-protector conftest.c >&5
> configure:16979: $? = 0
> configure:16988: result: yes
>
> So both checks are doing the same. Can you share the config.log?

Notice that gcc is invoked with -c for the first and -o for the second case. 
It's no surprise to me that the second test only fails on an exotic OS (read: 
OpenSolaris) :P

Cheers, Nils



More information about the varnish-dev mailing list