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

Side by Side Diff: gyp/jsoncpp.gyp

Issue 481823003: Roll jsoncpp, drop dependency on Chromium overrides. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 4 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 | « DEPS ('k') | no next file » | 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
5 # TODO: This file was copied from the external dependency
6 # third_party/externals/jsoncpp-chromium/jsoncpp.gyp , at revision 125399,
7 # with directory paths modified to work at this level.
8 #
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.
11 # See https://code.google.com/p/skia/issues/detail?id=1023
12 # and https://code.google.com/p/skia/source/detail?r=7115
13
14 { 4 {
15 'variables': { 5 'variables': {
16 'skia_warnings_as_errors': 0, 6 'skia_warnings_as_errors': 0,
17 }, 7 },
18 'targets': [ 8 'targets': [
19 { 9 {
20 'target_name': 'jsoncpp', 10 'target_name': 'jsoncpp',
21 'conditions': [ 11 'conditions': [
22 ['skia_use_system_json', { 12 ['skia_use_system_json', {
23 'type': 'none', 13 'type': 'none',
24 'direct_dependent_settings': { 14 'direct_dependent_settings': {
25 'libraries': [ 15 'libraries': [
26 'jsoncpp.a', 16 'jsoncpp.a',
27 ], 17 ],
28 }, 18 },
29 }, { 19 }, {
30 'type': 'static_library', 20 'type': 'static_library',
31 'defines': [ 21 'defines': [
32 'JSON_USE_EXCEPTION=0', 22 'JSON_USE_EXCEPTION=0',
33 ], 23 ],
34 'sources': [ 24 'sources': [
35 '../third_party/externals/jsoncpp/include/json/assertions.h',
bungeman-skia 2014/08/19 14:11:10 In general, I rather like having the header files
36 '../third_party/externals/jsoncpp/include/json/autolink.h',
37 '../third_party/externals/jsoncpp/include/json/config.h',
38 '../third_party/externals/jsoncpp/include/json/features.h',
39 '../third_party/externals/jsoncpp/include/json/forwards.h',
40 '../third_party/externals/jsoncpp/include/json/json.h',
41 '../third_party/externals/jsoncpp/include/json/reader.h',
42 '../third_party/externals/jsoncpp-chromium/overrides/include/json/va lue.h',
43 '../third_party/externals/jsoncpp/include/json/writer.h',
44 '../third_party/externals/jsoncpp/src/lib_json/json_batchallocator.h ',
45 '../third_party/externals/jsoncpp/src/lib_json/json_reader.cpp', 25 '../third_party/externals/jsoncpp/src/lib_json/json_reader.cpp',
46 '../third_party/externals/jsoncpp/src/lib_json/json_tool.h', 26 '../third_party/externals/jsoncpp/src/lib_json/json_value.cpp',
47 '../third_party/externals/jsoncpp-chromium/overrides/src/lib_json/js on_value.cpp',
48 '../third_party/externals/jsoncpp/src/lib_json/json_writer.cpp', 27 '../third_party/externals/jsoncpp/src/lib_json/json_writer.cpp',
49 ], 28 ],
50 'include_dirs': [ 29 'include_dirs': [
51 '../third_party/externals/jsoncpp-chromium/overrides/include/',
52 '../third_party/externals/jsoncpp/include/', 30 '../third_party/externals/jsoncpp/include/',
53 '../third_party/externals/jsoncpp/src/lib_json/', 31 '../third_party/externals/jsoncpp/src/lib_json/',
54 ], 32 ],
55 'direct_dependent_settings': { 33 'direct_dependent_settings': {
56 'include_dirs': [ 34 'include_dirs': [
57 '../third_party/externals/jsoncpp-chromium/overrides/include/',
58 '../third_party/externals/jsoncpp/include/', 35 '../third_party/externals/jsoncpp/include/',
59 ], 36 ],
60 }, 37 },
61 'cflags': [ 38 'cflags': [
62 '-w', 39 '-w',
63 ], 40 ],
64 }], 41 }],
65 ], 42 ],
66 }, 43 },
67 ], 44 ],
68 } 45 }
OLDNEW
« no previous file with comments | « DEPS ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698