2012-12-20 03:39:30 Branko Čibej napisał(a):
> I found the following in configure.ac which unconditionally adds a
> -Werror flag:
> 
> dnl Add -Werror=implicit-function-declaration to CFLAGS
> CFLAGS_KEEP="$CFLAGS"
> CFLAGS="$CFLAGS_KEEP -Werror=implicit-function-declaration"
> AC_COMPILE_IFELSE([AC_LANG_SOURCE([[]])], [CFLAGS_KEEP="$CFLAGS"])
> CFLAGS="$CFLAGS_KEEP"
> 
> 
> I'm almost sure this should only be done in maintainer-mode?

-Werror != -Werror=implicit-function-declaration
We want to always use -Werror=implicit-function-declaration (if compiler 
supports it) to ensure that nobody forgets
to include required headers. (Implicit function declarations are always illegal 
in C++, so this flag is needed only
in CFLAGS.)

-- 
Arfrever Frehtes Taifersar Arahesis

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to