diff options
| author | Ben Pfaff <blp@nicira.com> | 2009-11-19 13:25:42 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2009-11-23 11:34:54 -0800 |
| commit | d161c099279e9ab564636c72423268e6da69f795 (patch) | |
| tree | 5bbbafe4b00a3461e8084f03324b98bf0600f596 /acinclude.m4 | |
| parent | 83f6c0502c67ea117c73d3bbb91f9b9e0e81bf87 (diff) | |
Move C compiler warning (-W) flags from CFLAGS to AM_CFLAGS.
C compiler warning options added by the "configure" script have until
now been put into CFLAGS. However that option is supposed to be reserved
for the user under Automake rules, so move them to AM_CFLAGS.
Besides increased adherence to Automake rules, this is useful because
AM_CFLAGS can be overridden on a per-target (e.g. program or library)
basis, whereas CFLAGS cannot. In turn, building the sFlow library (which
will be added in an upcoming commit) requires overriding the compiler
flags to avoid some warning messages. (By modifying the warning flags, we
avoid the need to modify the sFlow library in any way.)
Diffstat (limited to 'acinclude.m4')
| -rw-r--r-- | acinclude.m4 | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/acinclude.m4 b/acinclude.m4 index 767414e6..e37a316d 100644 --- a/acinclude.m4 +++ b/acinclude.m4 @@ -230,8 +230,9 @@ AC_DEFUN([OVS_CHECK_CC_OPTION], dnl OVS_ENABLE_OPTION([OPTION]) dnl Check whether the given C compiler OPTION is accepted. -dnl If so, add it to CFLAGS. +dnl If so, add it to WARNING_FLAGS. dnl Example: OVS_ENABLE_OPTION([-Wdeclaration-after-statement]) AC_DEFUN([OVS_ENABLE_OPTION], - [OVS_CHECK_CC_OPTION([$1], [CFLAGS="$CFLAGS $1"])]) + [OVS_CHECK_CC_OPTION([$1], [WARNING_FLAGS="$WARNING_FLAGS $1"]) + AC_SUBST([WARNING_FLAGS])]) dnl ---------------------------------------------------------------------- |