Граф коммитов

106 Коммитов

Автор SHA1 Сообщение Дата
scottmg@chromium.org 6dac171ec6 add temporary debug prints to debug builders
git-svn-id: http://src.chromium.org/svn/trunk/src/build@252748 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-22 04:52:15 +00:00
scottmg@chromium.org 74bdf159c5 Re-enable 2013 with corrected hashes
TBR=iannucci@chromium.org
BUG=323300,309197

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@252738 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-22 03:24:51 +00:00
scottmg@chromium.org 6c16694602 temp disable 2013 switch
git-svn-id: http://src.chromium.org/svn/trunk/src/build@252736 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-22 03:03:13 +00:00
scottmg@chromium.org 9d11907524 Move control of updating toolchain into src/
Moved out of depot_tools here: https://codereview.chromium.org/168603004/

The control is in src/ now, but logic for de-duplication is still in depot_tools.

This changes the default, so that VS2013 will be the default toolchain.

R=iannucci@chromium.org
BUG=323300,309197

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@252733 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-22 02:41:37 +00:00
thakis@chromium.org 642050daf9 Let gyp_chromium set GYP_CROSSCOMPILE for android and ios.
Then envsetup doesn't need to set it, and the chromium.gyp_env files
used in the ios build no longer has to set this either.

BUG=330631
R=scottmg@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@252649 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-21 21:27:24 +00:00
scottmg@chromium.org dbecf0d182 Don't use automatic toolchain if GYP_MSVS_VERSION overridden to another version
This accomplishes three things:
- simple switch back to 2010 temporarily, if necessary
- making sure that all bots don't switch at exactly the same time (almost all have GYP_MSVS_VERSION=2010 in their settings)
- in particular, not having Official builders switch

Unfortunately, this will need to be merged on to the 34 branch, as it has the current "auto-use" code, so otherwise it'd switch to 2013 as soon as depot_tools starts updating the toolchain (previously, the only check was whether the toolchain exists on the local hard drive).

R=maruel@chromium.org
BUG=323300

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@252604 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-21 18:44:41 +00:00
mattm@chromium.org 69dd7501d1 Win automatic toolchain: copy runtime dlls properly when not run from chrome/src.
gclient runhooks runs gyp_chromium from the chrome/ dir, but the script
was looking for out/ relative to that.

BUG=323300

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@252500 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-21 05:14:18 +00:00
saiarcot895@gmail.com 4276ee8fd8 Move the "import find_depot_tools" line to sections that actually use it.
This shrinks the dependency of depot_tools to only Windows and Cygwin, and allows those on other OSes to just build tarballs of sources without also having the depot_tools.

BUG=none

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@252434 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-21 00:30:29 +00:00
thakis@chromium.org 007dfc2325 android envsetup: Stop honoring --target-arch parameter.
Most people don't use this parameter and get arm binaries. This is
still the default, so this change shouldn't affect most people.

Folks should instead pass -Dtarget_arch to gyp. (Or, soon, envsetup will stop
clobbering GYP_DEFINES, then you can just add target_arch to your GYP_DEFINES.)
Note that in gyp land, 'mips' is called 'mipsel' and 'x86' is called 'ia32'.
'arm' stays 'arm'.

So for example, instead of running

  . build/android/envsetup.sh --target-arch=mips
  android_gyp

you'd run

  . build/android/envsetup.sh
  android_gyp -Dtarget_arch=mipsel

I updated the bots I was able to find to pass the -D flag in addition to
--target-arch. After this CL here is in, I'll update the bots to stop passing
--target-arch, and then I'll make --target-arch a hard error in this script for
a while, to make sure nobody still uses it.

BUG=330631,34476
R=torne@chromium.org
TBR=yfriedman

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@252034 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-19 18:20:22 +00:00
scottmg@chromium.org 6028894fc4 Fix automatic toolchain when msvs-ninja in GYP_GENERATORS
Previously, we checked that GYP_GENERATORS was "ninja", but this
broke when using GYP_GENERATORS=ninja,msvs-ninja. As there are no
other supported genearators than those two now, there's no need
to check that we're using ninja at all, so just remove that
branch of the if.

BUG=323300
R=jschuh@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@251406 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-14 20:10:43 +00:00
brettw@chromium.org 771e71e45a GN iOS build fixes.
This properly sets the DEPLOYMENT_TARGET in the generated GYP file (previously this value wa salways hardcoded to empty -- whoops!).

Sets the simulator properly. Previously when not using an explicit SDK the Xcode generator would still end up with a simulator build. This turns the flag into a tri-state so gyp_chromium can set it explicitly when an SDK is specified, but otherwise we can trigger default behavior based on Xcode/Ninja.

TBR=scottmg

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@249643 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-07 09:02:37 +00:00
scottmg@chromium.org 7aef8e0121 Move landmines into gyp_chromium
This ensures that the environment that's passed to gyp is the one that
landmines.py uses to determine when to set landmines.

Also, fix '2013e' not being detected as a 2013 switch.

BUG=309197, 323300
R=iannucci@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@249242 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-06 02:44:55 +00:00
scottmg@chromium.org d278b247d8 Copy VS runtime DLLs to output directory
When using the automatic toolchain for vs2013, they won't necessarily
be system-installed, so copy them next to the output binaries.

R=dpranke@chromium.org
BUG=326357

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@248790 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-04 22:11:57 +00:00
scottmg@chromium.org 3719dcde20 Support Express for 2013 automatic toolchain
Pick up setting for Express 2013 to get WDK_DIR set (required
for ATL with Express).

Add additional "or 2013e" to build/common.gypi, but exclude
unneeded workaround for intsafe.h.

R=dpranke@chromium.org
BUG=323300, 340358

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@248747 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-04 17:50:01 +00:00
brettw@chromium.org f902545676 Work on GN iOS build.
Separates out some "SDK" related setup on Mac land Linux like we do on Windows. I'm trying to make the "compiler" BUILD file just compiler warnings and CPU options, and have system library stuff on the platform directories.

This adds the capability for GN to produce GYP files on Mac that vary according
to the GYP generator as well as target-vs-host. I added a bunch of logic to the
GN iOS build to set up stuff accordingly based on my current knowledge of
what's required.

Sadly, this means we now have an 8-way GN build (all combinations of
debug/release, host/target, and xcode/ninja). I did some refactoring of the GYP
code in GN to make this less unreasonable.

I checked that the GYP files look the way I want, but I didn't actually test
the resulting builds yet. There is still likely to be some conditions wrong or
things not being set properly. I'm going to follow up with a second pass based on actual testing.

I believe, however, that with this new GYP generator code in GN, we can express
in the .gn files what we need to do the iOS build.

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@248476 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-03 12:04:33 +00:00
thakis@chromium.org 123a18e910 gn/gyp: Escape \ in addition to $ and " in strings.
BUG=340055
R=halyavin@google.com
TBR=brettw@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@248410 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-02-02 16:17:12 +00:00
thakis@chromium.org 588ee2e9f5 Support joined -D flags for gn.
BUG=326230
TBR=brettw@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@246558 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-23 09:55:45 +00:00
thakis@chromium.org 931bc52341 gn: Pass through msan flag.
BUG=none
TEST=GYP_DEFINES=msan=1 build/gyp_chromium works.
TBR=brettw@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@246188 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-22 00:47:39 +00:00
jamesr@chromium.org 55a88f7f1b Support -Goutput_dir=blahblah in GN-GYP hybrid mode
This supports specifying the output_dir generator flag in the gn-generating-gyp
mode. The value of the output_dir flag is mapped to a gyp variable called gyp_output_dir
and a GN argument by the same name. References from gyp to gn-generated gyp files must
use this variable and references from BUILD.gn files must use the gn argument.

BUG=335760
TBR=brettw for build/config/BUILDCONFIG.gn (looked at in person)

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245800 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-18 23:51:41 +00:00
thakis@chromium.org aa5d793c46 Escape 'gomadir' for GN.
No real functionality change (except for people who have " or $ in their gomadir,
but since that didn't work up until this change nobody does that).

R=brettw@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245799 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-18 23:50:25 +00:00
brettw@chromium.org d3ce57c03f Add support for the arm_float_abi flag to the GN build.
BUG=

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245765 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-18 09:06:50 +00:00
pkasting@chromium.org 030cdf69de Switch gyp_chromium from third_party/python_26, which is gone, to
depot_tools/python276_bin.

BUG=335180
TEST=gclient runhooks works again on Cygwin
R=maruel@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245705 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-18 01:29:57 +00:00
brettw@chromium.org d34dce13b5 Add support for Dr memory and Windows tsan in the GN build.
This generalizes the GCC iterator debugging flag to now be platform-independent.

R=scottmg@chromium.org
TBR=scottmg

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245665 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-18 00:00:08 +00:00
brettw@chromium.org b3af3ccb35 Hook up clang_use_chrome_plugins to the GN build.
Previously this was ignored in the GN build but disabling the plugin is used on some buildbots.

BUG=http://crbug.com/330298
R=thakis@chromium.org, thakis

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245602 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-17 20:34:06 +00:00
brettw@chromium.org 8972c57d08 Hook up gomadir in the GN build.
Without this, GYP will fail if you manually specify gomadir in your GYP_DEFINES.

R=jamesr@chromium.org, jamesr

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245595 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-17 20:20:27 +00:00
brettw@chromium.org 8ad37c4b4e Hook up the tsan flag for the GN build.
Add a flag to disable GLIBCXX_DEBUG which is used in the GYP build on some bots.

TBR=scottmg

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245577 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-17 19:05:45 +00:00
scottmg@chromium.org 97d412978d win: Find and use toolchain installed by depot_tools
The toolchain directory inside depot_tools will only exist if
"set DEPOT_TOOLS_WIN_TOOLCHAIN=1" so this is currently a no-op.

It will break the "dogfood" of 'GYP_MSVS_USE_SYSTEM_TOOLCHAIN'
now renamed per above, but I think only a few people tested
that (cc'd here).

R=dpranke@chromium.org
BUG=323300, 326357

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245412 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-17 01:40:31 +00:00
brettw@chromium.org db4636728a Use ~/.gyp/include.gypi in GN build.
This also fixes the goma toolchain definition. My conditions syntax was missing a set of [].

R=scottmg@chromium.org
BUG=

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@245371 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-16 23:17:30 +00:00
scottmg@chromium.org e7db2b33d4 Move RunGN lower so that modifications to os.environ take effect
Without this the os.environ changes for various default values are
not taken into effect by GN.

R=brettw@chromium.org
BUG=323300

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@244963 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-15 19:45:29 +00:00
thakis@chromium.org 862b84ed2d android: Set gcc_version in build/common.gypi instead of in envsetup.
BUG=330631
R=torne@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@244615 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-14 00:14:49 +00:00
nick@chromium.org 8e2786ef74 Fix handling of visual studio override when using automated toolchain.
BUG=

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@243845 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-09 11:40:38 +00:00
scottmg@chromium.org b645fc64eb win: Use ninja by default
With this change, `gclient runhooks` creates ninja files by default on
Windows. To build, run `ninja -C out\Release chrome`. The output appears
in out\ instead of build\.

If you need to use a VS build for some reason, you can still
`set GYP_GENERATORS=msvs`. This is no longer a tested configuration.

Bots should automatically pick up the binaries in the new location (out).
If something does break, it should be safe to revert this change, and bots
should then automatically go back to picking up binaries from the old
location (build).

R=thakis@chromium.org
BUG=303291

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@243782 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-09 05:08:41 +00:00
brettw@chromium.org cd32cb2c96 Work on Android GN build.
Adds arm version and some android configuration build flags.

This adds most of the logic from common.gypi to the Android GN build.

This is currently missing the crtbegin/end stuff and won't actually make real Android builds. The logic in this patch is just the initial conversion that will require testing and several more passes of fixes.

R=torne@chromium.org, torne

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@243681 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-08 22:47:42 +00:00
brettw@chromium.org 384477bd44 Remove extra print statemt from GYP.
I accidentally checked this in.

TBR=scottmg

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@243618 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-08 18:59:48 +00:00
brettw@chromium.org d26952a6e6 GN: Autodetect Visual Studio versions
This searches the local system for Visual Studio versions like GYP. It enables specifically requesting one version, as well as overriding the directory like GYP.

BUG=
R=scottmg@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@243612 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-08 18:46:44 +00:00
brettw@chromium.org ad0ae7286a GN Command line -D parsing, goma on Windows
This makes gyp_chromium pass defines defined via -D on the command line to GN.

Fixes double-escaping of strings from supplemental.gypi files.

Pass use_goma to the GN build.

Handle "use_goma" and "use_goma=1" (same for other boolean values). Previously values with no equals sign were getting converted to 'true', which doesn't match '1' which we look for.

Output goma wrappers in the GYP header for Windows toolchains.

R=scottmg@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@243446 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-08 00:32:04 +00:00
petarj@mips.com 288464dda0 [MIPS] Add support for Chromium-Android for MIPS build to BUILD.gn
Fix build regressions for Chrome for Android for MIPS that came with r242674.

BUG= gclient runhooks fails for MIPS
TEST= build content_shell for Android MIPS

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@242797 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2014-01-02 21:15:53 +00:00
brettw@chromium.org e8cf6ff9f4 Add GN conversions for flags
Adds a remapping from important GYP flags used on the main waterfall for GN. For the other ones, I added a list of used flags and filed bugs for them.

R=scottmg@chromium.org
BUG=

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@242356 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-23 05:11:32 +00:00
brettw@chromium.org 3a9eea075c Convert clang=1 to the GN build.
This maps clang=1 in GYP to is_clang=true in GN.

BUG=
R=scottmg@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@242181 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-20 21:37:43 +00:00
brettw@chromium.org 0a09a4a4fb Don't double-quote strings when passing them into GN args.
Previously, both EscapeStringForGN and GetArgsStringForGN would add quotes.

BUG=
R=finnur@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@241075 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-17 00:02:45 +00:00
brettw@chromium.org 8ededa0c74 Hook up GN official build and branding.
This passes the official build and branding flags to the GN build. Currently, this is used to hook up the proper sysroot for compiling on Linux for official builds. The 32-bit branded build does not compile currently due to pending pkgconfig issues.

Fix a bug where assertions in the build config would not get reported properly. This was because I forgot to actually throw the error at the toplevel.

BUG=
R=piman@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@240918 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-16 17:58:16 +00:00
brettw@chromium.org 1e1b1c6294 Move all GYP->GN argument conversion to Python.
This moves build variable remapping from C++ to the Python wrapper. This will be much more scaleable since we can add stuff to the script atomically without rolling out new binaries.

R=scottmg@chromium.org, scottmg

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@240732 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-13 20:28:08 +00:00
scottmg@chromium.org 61e8b4e567 Include VS2013 runtime folders in PATH for automatic toolchain
Reproduced on a new Win7 x64 VM, and this fixed the build (at
least for the chrome target).

This doesn't address the need to copy the runtime DLLs into the
output directory so that chrome runs though, that's a separate
problem where something in chrome's gyp should be copying them
there, at least for a debug build.

TBR=mikelawther@chromium.org
BUG=326345

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@239671 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-10 05:32:16 +00:00
brettw@chromium.org 1ee17157e6 Re-enable GN as part of GYP execution.
BUG=288991
R=scottmg@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@239254 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-06 20:35:07 +00:00
brettw@chromium.org c32f2dc9b3 Treat cygwin the same as Windows when selecting the GN binary.
R=maruel@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@239028 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-05 20:35:31 +00:00
thakis@chromium.org 5800e20e83 Disable GN call, it breaks the win perf builder.
BUG=326146
TBR=brettw@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@238982 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-05 16:55:13 +00:00
thakis@chromium.org 0aa98a93d1 gyp_chromium: Escape " for gn files.
BUG=326024
TBR=brettw@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@238883 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-05 03:51:46 +00:00
thakis@chromium.org 2bfd88203a Fix build/gyp_chromium for supplement.gypi files with no variables.
BUG=325989
TBR=brettw@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@238859 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-05 02:11:23 +00:00
brettw@chromium.org d18ac89f2f Fix handling of GYP defines from supplemental files.
This adds a call to the escaping function for variables from supplmental files.

R=thakis@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@238792 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-04 22:11:56 +00:00
brettw@chromium.org 7bbadce4c1 Re-enable GN in the GYP build.
This fixes Android GYP defines by more conservatively rewriting all key
characters that aren't alphabetic.

BUG=321352
R=thakis@chromium.org

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

git-svn-id: http://src.chromium.org/svn/trunk/src/build@238752 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2013-12-04 20:28:10 +00:00