The default in the VS2013 compiler has been switched to Vista+,
however the runtime library stil...

Use cp -rpf instead of cp -r to copy files. This brings it more in line
with make and ninja whic...

BUG=gyp:379
[email protected]

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

git-sv...

BUG=377
[email protected]

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

git-svn-id...

[email protected]
BUG=309197

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

git-svn-...

Also add a test that this is true for 'copies' too.

BUG=chromium:315383
[email protected]

Re...

Add ibtool support for storyboards. Storyboards also seem to flex an issue
in ibtool where rela...

The msvs generator supports a property called 'resource_include_dirs',
which overrides 'include_...

BUG=377
[email protected]

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

git-svn-id...

moving around.

[email protected]

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

git-...

Update the xcode generator to set the target productType in the pbxproj file to "com.apple.produ...

Fix a few issues with parallel execution:
- exit code 1 on failure
- print a useful backtrace (i...

BUG=
[email protected]

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

git-svn-id: http...

Patch from tjfontaine!

git-svn-id: http://gyp.googlecode.com/svn/trunk@1771 78cadc50-ecff-11d...

Without DTPlatformVersion iOS apps on iOS7 have a weird black bar, and
don't show the snapshot o...

libtool on OS X 10.9 will output message such as:

/Applications/Xcode.app/Contents/Developer/To...

CLANG_WARN_OBJC_MISSING_PROPERTY_SYNTHESIS maps to
-Wobjc-missing-property-synthesis

R=thakis@c...

Use the proper version string, and omit the folder mappings section for
2010 Express.

Patch by ...

BUG=309412
Review URL: https://codereview.chromium.org/30813002/

Patch from James Kozianski <ko...

Caused errors like this:

111>..\build\precompile.cc : fatalerror C1083: Cannot open include fil...

Not sure why try jobs where green.

[email protected]

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

gyp r789 added a filelist feature, where

<(file.foo arg1 arg2 arg3)

would write "arg1 arg2 a...

[email protected]

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

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

[email protected]

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

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

This should have been part of r1755. No intended behavior change.

[email protected]

Rev...

Checks for a flag set in the toplevel dictionary of a file, and if its set, the includes for thi...

It was added in r531 and only used by the (no longer existing) scons generator.

No intended beh...

Also:
Warn for unimplemented code sign keys
(Resource Rules, Entitlements, and Other)
Add partia...

It's not used. It was added in gyp r502 but didn't have an effect back then,
and the now-defunct...

Add CFBundleSupportedPlatforms and UIDeviceFamily for ninja builds.
Required changing ExtraPlist...

When given several input files on the command line in parallel mode,
load them all in parallel i...

[email protected]

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

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

[email protected]

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

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

From https://codereview.chromium.org/24792003, it was identified that
the test case change broke...

If using product extension only in build files, MSVS will complain that the
target extension doe...

For builds that have iOS targets, clone each configuration with an "-iphoneos"
that builds for a...

Patch by Slava Chigrin <[email protected]>.

[email protected]

Review URL: https://c...

The test does not work on xcode.

BUG=
[email protected]

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

Normally, when a target in file1.gyp depends on a target in file2.gyp,
gyp will generate project...

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

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

Since the introduction of r1657, which added library_dirs support for
ninja, the test case did n...

Review URL: https://codereview.chromium.org/24902002/
Patch from David Edelsohn <[email protected]...

Rename sun_tool.py to flock_tool.py and use that on solaris and aix.

Remove apparently-unused S...

Currently, the ninja generator only supports the msvs_disabled_warnings
property. This patch ad...

Patch from Slava Chigrin <[email protected]>!
Review URL: https://codereview.chromium.org/...

BUG=chromium:295322
Review URL: https://codereview.chromium.org/24243013/

Patch from Trent Apte...

This CL allows some flags to be per configuration instead of per target, so
iOS ninja can build ...

During the review for commit r1713 it was determined that ez_setup.py
was not going to be part o...

[email protected]

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

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

Namely, insert BuildMachineOSBuild, DTSDKName, DTSDKBuild, DTXcode, DTXcodeBuild

The values for...

BUG=280718
[email protected]

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

git-svn-...

This matches the behavior of the xcode generator. As sources are generated by
actions and rules...

The order even happens to match the one Xcode chooses.

BUG=280718
[email protected]

Revie...

I misinterpreted what "Bundle products" meant in the "Default value" row
for STRIP_STYLE on
http...

Check link_deps and mac_bundle_depends earlier.

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

tilde, i.e. from \x20 to \x7e. Therefore it matched all non-ASCII characters,
despite this commen...

The core count was a poor abstraction of this. Since we can't easily+accurately model the bottle...

Happening on this Blink bot:
http://chromegw.corp.google.com/i/chromium.webkit/builders/WebKit%2...

- Use vs2013 x64->x86 cross compiler if available
- Add /FS compiler flag which maintains previo...

This will cap the number of links at the min of cpu_count v. mem / 4G.

[email protected]

...

This adds a flag to the ninja generator to specify a file to write library
deps to as generation...

It seems like Chrome (and potentially other projects using gyp) depend on the
bug that was fixed...

The most important part of this change is that LINK will now default to what
is set for CXX (in ...

which is long gone.

[email protected]

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

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

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

Python's decode() function doesn't remove the BOM when decoding 'UTF-16LE' and
'UTF-16BE', so gy...

The LinkDependencies function was used in two places:
1) determining targets whose link_settings...

BUG=chromium:280718
[email protected], [email protected]

Review URL: https://codereview.chro...

Apparently `chmod +x` _after_ running `svn add`ing a file doesn't do the trick.
Maybe `svn props...

Also add a test for various bundle resource scenarious.

BUG=chromium:280718
[email protected]...

Due to r1667, setup.py was left broken installing 'gyp' as a script, which happened to be the *n...

This was added to fix a problem with file renames that rename a file only
in captialization (htt...

r1709 set the output of a target to the stamp file if the target has no
linkable object files. ...

This is to prevent mistakes like r1708 in the future.

[email protected]

Review URL: https://...

I fixed this issue once in https://codereview.chromium.org/9121011/ but added
the test in the wr...

The test runner silently runs gyptest*.gyp files if invoked with a single test,
but doesn't run ...

OS X iconv makes "UTF-16" write UTF-16BE data. Just do the recoding in
Python, which lets "UTF-...

Allow one concurrent ld process for every 4 GB of ram. This matches what
ninja/windows does.

R=...

1704 was reviewed at https://codereview.chromium.org/23461010/

TBR=mark

Review URL: https://co...

Target.UsesToc() already returns False for framework bundles, so don't run
a linker command that...

No code changes needed for app bundles. (Framework bundles need a bit of work
to make this true,...

[email protected]

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

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

This method is used to determine the size of the Ninja pool
which is used for links during the b...

Example:

gyp: Cycles in .gyp file dependency graph detected:
Cycle: build/linux/system.gyp -> t...

Enable the code on Win/Mac as well as Linux.

[email protected]

Review URL: https://codere...

Rather than using optimization flags to test cflags,
use our own explicit defined flag. This is...

Follows behavior when using MSVS generators. If target ends up with no
'Optimization' option spe...

http://build.chromium.org/p/tryserver.nacl/builders/gyp-linux/builds/1119

assertItemsEqual is n...

Example:

gyp: Cycles in .gyp file dependency graph detected:
Cycle: build/linux/system.gyp -> t...

The change in r1691 caused us to add environment variables to
host and target toolsets. We shoul...

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/...