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

Side by Side Diff: gyp/flags.gyp

Issue 343283002: No need to include ../tools/flags in include_dirs when depending on flags target. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: rebase 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/experimental.gyp ('k') | gyp/pathops_unittest.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # GYP file to build flag parser 1 # GYP file to build flag parser
2 # 2 #
3 { 3 {
4 'targets': [ 4 'targets': [
5 { 5 {
6 'target_name': 'flags', 6 'target_name': 'flags',
7 'type': 'static_library', 7 'type': 'static_library',
8 'sources': [ 8 'sources': [
9 '../tools/flags/SkCommandLineFlags.cpp',
9 '../tools/flags/SkCommandLineFlags.h', 10 '../tools/flags/SkCommandLineFlags.h',
10 '../tools/flags/SkCommandLineFlags.cpp',
11 ], 11 ],
12 'dependencies': [ 12 'dependencies': [
13 'skia_lib.gyp:skia_lib', 13 'skia_lib.gyp:skia_lib',
14 ], 14 ],
15 'direct_dependent_settings': { 15 'direct_dependent_settings': {
16 'include_dirs': [ 16 'include_dirs': [
17 '../tools/flags', 17 '../tools/flags',
18 ], 18 ],
19 } 19 }
20 }, 20 },
21 ], 21 ],
22 } 22 }
OLDNEW
« no previous file with comments | « gyp/experimental.gyp ('k') | gyp/pathops_unittest.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698