[master] cb00896 distcheck: use the original CFLAGS for the inner configure

Nils Goroll nils.goroll at uplex.de
Thu Dec 8 15:24:05 CET 2016


commit cb00896414ba72c7cc10882c1be7e23dfb796dd3
Author: Nils Goroll <nils.goroll at uplex.de>
Date:   Thu Dec 8 15:19:05 2016 +0100

    distcheck: use the original CFLAGS for the inner configure
    
    In particular with --enable-developer-warnings, we select strict compiler
    options for CFLAGS, which are too strict for the configure built-in conftest.c
    code.
    
    Make distcheck calls configure again for an out-of-tree build, so it would
    see the CFLAGS the "outer" configure built. To avoid this situation, we
    restore the original CFLAGS for the inner configure.
    
    I wonder if there is a better, more autoconf-ish, solution to this problem.

diff --git a/Makefile.am b/Makefile.am
index db3f7e3..c713812 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -21,7 +21,8 @@ EXTRA_DIST = \
 DISTCHECK_CONFIGURE_FLAGS = \
     --enable-developer-warnings \
     --enable-debugging-symbols \
-    --enable-dependency-tracking
+    --enable-dependency-tracking \
+    CFLAGS="$(OCFLAGS)"
 
 install-data-local:
 	$(install_sh) -d -m 0755 $(DESTDIR)$(localstatedir)/varnish



More information about the varnish-commit mailing list