Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(306)

Issue 10408016: Adding native_client/build to the gyp module include path. (Closed)

Created:
8 years, 7 months ago by bradn
Modified:
8 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Adding native_client/build to the gyp module include path. This will allow nacl's scan_sources.py script to switch to using the pymod_do_main feature in gyp to: - reduce runtime - circumvent command line length limits on windows BUG=None TEST=None R=noelallen@chromium.org,bbudge@chromium.org,mseaborn@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=137783

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M build/gyp_chromium View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
bradn
8 years, 7 months ago (2012-05-18 00:02:46 UTC) #1
noelallen1
lgtm
8 years, 7 months ago (2012-05-18 00:31:55 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bradnelson@google.com/10408016/1
8 years, 7 months ago (2012-05-18 00:33:26 UTC) #3
commit-bot: I haz the power
8 years, 7 months ago (2012-05-18 00:33:28 UTC) #4
Failed to apply patch for build/gyp_chromium:
While running patch -p1 --forward --force;
patching file build/gyp_chromium
Hunk #1 FAILED at 22.
1 out of 1 hunk FAILED -- saving rejects to file build/gyp_chromium.rej

Powered by Google App Engine
This is Rietveld 408576698