about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAngus Lees <gus@inodes.org>2016-01-24 21:07:47 +1100
committerXimin Luo <infinity0@pwned.gg>2016-03-05 14:45:25 +0100
commit7fdb9fd94116bf1d46c2d23322313b0db04908a0 (patch)
tree4d9bab905b88fcd2d7436269d52e53e1174758c4
parentd31d8a9a919b705fb8d22ba99a693d9f96b8bdd5 (diff)
downloadrust-7fdb9fd94116bf1d46c2d23322313b0db04908a0.tar.gz
rust-7fdb9fd94116bf1d46c2d23322313b0db04908a0.zip
More reliable and consistent method of cancelling -Werror*
Adding -Wno-error is more reliable and simple than trying to modify existing
flags. We've been using this in Debian already for the past few releases.
Making this change also encourages future maintainers towards "best practises".
Also take the opportunity to use the same method at all places in the file.
-rw-r--r--mk/rt.mk4
1 files changed, 2 insertions, 2 deletions
diff --git a/mk/rt.mk b/mk/rt.mk
index bd17490955d..5f46b3a20c9 100644
--- a/mk/rt.mk
+++ b/mk/rt.mk
@@ -253,7 +253,7 @@ COMPRT_AR_$(1) := $$(AR_$(1))
 # We chomp -Werror here because GCC warns about the type signature of
 # builtins not matching its own and the build fails. It's a bit hacky,
 # but what can we do, we're building libclang-rt using GCC ......
-COMPRT_CFLAGS_$(1) := $$(filter-out -Werror -Werror=*,$$(CFG_GCCISH_CFLAGS_$(1))) -std=c99
+COMPRT_CFLAGS_$(1) := $$(CFG_GCCISH_CFLAGS_$(1)) -Wno-error -std=c99
 
 # FreeBSD Clang's packaging is problematic; it doesn't copy unwind.h to
 # the standard include directory. This should really be in our changes to
@@ -361,7 +361,7 @@ $$(BACKTRACE_BUILD_DIR_$(1))/Makefile: $$(BACKTRACE_DEPS) $$(MKFILE_DEPS)
 	      CC="$$(CC_$(1))" \
 	      AR="$$(AR_$(1))" \
 	      RANLIB="$$(AR_$(1)) s" \
-	      CFLAGS="$$(CFG_GCCISH_CFLAGS_$(1):-Werror=) -fno-stack-protector" \
+	      CFLAGS="$$(CFG_GCCISH_CFLAGS_$(1)) -Wno-error -fno-stack-protector" \
 	      $(S)src/libbacktrace/configure --build=$(CFG_GNU_TRIPLE_$(CFG_BUILD)) --host=$(CFG_GNU_TRIPLE_$(1)))
 	$$(Q)echo '#undef HAVE_ATOMIC_FUNCTIONS' >> \
 	      $$(BACKTRACE_BUILD_DIR_$(1))/config.h