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

Side by Side Diff: gyp/jsoncpp.gyp

Issue 303913002: In Android framework, make tools depend on jsoncpp (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Do as Thoreau would do. Created 6 years, 6 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
« no previous file with comments | « gyp/common_variables.gypi ('k') | include/utils/SkJSONCPP.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 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 # TODO: This file was copied from the external dependency 5 # TODO: This file was copied from the external dependency
6 # third_party/externals/jsoncpp-chromium/jsoncpp.gyp , at revision 125399, 6 # third_party/externals/jsoncpp-chromium/jsoncpp.gyp , at revision 125399,
7 # with directory paths modified to work at this level. 7 # with directory paths modified to work at this level.
8 # 8 #
9 # It would be better for us to depend on that gypfile within the external 9 # It would be better for us to depend on that gypfile within the external
10 # dependency, but so far we have been unable to make that work reliably. 10 # dependency, but so far we have been unable to make that work reliably.
11 # See https://code.google.com/p/skia/issues/detail?id=1023 11 # See https://code.google.com/p/skia/issues/detail?id=1023
12 # and https://code.google.com/p/skia/source/detail?r=7115 12 # and https://code.google.com/p/skia/source/detail?r=7115
13 13
14 { 14 {
15 'variables': { 15 'variables': {
16 'skia_warnings_as_errors': 0, 16 'skia_warnings_as_errors': 0,
17 }, 17 },
18 'targets': [ 18 'targets': [
19 { 19 {
20 'target_name': 'jsoncpp', 20 'target_name': 'jsoncpp',
21 'type': 'static_library',
22 'conditions': [ 21 'conditions': [
23 ['skia_build_json_writer', { 22 ['skia_use_system_json', {
23 'type': 'none',
24 'direct_dependent_settings': {
25 'libraries': [
26 'jsoncpp.a',
27 ],
28 },
29 }, {
30 'type': 'static_library',
24 'defines': [ 31 'defines': [
25 'JSON_USE_EXCEPTION=0', 32 'JSON_USE_EXCEPTION=0',
26 ], 33 ],
27 'sources': [ 34 'sources': [
28 '../third_party/externals/jsoncpp/include/json/assertions.h', 35 '../third_party/externals/jsoncpp/include/json/assertions.h',
29 '../third_party/externals/jsoncpp/include/json/autolink.h', 36 '../third_party/externals/jsoncpp/include/json/autolink.h',
30 '../third_party/externals/jsoncpp/include/json/config.h', 37 '../third_party/externals/jsoncpp/include/json/config.h',
31 '../third_party/externals/jsoncpp/include/json/features.h', 38 '../third_party/externals/jsoncpp/include/json/features.h',
32 '../third_party/externals/jsoncpp/include/json/forwards.h', 39 '../third_party/externals/jsoncpp/include/json/forwards.h',
33 '../third_party/externals/jsoncpp/include/json/json.h', 40 '../third_party/externals/jsoncpp/include/json/json.h',
(...skipping 18 matching lines...) Expand all
52 ], 59 ],
53 }, 60 },
54 'cflags': [ 61 'cflags': [
55 '-w', 62 '-w',
56 ], 63 ],
57 }], 64 }],
58 ], 65 ],
59 }, 66 },
60 ], 67 ],
61 } 68 }
OLDNEW
« no previous file with comments | « gyp/common_variables.gypi ('k') | include/utils/SkJSONCPP.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698