Signed-off-by: Uilian Ries <[email protected]>

Signed-off-by: Uilian Ries <[email protected]>

Fix OSX version validation

Signed-off-by: Uilian Ries <[email protected]>

Fix msvc support on Windows

Signed-off-by: Uilian Ries <[email protected]>

Fix msvc and osx support for python3

Signed-off-by: Uilian Ries <[email protected]>

https://bugs.chromium.org/p/gyp/issues/detail?id=36
https://chromium-review.googlesource.com/c/ex...

Rietveld CQ has already been disabled and is no longer supoorted.

No-Try: True
Bug: chromium:77...

[email protected]
NOTRY=true

Change-Id: I962a0cd6b4d00014851120d578dd00dfd9200dcc
Reviewed...

Some customers might not want to upgrade from python 2.7.6
to 2.7.13. So when sys.platform retur...

Seems that the initial z/OS commit was missing this rather
important argument to the link comman...

Older python releases returned sys.platform as "os390".
The latest version returns "zos" which s...

Currently CPPFLAGS_host, CFLAGS_host and CXXFLAGS_host are respected by
the make and ninja gener...

Change-Id: Iad8dea85047e5f2f4b6623717fb56b268810216d
Reviewed-on: https://chromium-review.google...

When building with Visual Studio 2017, gyp may fail with a
non-descriptive message if Windows ha...

Bug: https://github.com/nodejs/node-gyp/pull/1203
Change-Id: I30d71a2bb3d4b09e7bd9409c3c45c32bd1...

Without check_output() the Popen() will always return None as returncode, this
prevents error ha...

* also "fix" the paths in the message
* un-skip test/copies/gyptest-all.py

BUG=gyp:536

Change-...

A host-native executable will always be used, and it will be a cross
compiler if the target arch...

BUG=683729

Change-Id: Ic8c227960b859ddc3c19fce0e98144510f5e74bf
Reviewed-on: https://chromium-r...

[email protected]
BUG=536

Change-Id: Icced5ec1d10e45231d9d73d11aad2e2339f2d21c
Reviewed-...

* add compatibility with VS2017
* adjust `_TargetConfig` and `/FS` for VS2017 compat
* find new ...

This change removes a lot of unneeded complexity
from gyptest.py, and changes the output format
...

It looks like the xcode-ninja generator is at
least partially broken on 10.12/Xcode8.2, plus
a f...

This updates the test to indicate which toolchains
and generators are properly computing whether...

1) Be aware of Microsoft Visual Studio 2017 (ver 15.0 / toolset v141)
2) Try to detect from regi...

Allow build to fail if WindowsKernelModeDriver not installed (indicates GYP run correct)

Change...

Goal is to ensure that unreviewed and untrusted code isn't sent through
CQ to try bots, by accid...

BUG=682746

Change-Id: I9d51f0d5e7186e9aa5faf4ae2758e32ab5785a12
Reviewed-on: https://chromium-r...

BUG=682746

Change-Id: I37d5018820197f5dcdbcd0904caa251cd8d62f14
Reviewed-on: https://chromium-r...

Based on https://crrev.com/2492233002#ps1) from [email protected].

[email protected], sxa@uk....

This change makes the msvs_target_platform_{version,minversion} settings
useful even when msvs_e...

Hash the intermediate Makefile target used for multi-output rules
so that it still works when th...

In "gyp/gyptest.py", shlex is used, but it is not imported. So it
will throw NameError at runtim...

* Modify GYP to set the PlatformToolset, the DriverType and the TargetVersion
* Add msvs_target_v...

If you have a shared library that has no sources, but links a static library that contains C++ s...

Add the remaining destination-related Xcode build setting names to the
list in class PBXCopyFile...

The :rfc1034identifier substitution converts invalid characters to '-'
because an underscore is ...

When convert_to_binary is True, there is no point converting the .strings
file encoding to UTF-1...

BUG=
[email protected]

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

Xcode expects the default SDKROOT to be iphoneos, while gyp-ninja has been
expecting a default v...

BUG=597459
[email protected]

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

This change enables someone using gyp to specify the CompileAsWinRT option
(i.e. use the /ZW com...

See: https://developer.apple.com/videos/play/wwdc2015/406/

[email protected]

Review URL: h...

When testing on device, Xcode xctest's are copied into $TEST_HOST with two
special frameworks.

...

Ensure that capacity is always the next power of two over count. This
fixes a bug where capacity...

BUG=495204
[email protected]

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

BUG=495204
TBR=scottmg

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

BUG=495204
[email protected]

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

As "plutil" does modify the file in place, using hardlinks cause files
in the checked out tree t...

Remove the destination file if it exists as os.link fails with
"[Errno 17] File exists" if the d...

Hard-link are cheaper than copying the whole data, so if possible use
them but fallback to copy ...

Adds DEVELOPER_DIR to gyp mac_tool.py, to LINK commands in ninja
and to the default Xcode emulat...

VC++ 2015 defaults to *not* installing the C++ toolchain. Several
developers have failed to noti...

iOS frameworks have a different output structure with normal
frameworks, and often require heade...

Current rpath is hardcoded to $ORIGIN/lib. Allow the user to use
a custom one through a new targ...

[email protected], [email protected]

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

Before this change, we passed /FIprecompile.h but build isn't on the include
search path. This ...

This is similar to what was done in the Xcode generator in revision 877
(AKA git commit bc09226,...

Xcode 7 prints "Beginning test session" to STDERR.

Also removes xcode-ninja test from xcode gcc...

Allows codesigning iOS apps from login keychain in gyptest.

BUG=
[email protected]

Revi...

BUG=
[email protected]

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

This fixes an issue with old (10.6 era) Xcode versions.

BUG=577711
[email protected]

Revie...

Update Xcode emulation extra plist items for Xcode 7.2

Adds DTCompiler, MinimumOSVersion and D...

This reverts commit 51383232066a9180bf88cb24a7401258c98efb2d.

BUG=
[email protected]

Rev...

Adds DTCompiler, MinimumOSVersion and DTPlatformBuild.

Corrects DTSDKName and DTSDKBuild.

Als...

Adds --auto-activate-custom-fonts, --target-device and
--minimum-deployment-target to ibtool arg...

This reverts commit 18ce4ecf6cf2498757b3b61b438d8a4f1ef9e33a.

BUG=
[email protected]

Rev...

If you have DEPOT_TOOLS_WIN_TOOLCHAIN=0 and you have run vcvarsall.bat
(VS 2013 or 2015 versions...

Adds an extra flag for ibtool when compiling xib and storyboard
files.

BUG=577711
R=thakis@chro...

Xcode 9 sets the entrypoint for today extensions with -e _NSExtensionMain, and
no longer needs t...

Keep gyp codesign flags in line with Xcode.

BUG=577711
[email protected], thakis@chromiu...

BUG=577711
[email protected]

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

This adds enough support for Mac to the CMake generator to build simple
targets. This allows for...

Previously only Info.plist and .strings files with convert_to_binary were
ran thru plutil to con...

It seems ResourceRules.plist is no longer necessary in codesign.

BUG=577711
[email protected]...

Currently, the GYP build is non-deterministic: See issue 557851 for
discussion of how this came ...

We used to not use xcrun because it used to not exist long ago. Nowadays,
xcrun should be prese...

[email protected]

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

[email protected]

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

Now also matches lines such as

path/to/libtool: for architecture: x86_64 file: foo.a(bar.o) has...

The other generators I tested (make, xcode) all get this right already.

BUG=chromium:552692
R=s...

Update so XCTests are properly built by ninja, after generating
the projects with gyp. This incl...

Occasionally developers hit this cryptic error:

Exception: Environment x variable "SYSTEMROOT" ...

Patch from [email protected].

[email protected]

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

The function CalculateGeneratorInputInfo() was broken during patchset #2
of https://codereview.c...

Configure generator_filelist_paths used by main gyp input parse to build
path of "<|()" expansio...

BUG=
[email protected]

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

BUG=
[email protected], [email protected], [email protected]

Review URL: https...

[email protected]
BUG=chromium:552146

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

In order to run test_targets we have to make sure they are actually
built.

BUG=557505
TEST=cove...

If no targets were found analyze would incorrectly write an empty
invalid_targets list, which ca...

Analyzer now gets two inputs:
. test_targets: targets to search from.
. additional_compile_targe...

I'm going to add support for 'all'. This is a precursor to that. This
patch effectively adds bac...

[email protected]

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

Patch from Alex H...

In projects with just one CMakeLists.txt, these two variables
are equivalent. In multiproject bu...

Causing failure in Chrome per https://codereview.chromium.org/1416453003/#msg8

(SetXcodeVersion...