Do not use CPPFLAGS / CFLAGS together with CC_FOR_BUILD (#882)

* Do not use CPPFLAGS / CFLAGS together with CC_FOR_BUILD
* Restore -std=c99 when building makedoc
* Make sure host-independent CFLAGS are passed to CC_FOR_BUILD
* Let CC_FOR_BUILD carry CFLAGS_FOR_BUILD, there's no reason to split them

diff --git a/Makefile.autosetup b/Makefile.autosetup
index f017c3f75fda545c0cbbbd31b39462493f43946a..33ab69cd14d483d00e51ecac4d073f60eb6aabe8 100644 (file)
--- a/Makefile.autosetup
+++ b/Makefile.autosetup
@@ -7,7 +7,7 @@ PACKAGE_VERSION=@PACKAGE_VERSION@
 PWD=           @PWD@
 AR=            @AR@
 CC=            @CC@
-CC_FOR_BUILD=  @CC_FOR_BUILD@
+CC_FOR_BUILD=  @CC_FOR_BUILD@ @CFLAGS_FOR_BUILD@
 CPP=           @CPP@
 CFLAGS=                @CPPFLAGS@ @CFLAGS@ -I. -I@top_srcdir@ -Wall $(EXTRA_CFLAGS)
 LDFLAGS=       @LDFLAGS@ $(EXTRA_LDFLAGS)

diff --git a/auto.def b/auto.def
index 3f6b23b3b32957879e36b185b137f8261cf8da97..9609f174b9ce28b67273dba5e81eaa8718ac0466 100644 (file)
--- a/auto.def
+++ b/auto.def
@@ -183,6 +183,7 @@ if {1} {
   if {[cc-check-standards c99] eq {}} {
     user-error "C99 is required"
   }
+  define-append CFLAGS_FOR_BUILD -std=c99
 
   # Check for tools and programs
   cc-check-tools ar ranlib strip
@@ -212,13 +213,15 @@ if {1} {
   # Enable extensions (reverse-engineered from AC_SYSTEM_EXTENSIONS)
   if {1} {
     if {[cc-check-includes minix/config.h]} {
-      define-append CFLAGS -D_POSIX_SOURCE=1
-      define-append CFLAGS -D_POSIX_1_SOURCE=2
-      define-append CFLAGS -D_MINIX=1
+      lappend extensions -D_POSIX_SOURCE=1
+      lappend extensions -D_POSIX_1_SOURCE=2
+      lappend extensions -D_MINIX=1
     }
-    define-append CFLAGS -D_ALL_SOURCE=1
-    define-append CFLAGS -D_GNU_SOURCE=1
-    define-append CFLAGS -D__EXTENSIONS__
+    lappend extensions -D_ALL_SOURCE=1
+    lappend extensions -D_GNU_SOURCE=1
+    lappend extensions -D__EXTENSIONS__
+    define-append CFLAGS_FOR_BUILD {*}$extensions
+    define-append CFLAGS {*}$extensions
   }
   cc-with [list -cflags [get-define CFLAGS]]
 

diff --git a/doc/Makefile.autosetup b/doc/Makefile.autosetup
index a262343406eef61c81d96752f47b307560a3ce8d..b1eb7b0090ec312fddfd853607febce9fa62653b 100644 (file)
--- a/doc/Makefile.autosetup
+++ b/doc/Makefile.autosetup
@@ -1,7 +1,7 @@
-MAKEDOC_CPP = $(CC_FOR_BUILD) $(CPPFLAGS) -D_MAKEDOC -E -C -I.
+MAKEDOC_CPP = $(CC_FOR_BUILD) -D_MAKEDOC -E -C -I.
 
 doc/makedoc$(EXEEXT): doc/makedoc.c
-       $(CC_FOR_BUILD) $(CPPFLAGS) $(CFLAGS) -o $@ doc/makedoc.c
+       $(CC_FOR_BUILD) -I. -o $@ doc/makedoc.c
 
 doc/neomuttrc: $(SRCDIR)/init.h doc/makedoc$(EXEEXT) $(SRCDIR)/doc/neomuttrc.head
        sed -e 's,@docdir@,$(docdir),' $(SRCDIR)/doc/neomuttrc.head \