Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(188)

Unified Diff: build/common.gypi

Issue 224713007: Make it possible to disable fatal linker warnings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index cff6f608d5f00c0e533b23d92c2f05b5ce3b68a6..b8c8f1a2679a604c3c37167063f145fecbad775a 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -1230,6 +1230,10 @@
# whether warnings are treated as errors.
'chromium_code%': 0,
+ # Disable fatal linker warnings, similarly to how we make it possible
+ # to disable -Werror (e.g. for different toolchain versions).
+ 'disable_fatal_linker_warnings%': 0,
+
'release_valgrind_build%': 0,
# TODO(thakis): Make this a blacklist instead, http://crbug.com/101600
@@ -3073,11 +3077,19 @@
['os_posix==1 and (chromeos==0 or target_arch!="arm")', {
jochen (gone - plz use gerrit) 2014/04/04 15:02:40 can you move the 'and (...)' part to the condition
Paweł Hajdan Jr. 2014/04/04 15:09:33 Done. Note that now this might unexpectedly break
'target_defaults': {
'ldflags': [
- '-Wl,--fatal-warnings',
'-Wl,-z,now',
'-Wl,-z,relro',
],
},
+ 'conditions': [
+ ['disable_fatal_linker_warnings==0', {
+ 'target_defaults': {
+ 'ldflags': [
+ '-Wl,--fatal-warnings',
+ ],
+ },
+ }],
+ ],
}],
['os_posix==1 and chromeos==0', {
# Chrome OS enables -fstack-protector-strong via its build wrapper,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698