Ref: http://www.opensource.apple.com/source/cctools/cctools-809/misc/libtool.c

Adhoc testing wi...

This commit does two things to fix rules for VS2013:

* Don't exclude rule 'input' files (i.e. t...

Chromium only supports 2013 now, and the bots no longer have older versions.

Support for other ...

This fixes the indentation for some of the gyp files using 4 spaces
indentation instead of the u...

Previously if an invalid_target was supplied analyze would report it
via 'warning'. I'm moving t...

Chromium doesn't care about MSVC08 or Mac OS X makefiles anymore,
whereas some third_party libra...

Finding cycles in a directed graph only requires a simple depth first
traversal, it does not req...

Add supports for assets catalog to ninja generator (make generator does
not support them but avo...

To allow the chromium WebView to build unbundled, we need to stop
linking to system libraries an...

Update the list of identity variables in gypd so that the generator will
run successfully on Chr...

BUG=gyp:462
[email protected]

Review URL: https://codereview.chromium.org/629543002

git-s...

Right now, ninja generates unique names for actions/rules by combining the
action/rule name with...

This adds support for 'midl_include_dirs', which is handled like 'include_dirs'
and 'resource_in...

Patch from [email protected].

BUG=
[email protected]

Review URL: https://codereview.chr...

Each calling convention decorates C functions in a different way, so we can use
that to test the...

This reverts commit 397b1c684b0b07a38f485f091cc4394a3d733334.

Failing on win64 bots per http://...

BUG=

Review URL: https://codereview.chromium.org/594923002

git-svn-id: http://gyp.googlecode.c...

This change is required as a prerequisite to adding proper Windows Store (8.1 only) support to G...

BUG=
Patch from Josh Converse <[email protected]>!

[email protected], thomasvl@chro...

Build fix

[email protected]

Review URL: https://codereview.chromium.org/598643003

git...

This fallback behaviour may have worked at some point, but doesn't
any more, so simply error out...

Add a key 'aosp_build_settings' which a target can use to set Android
build system variables in ...

To make the generated makefiles smaller and slightly easier to debug,
omit information that is u...

Closing brackets are now properly indented with the opening
bracket.
Also adds "cflags_cc" in th...

This builds on https://codereview.chromium.org/543743003/ .

BUG=410410
TEST=covered by test now...

This way you can pass these targets to the build system and not get
errors.

BUG=410410
TEST=cov...

Matching wasn't working if paths supplied were absolute. gyp_chromium
triggers absolute paths, s...

Background: nm and readelf are used by ninja.py to generate
.TOC files. These .TOC files are wha...

The Android build system now supports multilib 32/64-bit host builds as
well as target builds, a...

This makes it easier to detect if something is going to be built
without knowing all the names o...

In order to support rolling to a new version of AOSP to run gyp tests
(to pick up android build ...

if comparison was incorrect for truthy values when added at
https://codereview.chromium.org/1264...

The set of build targets is the minimal set of targets reachable
from the all target that contai...

This was from the first version and is no longer used. Also cleaned up
tests as well.

BUG=none
...

For IDL files, GYP ninja normally generates rules to build MIDL
unless there are explicit rules....

__ -> _
and lots of single newlines to double newlines.

BUG=none
[email protected]

Review ...

stack. By that I mean if a gyp* file has been modified I don't run
analyze and assume everything ...

The tests gyptest-actions, gyptest-copies and gyptest-rules fail when
using the xcode-ninja gene...

e.g. test/generator-output/src/prog1.gyp

TEST=Run
./gyptest.py -f xcode-ninja test/generato...

File is specified by way of analyzer_output_path. I'm also changing
from a hard error if a targe...

Some generators like gypd and gypsh define the generator_default_variables
as an identity expans...

Patch by Johannes Sartisohn <[email protected]>

Review URL: https://codereview.chromium.org...

Here's a list of the changes:
. I'm changing input file to be a JSON file. This allows for passi...

Setting this to 1 will make ninja use 'pool = console' for the action/rule.

BUG=
R=thakis@chrom...

This CL adds the extension target type and adds the link flags to
compile .appex extensions.

BU...

Paths in INCLUDE are considered system headers by clang-cl and it will suppress
warnings from s...

Maybe the try failures are due to the try server not adding the new files
correctly, since the t...

Was using os.getcwd when it needs to use options.toplevel_dir

[email protected]
BUG=383609
TE...

Wasn't delaing with relative paths outside the repo. NACL does this
for things like /usr/include...

.xcdatamodeld documents are used for Model File Format.
https://developer.apple.com/library/mac...

With this CL, the check of duplicate basenames in the same source list can be disabled with '--n...

When running in chrome paths are absolute.

Also makes chrome build work with analyzer on window...

The old way of joining multiple commands would take care of failing correctly
when there are mul...

When doing 32bit release component builds of chromium, nacl_helper fails to
link like so:

g++ [...

Note that this includes buildbot steps to start and stop an Android emulator
for testing.

BUG=g...

With this CL, Ninja generator starts recognizing 'AR.host' specified in 'make_global_settings', ...

With this CL, Ninja generator starts recognizing 'LD'/'LD.host' specified in 'make_global_settin...

This is a follow up CL for r1931 and r1935. This is just a revise of a unittest and does not cha...

This is a follow up CL for r1937. This is just a revise of a unittest and does not changes exist...

Newer versions of the Android build system have -Wunused-params enabled
by default. Drop the man...

This CL describes the current behavior with some unittests, before making Ninja generator aware ...

This version reads a file to get a list of input files, then outputs
if any of the input files a...

With this CL, 'AR' in 'make_global_settings' can be recognized by Ninja generator, like Make gen...

This is a follow up CL for gyp r1931, just for the code cleanup. Inspired by Nico's gyp r1893.
...

Builds on discussion in https://codereview.chromium.org/321813002/.

Make sure that cc_host and ...

If a user's language is set to anything other than English, the output
of various tools may vary...

Currently sharing the code among host/NaCl/NDK/Emscripten on top of Ninja+gyp isn't so easy as i...

Skia's buildbots, particularly Windows builds with LTO, are held back by the
link pool. We'd li...

Currently the clean command is specified as

ninja.exe -C $(OutDir) -t clean $(ProjectName)

but...

Makes no difference for me but might help others because gyp can easily
keep lots of processors ...

A chromium gyp rule for generating assembly offsets for libvpx/webrtc
needs to be able to refer ...

_FixPath is designed to take gyp paths and convert them to msvs project paths.
This translates a...

The test uses a "bad" idl to verify that it gets excluded from the msvs build.
If it doesn't get...

$(InputDir) and $(InputPath) assume relative paths, but they're mapped to
%(RootDir)%(Directory)...

The issue happens with e.g. accessibility.gyp for the ax_gen target:
Because ax_enums.idl is als...

Windows x64 build otherwise emits noise like:

"
64 bit Processing ..\..\third_party\iaccessible...

When using C++ library, pass the -stdlib flag to the linker so that the
correct library is linke...

MSVS doesn't compile files for 'none' targets, so there is no need to "exclude"
them.

BUG=
R=sc...

BUG=chromium:372716
[email protected]

Review URL: https://codereview.chromium.org/2894...

A few actions/rules need to be able to call the intermediates-dir-for
function from the Android ...

This copy of pylintrc is never used, and its presence is confusing. The
canned pylint check uses...

BUG=
[email protected]

Review URL: https://codereview.chromium.org/260933005

git-svn-id: ...

Silence two warnings:
OrderedSet.discard: Redefining built-in 'next'
OrderedSet.pop: Arguments n...

Adds new GYP_GENERATOR_FLAG xcode_ninja_target_pattern. This
customization allows for non-execut...

Some commands are sensitive to the directory they're executed from. Chrome's
Mojo is trying to u...

BUG=chromium:369487
TEST=./build/gyp_chromium succeeds in detecting include paths for eclipse ge...

This reverts commit 56a70c5641bbb01683dfeb7aa8a65800a00fb936.

The change made bug 112 trigger o...

Host modules shouldn't set LOCAL_MODULE_TARGET_ARCH or other multilib
build related variables, a...

IsStrCanonicalNumber is called on everything in a gyp file
and for a build system the size of ch...

The system deepcopy supports recursive objects or non-tree structures.
Those structures don't ex...

1) Add test coverage for RULE_INPUT_DIRNAME being empty to the
rules-dirname test by adding a ne...

Save some time by avoiding "x in list" operations in the dependency
calculations. In a project t...

isinstance is crazily slow. Save another 3 CPU seconds (<1 real second
with 8 cores) by using ty...

We spent a few CPU seconds copying items we deleted a few lines further
down. Seems a bit waste...

[email protected]

Patch by [email protected].

Review URL: https://codereview.chromium.org/2...

After much investigation (linked bug and earlier ps), I don't
really know much other than that t...

The 'msvs_external_builder_clcompile_cmd' logic is preserved, this commit
simply removes the def...

This is in the parallel part so net gain is somewhere between
5/core_count% and 5%. In chromium...

Another O(n^2) over the number of targets which used about half a second
for a 1000 target syst...

With 1000 targets an O(n^2) in the dependency management starts to be
noticed. This patch takes ...