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

Issue 115869: Moving things a little so this will work when chrome.sln gets eaten by gyp. (Closed)

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

Description

Moving things a little so this will work when chrome.sln gets eaten by gyp. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=17117

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+311 lines, -304 lines) Patch
M chrome/app/locales/locales.gyp View 1 3 chunks +310 lines, -304 lines 0 comments Download
M chrome/chrome.gyp View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
bradn
11 years, 6 months ago (2009-05-28 18:40:31 UTC) #1
sgk
lgtm, but let's use variables for the other paths, too. http://codereview.chromium.org/115869/diff/1/2 File chrome/app/locales/locales.gyp (right): http://codereview.chromium.org/115869/diff/1/2#newcode45 ...
11 years, 6 months ago (2009-05-28 18:52:28 UTC) #2
sgk
11 years, 6 months ago (2009-05-28 20:06:32 UTC) #3
sweet, lgtm

Powered by Google App Engine
This is Rietveld 408576698