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

Issue 100035: Add gyp file to v8 repository.... (Closed)

Created:
11 years, 8 months ago by Mads Ager (chromium)
Modified:
9 years, 7 months ago
CC:
v8-dev, Erik does not do reviews, M-A Ruel
Visibility:
Public.

Description

Add gyp file to v8 repository. Gyp is used to generate project files used to build chromium. Moving the v8.gyp file to the v8 repository makes it easier to have V8 and the v8.gyp file synchronized (so a DEPS update in chromium gets the right v8.gyp file associated with that revision of V8). Comitted in v8 revision 1795.

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+709 lines, -0 lines) Patch
A tools/gyp/v8.gyp View 1 1 chunk +709 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Mads Ager (chromium)
11 years, 8 months ago (2009-04-27 08:45:41 UTC) #1
Christian Plesner Hansen
Rubber-stamp lgtm. I've looked through the file but probably wouldn't be able to spot errors ...
11 years, 8 months ago (2009-04-27 08:49:37 UTC) #2
Mark Mentovai
11 years, 7 months ago (2009-04-29 16:48:25 UTC) #3
Looks like you didn't set svn:ignore on tools/gyp for the files that will be
generated by GYP from v8.gyp.  This causes generated files to show up as "?" in
"svn status" output, which in turn will cause the try servers to remove the
generated files when they run "gclient revert" to clean up the environment
between try runs.

Powered by Google App Engine
This is Rietveld 408576698