Now that ARCHS is supported, someone might try to use FOO[arch=i386], that
shouldn't fail silent...

The basic design is that actions, rules, and copies run exactly like before
(only once per targe...

- CFLAGS
- CXXFLAGS
- LDFLAGS

BUG=251122, gyp:360
[email protected], [email protected]

Re...

This way, all branches in WriteTarget() write self.target.binary, which is a
bit more regular.

...

For fat binary support on mac, compile and link lines will need different
flags for every arch (...

"variables" is only used on mac and linux in this block, so don't declare it
for other platforms...

Refactor the "only write a postbuild if needed" bit into a new function.

No functionality chang...

ComputeOutput() does the right thing for bundles, so don't have a distinct
method for bundles. (...

Because the latter doesn't exist.

BUG=none
[email protected]

Review URL: https://coderevie...

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

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

This is problematic when the generator output is set to a different drive
from the gyp files and...

* $(TargetDir) is equivalent to $(OutDir)
* $(TargetName) logic is taken from the logic that set...

This is needed for projects that want to use ARC
(http://clang.llvm.org/docs/AutomaticReferenceC...

[email protected]

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

git-svn-id: http:/...

[email protected]
BUG=

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

git-svn-id: h...

[email protected]

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

git-svn-id: http:/...

Including <winuser.h> directly caused the following warning on VS2010:

string.h(54) :...

Copy of VS 2012. This code is increasingly icky.

[email protected]

Review URL: https://cod...

[email protected]

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

git-svn-id: http:...

Since these rules now use "deps = msvc", the depfile attribute is not used.

R=scottmg@chromium....

[email protected]

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

git-svn-id: http:/...

Should fix -regen tests on Mac under make.

[email protected]

Review URL: https://coderevie...

Tests fail on the bots, see http://crbug.com/154205

[email protected]

Review URL: https:/...

[email protected]

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

git-svn-id: http://...

After r1667.

[email protected]

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

git...

(causing tests to fail currently)

[email protected]

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

Previously gyp.bat runs "python gyp" and "gyp" imports "gyp" expecting to
get pylib/gyp/__init__...

Setting GYP_PARALLEL=0 on posix will turn off parallel gyp.

BUG=chromium:154205
R=dmazzoni@chro...

No intended functionality change: They are only used for targets with a toolset
of 'host', which...

$libdirs was always in front of $libs, so there's no need for a second variable.
If $libdirs is ...

BUG=gyp:257
TEST=python gyptest.py -a -f xcode,make

Patch by Jan Svarovsky <[email protected]>
R...

Use `ExpandSpecial(join('$!PRODUCT_DIR', path))` instead of
`join(ExpandSpecial('$!PRODUCT_DIR')...

No intended functionality change:
These variables are empty in the majority of targets, they are...

[email protected]

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

git-svn-id: http:/...

This is so that when the .def file is modified, a relink will be triggered.

R=scottmg@chromium....

EscapeXCodeDefine, changed escaping method for spaces, added test.

BUG=gyp:257
TEST=Run small te...

was never actually implemented.
library_dirs is essentially equivalent to adding "-Lfoo" to ldfl...

Otherwise, the gyp binary won't be found when trying to regenerator the output
files

R=thakis@c...

BUG=gyp:321
[email protected], [email protected]

Review URL: https://codereview.chromi...

Support building multiple configurations in the Android backend (by
setting GYP_CONFIGURATION wh...

As a follow up to r1618, these are some changes that make the generated msvs
projects easier to ...

[email protected]
BUG=

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

git-svn-id: h...

[email protected]
BUG=

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

git-svn-id: h...

When AdditionalDependencies! and DelayLoadDLLs! directives are used e.g.
in chrome_dll.gypi, war...

Per
https://groups.google.com/forum/?fromgroups#!topic/gyp-developer/OviErZ6vR1A
move 2008 to 20...

flock not used on ninja now in favour of pool functionality.

[email protected]

Review URL:...

already enabled on non-windows. keeps use_deps=0 available as opt-out.

[email protected]
BU...

This uses the ninja "pool" feature to limit the number of concurrent
links to a reasonable numbe...

This test failed on Windows with ninja 1.3.3 but
should pass with ninja 1.3.4 which has
https://...

We landed ninja 1.3.4, which should have fixed crbug.com/245034. Trying this
again.

BUG=chromiu...

The xcode gyp generator has no issue with this, but the msvs, make, and ninja gyp generators hav...

See crbug.com/245034

TBR=scottmg
Review URL: https://codereview.chromium.org/15927010/

git-s...

Windows will follow soon, if the new 1.3.0.git binary has worked well for a week
or so.

The Ope...

build/common.gypi).

Also revert r1638 since it's no longer needed.

Add tests for default variab...

Without this fix,

cd test/variables/commands && ./update_golden && cd - && \
./gyptest.py tes...

With r1637 default varaibles used as paths are now being relativized.
However, if the default va...

Variables set as defaults with the "foo%" syntax were not being
relativized because the % was be...

$(gyp_intermediate_dir) is sometimes an absolute path, and so trying to
convert it from relative...

Workaround for environment vars being uppercase on Windows Python.

Quote the path to cl.exe in ...

We were using the wrong variable in the Android stub version of
run_built_executable. This doesn...

Let it use deps=msvc, but still use -t msvc to set up the environment.

BUG=chromium:232421
TBR=...

Rules that generate multiple output files (like protoc) need the same
workaround as actions; the...

Target iphonesimulator6 or 5 aren't on the bots.
Review URL: https://codereview.chromium.org/149...

Ninja was defaulting to building the mac desktop bundle structure. Added a way
to build the iOS ...

OpenBSD's binutils doesn't support thin archives. The make generator
backend already knows to a...

for finding include.gypi (and other configuration options).

The user may also set GYP_CONFIG_DIR...

NormalizeLdFlags existed to filter out various linker arguments that
don't belong in the Android...

git-svn-id: http://gyp.googlecode.com/svn/trunk@1624 78cadc50-ecff-11dd-a971-7dbc132099af

Remove keys with associated values that have not been replaced by
environment variables. This du...

BUG=chromium:236517
TEST=Added test to non-strs-flattened-to-env and postbuilds
Review URL: http...

msvs_external_builder is MSBuild-specific and so fails on 2008. Fixes currently red win32/64 bot...

Keep "cc = cl.exe" around because c99conv uses it to try to find what it
should use to preproces...

git-svn-id: http://gyp.googlecode.com/svn/trunk@1619 78cadc50-ecff-11dd-a971-7dbc132099af

This change allows plugging in external builders to the
msvs projects. This is activated by def...

git-svn-id: http://gyp.googlecode.com/svn/trunk@1617 78cadc50-ecff-11dd-a971-7dbc132099af

(More fallout from r1612)

[email protected]

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

[email protected]

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

git-svn-id: http:...

- Don't generate windows environment files on non-Windows
- Don't use `where` to locate cl.exe -...

The scons generator has not been used for some time and is known to no longer
be at parity with ...

Part of upgrading to ninja trunk with more efficient dependency
tracking.

Does path search for ...

For someone, `xcodebuild -version -sdk macosx10.6 Path` would print

2013-04-12 17:18:59.803...

This is supposed to be used for letting a couple of folks opt in to this new
feature (which requ...

This is ninja_syntax.py as of ninja commit
368f5bc64ddd2dcea2662d6c932157314be3e09b . It's newer...

Add a human-readable error if a gyp file doesn't evaluate to a
dictionary. I lost about 30 minu...

This came up while bringing up the components build for Android. Trying
to run v8_shell failed: ...

This broke in r1595. The old logic was to compute a path for anything that
began with "macosx". ...

Filter out "No recorder, buildTask: <Xcode3BuildTask:" lines from stderr
when using Xcode. They ...

The make generator does not keep track of which targets are shared
objects and instead relies o...

The target_list has to be in topological order (any dependencies come before those targets that ...

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

git-svn-id: http://gyp.googlecode.com/svn/t...

This undoes the change made in https://codereview.chromium.org/12389082 .

I have tested this on...

This creates a workaround for targets whose PDB sizes exceeds 1GB, using the same trick original...

Two tests currently fail with
error: can't exec '/Developer/Library/Xcode/PrivatePlugIns/Xcode3C...

TBR=mark
Review URL: https://codereview.chromium.org/12916006

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

Fallout from r1594.

TBR=mark
Review URL: https://codereview.chromium.org/12901002

git-svn-id: ...

Not sure if 4.2 can target 10.7, so maybe it won't work.

TBR=mark
Review URL: https://coderevie...

+ Improved SDK path lookup: now also supports iPhoneOS and iPhoneSimulator SDKs
+ Adds support f...

Changes:
* cc is now clang, not gcc
* move sdk to 10.7
* make an error message slightly less spe...

Gyp-mode uses forward-sexp which can be very slow on Emacs 24.3.50.1,
making it appear that Ema...

BUG=none

Patch by Pawel Hajdan <[email protected]>
Review URL: https://codereview.chromiu...