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

Unified Diff: build/common.gypi

Issue 8228005: openbsd and freebsd fixes for base (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Add myself to the AUTHORS Created 9 years, 2 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 | « base/threading/platform_thread_posix.cc ('k') | tools/clang/scripts/update.sh » ('j') | 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 9239aac114fef4380467f08464279de17842c763..47aa3b02886feaf29cf867be198dcc5cffa3f5c9 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -1135,13 +1135,15 @@
'-Wsign-compare',
]
}],
- [ 'os_posix==1 and OS!="mac" and chromeos==0', {
+ [ 'os_posix==1 and OS!="mac" and OS!="openbsd" and chromeos==0', {
'cflags': [
# Don't warn about ignoring the return value from e.g. close().
# This is off by default in some gccs but on by default in others.
# Currently this option is not set for Chrome OS build because
# the current version of gcc (4.3.4) used for building Chrome in
# Chrome OS chroot doesn't support this option.
+ # OpenBSD does not support this option either, since it's using
+ # gcc 4.2.1, which does not have this flag yet.
# TODO(mazda): remove the conditional for Chrome OS when gcc
# version is upgraded.
'-Wno-unused-result',
« no previous file with comments | « base/threading/platform_thread_posix.cc ('k') | tools/clang/scripts/update.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698