Skip to content

Commit 13754aa

Browse files
real-or-randomdderjoel
authored andcommitted
configure: Improve rationale for WERROR_CFLAGS
1 parent 1ed8622 commit 13754aa

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

configure.ac

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,9 @@ if test x"$enable_valgrind" = x"yes"; then
342342
SECP_INCLUDES="$SECP_INCLUDES $VALGRIND_CPPFLAGS"
343343
fi
344344

345-
# Add -Werror and similar flags passed from the outside (for testing, e.g., in CI)
345+
# Add -Werror and similar flags passed from the outside (for testing, e.g., in CI).
346+
# We don't want to set the user variable CFLAGS in CI because this would disable
347+
# autoconf's logic for setting default CFLAGS, which we would like to test in CI.
346348
SECP_CFLAGS="$SECP_CFLAGS $WERROR_CFLAGS"
347349

348350
###

0 commit comments

Comments
 (0)