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

Issue 115474: Rename official builds to "Google Chrome" on disk and "Chrome" in the menubar... (Closed)

Created:
11 years, 7 months ago by TVL
Modified:
9 years, 7 months ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Rename official builds to "Google Chrome" on disk and "Chrome" in the menubar. BUG=11826 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=16313

Patch Set 1 #

Total comments: 1

Patch Set 2 : '' #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -21 lines) Patch
M build/mac/build_app_dmg View 1 chunk +12 lines, -3 lines 0 comments Download
M build/mac/dump_app_syms View 1 chunk +8 lines, -1 line 1 comment Download
M build/mac/tweak_app_infoplist View 2 chunks +10 lines, -2 lines 0 comments Download
M chrome/app/app-Info.plist View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome.gyp View 4 chunks +21 lines, -12 lines 0 comments Download
M chrome/common/mac_app_names.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/tools/build/mac/FILES-official View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
TVL
11 years, 7 months ago (2009-05-18 20:14:23 UTC) #1
Mark Mentovai
LG http://codereview.chromium.org/115474/diff/1/3 File build/mac/build_app_dmg (right): http://codereview.chromium.org/115474/diff/1/3#newcode38 Line 38: DST_DMG_PATH="${BUILT_PRODUCTS_DIR}/${DMG_NAME}" Maybe you can just do "${BUILT_PRODUCTS_DIR}/$(echo ...
11 years, 7 months ago (2009-05-18 20:26:29 UTC) #2
TVL
cl updated, i missed a script i'm gonna keep dmg name for now, i'd like ...
11 years, 7 months ago (2009-05-18 20:30:18 UTC) #3
Mark Mentovai
11 years, 7 months ago (2009-05-18 20:41:09 UTC) #4
lg

http://codereview.chromium.org/115474/diff/1005/1008
File build/mac/dump_app_syms (right):

http://codereview.chromium.org/115474/diff/1005/1008#newcode1
Line 1: #!/bin/sh
This file is fine too.

Powered by Google App Engine
This is Rietveld 408576698