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

Issue 100076: Switching to v8.gyp in the v8 project (so they can maintain it themselves). (Closed)

Created:
11 years, 8 months ago by bradn
Modified:
9 years, 7 months ago
Reviewers:
Mark Mentovai, sgk
CC:
chromium-reviews_googlegroups.com
Base URL:
svn://chrome-svn.corp.google.com/chrome/trunk/src/
Visibility:
Public.

Description

Switching to v8.gyp in the v8 project (so they can maintain it themselves). Rolling forward v8 version to pull in one with v8.gyp. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=14786

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Patch Set 5 : '' #

Patch Set 6 : '' #

Patch Set 7 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+45 lines, -51 lines) Patch
M DEPS View 3 4 5 6 1 chunk +1 line, -1 line 0 comments Download
M build/all.gyp View 1 2 3 4 5 6 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome.sln View 1 2 3 4 5 6 13 chunks +22 lines, -25 lines 0 comments Download
M net/net.gyp View 1 2 3 4 5 6 1 chunk +1 line, -1 line 0 comments Download
M webkit/webkit.gyp View 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/webkit.sln View 1 2 3 4 5 6 10 chunks +18 lines, -21 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Craig Schlenter
You need to change webkit.gyp too I think: http://codereview.chromium.org/99056
11 years, 8 months ago (2009-04-28 16:49:04 UTC) #1
bradn
On 2009/04/28 16:49:04, Craig Schlenter wrote: > You need to change webkit.gyp too I think: ...
11 years, 8 months ago (2009-04-28 20:57:25 UTC) #2
bradn
11 years, 8 months ago (2009-04-28 20:58:04 UTC) #3
sgk
lgtm (module rubber-stamping all the uuids)
11 years, 8 months ago (2009-04-28 21:32:11 UTC) #4
Mark Mentovai
11 years, 8 months ago (2009-04-28 21:32:35 UTC) #5
Looks fine.

Powered by Google App Engine
This is Rietveld 408576698