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

Unified Diff: gyp/utils.gyp

Issue 12963002: Refactor skia_warnings_as_errors (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Use variable instead of disable_warnings_as_errors.gypi Created 7 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
« gyp/cityhash.gyp ('K') | « gyp/libjpeg.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gyp/utils.gyp
===================================================================
--- gyp/utils.gyp (revision 8310)
+++ gyp/utils.gyp (working copy)
@@ -6,7 +6,7 @@
'type': 'static_library',
'standalone_static_library': 1,
'dependencies': [
- 'cityhash',
+ 'cityhash.gyp:cityhash',
],
'include_dirs': [
'../include/config',
@@ -204,7 +204,7 @@
# missing byteswap.h which is needed by CityHash.
# TODO(borenet): Find a way to either provide this dependency or
# replace it.
- 'cityhash',
+ 'cityhash.gyp:cityhash',
],
}],
[ 'skia_os == "android"', {
@@ -219,52 +219,6 @@
],
},
},
- {
- 'target_name': 'cityhash',
- 'type': 'static_library',
- 'standalone_static_library': 1,
- 'include_dirs': [
- '../include/config',
- '../include/core',
- '../src/utils/cityhash',
- '../third_party/externals/cityhash/src',
- ],
- 'sources': [
- '../third_party/externals/cityhash/src/city.cc',
- ],
- 'direct_dependent_settings': {
- 'include_dirs': [
- '../third_party/externals/cityhash/src',
- ],
- },
- 'conditions': [
- [ 'skia_os == "mac"', {
- 'xcode_settings': {
- 'OTHER_CPLUSPLUSFLAGS!': [
- '-Werror',
- ]
- },
- }],
- [ 'skia_os == "win"', {
- 'msvs_settings': {
- 'VCCLCompilerTool': {
- 'WarnAsError': 'false',
- },
- },
- }],
- [ 'skia_os == "android"', {
- 'cflags!': [
- '-Wall',
- '-Werror',
- ],
- }],
- ['skia_os in ["linux", "freebsd", "openbsd", "solaris", "nacl"]', {
- 'cflags!': [
- '-Werror',
- ],
- }],
- ],
- },
],
}
« gyp/cityhash.gyp ('K') | « gyp/libjpeg.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698