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

Issue 14646033: Fix and simplify unbundled v8.gyp: (Closed)

Created:
7 years, 7 months ago by Paweł Hajdan Jr.
Modified:
7 years, 7 months ago
Reviewers:
Ryan Sleevi
CC:
chromium-reviews
Visibility:
Public.

Description

Fix and simplify unbundled v8.gyp: - use shim_headers.gypi instead of expanding it inline - enable host and target toolsets unconditionally, they're needed as deps (and make no difference anyway) BUG=226860 R=rsleevi@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199063

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -51 lines) Patch
M build/linux/unbundle/v8.gyp View 2 chunks +12 lines, -51 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Paweł Hajdan Jr.
In case you wonder why this patch is separate from the other one: I thought ...
7 years, 7 months ago (2013-05-08 23:38:48 UTC) #1
Ryan Sleevi
lgtm
7 years, 7 months ago (2013-05-08 23:41:36 UTC) #2
Paweł Hajdan Jr.
7 years, 7 months ago (2013-05-09 00:02:38 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r199063 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698