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

Side by Side Diff: chrome/installer/mac/app/BUILD.gn

Issue 2359953003: Fix up the Mac installer’s Info.plist, build with branding info (Closed)
Patch Set: Derive the installer's bundle ID from the main one, drop the tweak_info_plist step, add missing substitutions Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « build/util/branding.gni ('k') | chrome/installer/mac/app/Info.plist » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/mac/rules.gni")
6 import("//build/mac/tweak_info_plist.gni")
Robert Sesek 2016/09/22 14:46:56 Unused.
7 import("//build/util/branding.gni")
5 import("//testing/test.gni") 8 import("//testing/test.gni")
6 import("//build/config/mac/rules.gni")
7 9
8 source_set("mac_installer_base") { 10 source_set("mac_installer_base") {
9 visibility = [ ":*" ] 11 visibility = [ ":*" ]
10 sources = [ 12 sources = [
11 "Downloader.h", 13 "Downloader.h",
12 "Downloader.m", 14 "Downloader.m",
13 "NSAlert+ChromeInstallerAdditions.h", 15 "NSAlert+ChromeInstallerAdditions.h",
14 "NSAlert+ChromeInstallerAdditions.m", 16 "NSAlert+ChromeInstallerAdditions.m",
15 "NSError+ChromeInstallerAdditions.h", 17 "NSError+ChromeInstallerAdditions.h",
16 "NSError+ChromeInstallerAdditions.m", 18 "NSError+ChromeInstallerAdditions.m",
17 "OmahaCommunication.h", 19 "OmahaCommunication.h",
18 "OmahaCommunication.m", 20 "OmahaCommunication.m",
19 "OmahaXMLParser.h", 21 "OmahaXMLParser.h",
20 "OmahaXMLParser.m", 22 "OmahaXMLParser.m",
21 "OmahaXMLRequest.h", 23 "OmahaXMLRequest.h",
22 "OmahaXMLRequest.m", 24 "OmahaXMLRequest.m",
23 "SystemInfo.h", 25 "SystemInfo.h",
24 "SystemInfo.m", 26 "SystemInfo.m",
25 "Unpacker.h", 27 "Unpacker.h",
26 "Unpacker.m", 28 "Unpacker.m",
27 ] 29 ]
28 public_configs = [ "//build/config/compiler:enable_arc" ] 30 public_configs = [ "//build/config/compiler:enable_arc" ]
29 } 31 }
30 32
31 mac_app_bundle("mac_installer_app") { 33 mac_app_bundle("mac_installer_app") {
34 output_name = "$chrome_product_installer_full_name"
32 info_plist = "Info.plist" 35 info_plist = "Info.plist"
33 extra_substitutions = [ "MACOSX_DEPLOYMENT_TARGET=10.9" ] 36 extra_substitutions = [
37 "MACOSX_DEPLOYMENT_TARGET=10.9",
38 "PRODUCT_INSTALLER_FULLNAME=$chrome_product_installer_full_name",
39 "MAC_INSTALLER_BUNDLE_ID=$chrome_mac_bundle_id.installer",
Robert Sesek 2016/09/22 14:46:56 For consistency with chrome/BULD.gn, use CHROMIUM_
Sidney San Martín 2016/09/22 15:43:48 Done.
40 ]
34 sources = [ 41 sources = [
35 "AppDelegate.h", 42 "AppDelegate.h",
36 "AppDelegate.mm", 43 "AppDelegate.mm",
37 "AuthorizedInstall.h", 44 "AuthorizedInstall.h",
38 "AuthorizedInstall.m", 45 "AuthorizedInstall.m",
39 "InstallerWindowController.h", 46 "InstallerWindowController.h",
40 "InstallerWindowController.m", 47 "InstallerWindowController.m",
41 "main.m", 48 "main.m",
42 ] 49 ]
43 50
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 "DiskArbitration.framework", 95 "DiskArbitration.framework",
89 "Security.framework", 96 "Security.framework",
90 ] 97 ]
91 data = [ 98 data = [
92 "//chrome/test/data/mac_installer/requestCheck.dtd", 99 "//chrome/test/data/mac_installer/requestCheck.dtd",
93 "//chrome/test/data/mac_installer/requestSample.xml", 100 "//chrome/test/data/mac_installer/requestSample.xml",
94 "//chrome/test/data/mac_installer/responseExample.xml", 101 "//chrome/test/data/mac_installer/responseExample.xml",
95 "//chrome/test/data/mac_installer/test-dmg.dmg", 102 "//chrome/test/data/mac_installer/test-dmg.dmg",
96 ] 103 ]
97 } 104 }
OLDNEW
« no previous file with comments | « build/util/branding.gni ('k') | chrome/installer/mac/app/Info.plist » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698