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

Side by Side Diff: gyp/libjpeg.gyp

Issue 12963002: Refactor skia_warnings_as_errors (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Fix merge conflicts and trybot failures 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright 2012 The Chromium Authors. All rights reserved. 1 # Copyright 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # This is a copy of ../third_party/externals/libjpeg/libjpeg.gyp , modified 5 # This is a copy of ../third_party/externals/libjpeg/libjpeg.gyp , modified
6 # such that all source paths point into that directory. 6 # such that all source paths point into that directory.
7 # See http://code.google.com/p/skia/issues/detail?id=543 ('wrap libjpeg.gyp 7 # See http://code.google.com/p/skia/issues/detail?id=543 ('wrap libjpeg.gyp
8 # from Chrome's libjpeg port, rather than making our own copy') for a better 8 # from Chrome's libjpeg port, rather than making our own copy') for a better
9 # long-term solution. 9 # long-term solution.
10 10
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 '../third_party/externals/libjpeg/jquant2.c', 88 '../third_party/externals/libjpeg/jquant2.c',
89 '../third_party/externals/libjpeg/jutils.c', 89 '../third_party/externals/libjpeg/jutils.c',
90 '../third_party/externals/libjpeg/jversion.h', 90 '../third_party/externals/libjpeg/jversion.h',
91 ], 91 ],
92 'direct_dependent_settings': { 92 'direct_dependent_settings': {
93 'include_dirs': [ 93 'include_dirs': [
94 '../third_party/externals/libjpeg', 94 '../third_party/externals/libjpeg',
95 ], 95 ],
96 }, 96 },
97 'conditions': [ 97 'conditions': [
98 ['OS!="win"', { 98 [ 'skia_os != "win"', {
99 'product_name': 'jpeg', 99 'product_name': 'jpeg',
100 'cflags': [ 100 'cflags': [
101 '-Wno-main', # supresses warnings about naming things "main" 101 '-Wno-main', # supresses warnings about naming things "main"
102 ], 102 ],
103 }], 103 }],
104 ['OS=="android"', { 104 # Turn off warnings as errors.
105 'cflags!': [ 105 [ 'skia_warnings_as_errors', {
106 '-fno-rtti', # supresses warnings about invalid option of non -C++ code 106 'conditions': [
107 '-Wall', 107 [ 'skia_os == "mac" or skia_os == "ios"', {
108 '-Werror', 108 'xcode_settings': {
109 ], 109 'OTHER_CPLUSPLUSFLAGS!': [
110 }], 110 '-Werror',
111 ['OS in ["linux", "freebsd", "openbsd", "solaris", "nacl"]', { 111 ]
112 'cflags!': [ 112 },
113 '-Werror', 113 }],
114 [ 'skia_os == "win"', {
115 'msvs_settings': {
116 'VCCLCompilerTool': {
117 'WarnAsError': 'false',
118 },
119 },
120 }],
121 [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "na cl"]', {
122 'cflags!': [
123 '-Werror',
124 ],
125 }],
114 ], 126 ],
115 }], 127 }],
116 ], 128 ],
117 }, 129 },
118 ], 130 ],
119 }, { 131 }, {
120 'targets': [ 132 'targets': [
121 { 133 {
122 'target_name': 'libjpeg', 134 'target_name': 'libjpeg',
123 'type': 'none', 135 'type': 'none',
(...skipping 13 matching lines...) Expand all
137 ], 149 ],
138 }], 150 }],
139 ], 151 ],
140 } 152 }
141 153
142 # Local Variables: 154 # Local Variables:
143 # tab-width:2 155 # tab-width:2
144 # indent-tabs-mode:nil 156 # indent-tabs-mode:nil
145 # End: 157 # End:
146 # vim: set expandtab tabstop=2 shiftwidth=2: 158 # vim: set expandtab tabstop=2 shiftwidth=2:
OLDNEW
« gyp/jsoncpp.gyp ('K') | « gyp/jsoncpp.gyp ('k') | gyp/utils.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698