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

Issue 11412230: mac: Get revision information from LASTCHANGE like windows and linux do (Closed)

Created:
8 years ago by Nico
Modified:
8 years ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews
Visibility:
Public.

Description

mac: Get revision information from LASTCHANGE like windows and linux do BUG=161488 TEST=Check that Chromium.app/Contents/Info.plist still gets the right SCMRevision after syncing and building. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170085

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -7 lines) Patch
M build/mac/tweak_info_plist.py View 4 chunks +11 lines, -7 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Nico
After this, tweak_info_plist.py needs to grow explicit dependencies on chrome/tools/build/version.py, build/util/LASTCHANGE, and chrome/VERSION and then ...
8 years ago (2012-11-28 21:51:50 UTC) #1
Mark Mentovai
LGTM
8 years ago (2012-11-28 21:52:33 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/11412230/1
8 years ago (2012-11-28 21:53:27 UTC) #3
commit-bot: I haz the power
8 years ago (2012-11-28 22:25:22 UTC) #4
Sorry for I got bad news for ya.
Compile failed with a clobber build on win.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

Powered by Google App Engine
This is Rietveld 408576698