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

Unified Diff: pylib/gyp/generator/ninja.py

Issue 27353003: Adds the ability to skip includes. (Closed) Base URL: https://chromium.googlesource.com/external/gyp.git@master
Patch Set: Created 7 years, 2 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
« no previous file with comments | « no previous file | pylib/gyp/input.py » ('j') | pylib/gyp/input.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pylib/gyp/generator/ninja.py
diff --git a/pylib/gyp/generator/ninja.py b/pylib/gyp/generator/ninja.py
index 229cba410a5feb13b92ce69f690027dfb96c58a3..0a47e99950b9e66fe3d0e9585dab87d472d9be67 100644
--- a/pylib/gyp/generator/ninja.py
+++ b/pylib/gyp/generator/ninja.py
@@ -212,7 +212,7 @@ class Target:
class NinjaWriter:
def __init__(self, qualified_target, target_outputs, base_dir, build_dir,
output_file, toplevel_build, output_file_name, flavor,
- deps_file=None, toplevel_dir=None):
+ toplevel_dir=None):
"""
base_dir: path from source root to directory containing this gyp file,
by gyp semantics, all input paths are relative to this
@@ -246,7 +246,6 @@ class NinjaWriter:
# Relative path from base dir to build dir.
base_to_top = gyp.common.InvertRelativePath(base_dir, toplevel_dir)
self.base_to_build = os.path.join(base_to_top, build_dir)
- self.link_deps_file = deps_file
def ExpandSpecial(self, path, product_dir=None):
"""Expand specials like $!PRODUCT_DIR in |path|.
@@ -985,13 +984,6 @@ class NinjaWriter:
continue
linkable = target.Linkable()
if linkable:
- if self.link_deps_file:
- # Save the mapping of link deps.
- self.link_deps_file.write(self.qualified_target)
- self.link_deps_file.write(' ')
- self.link_deps_file.write(target.binary)
- self.link_deps_file.write('\n')
-
new_deps = []
if (self.flavor == 'win' and
target.component_objs and
@@ -2016,13 +2008,6 @@ def GenerateOutputForConfig(target_list, target_dicts, data, params,
# objects.
target_short_names = {}
- # Extract the optional link deps file name.
- LINK_DEPS_FILE = 'link_deps_file'
- if LINK_DEPS_FILE in generator_flags:
- link_deps_file = open(generator_flags.get(LINK_DEPS_FILE), 'wb')
- else:
- link_deps_file = None
-
for qualified_target in target_list:
# qualified_target is like: third_party/icu/icu.gyp:icui18n#target
build_file, name, toolset = \
@@ -2048,8 +2033,7 @@ def GenerateOutputForConfig(target_list, target_dicts, data, params,
writer = NinjaWriter(qualified_target, target_outputs, base_path, build_dir,
ninja_output,
toplevel_build, output_file,
- flavor, link_deps_file,
- toplevel_dir=options.toplevel_dir)
+ flavor, toplevel_dir=options.toplevel_dir)
target = writer.WriteSpec(spec, config_name, generator_flags)
« no previous file with comments | « no previous file | pylib/gyp/input.py » ('j') | pylib/gyp/input.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698