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

Unified Diff: remoting/host/BUILD.gn

Issue 2169273002: [Mac/GN] Fix all the //remoting Info.plist substitutions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/util/version.gni ('k') | remoting/host/installer/mac/uninstaller/remoting_uninstaller-Info.plist » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/BUILD.gn
diff --git a/remoting/host/BUILD.gn b/remoting/host/BUILD.gn
index 720e4594b0dd176685fbf45130b814e109e3762c..0df1ea9efbdf2528e3730c5a86225c1651a137f2 100644
--- a/remoting/host/BUILD.gn
+++ b/remoting/host/BUILD.gn
@@ -995,6 +995,12 @@ if (enable_me2me_host) {
if (is_mac) {
extra_configs = [ "//remoting:version" ]
info_plist = "remoting_me2me_host-Info.plist"
+ extra_substitutions = [
+ "BUNDLE_ID=$host_bundle_id",
+ "VERSION_FULL=$remoting_version_full",
+ "VERSION_SHORT=$remoting_version_short",
+ "MACOSX_DEPLOYMENT_TARGET=10.7",
+ ]
} else {
configs += [ "//remoting:version" ]
}
@@ -1081,6 +1087,12 @@ if (enable_me2me_host) {
if (is_mac) {
info_plist = "setup/native_messaging_host-Info.plist"
extra_configs = [ "//build/config/compiler:wexit_time_destructors" ]
+ extra_substitutions = [
+ "BUNDLE_ID=$native_messaging_host_bundle_id",
+ "VERSION_FULL=$remoting_version_full",
+ "VERSION_SHORT=$remoting_version_short",
+ "MACOSX_DEPLOYMENT_TARGET=10.7",
+ ]
} else {
configs += [ "//build/config/compiler:wexit_time_destructors" ]
}
@@ -1416,6 +1428,12 @@ if (enable_me2me_host) {
mac_app_bundle("remoting_host_uninstaller") {
info_plist = _uninstaller_plist
+ extra_substitutions = [
+ "BUNDLE_ID=$uninstaller_bundle_id",
+ "VERSION_FULL=$remoting_version_full",
+ "VERSION_SHORT=$remoting_version_short",
+ "MACOSX_DEPLOYMENT_TARGET=10.7",
+ ]
defines = [
"HOST_BUNDLE_NAME=\"" + host_bundle_name + "\"",
@@ -1472,7 +1490,7 @@ if (enable_me2me_host) {
deps = [
":remoting_host_prefpane",
":remoting_host_prefpane_bundle_data",
- ":remoting_host_prefpane_plist",
+ ":remoting_host_prefpane_plist_bundle_data",
":remoting_host_prefpane_resources",
":remoting_host_prefpane_xibs",
":remoting_infoplist_strings",
@@ -1537,13 +1555,25 @@ if (enable_me2me_host) {
}
}
- bundle_data("remoting_host_prefpane_plist") {
- sources = [
- "mac/me2me_preference_pane-Info.plist",
+ mac_info_plist("remoting_host_prefpane_plist") {
+ info_plist = "mac/me2me_preference_pane-Info.plist"
+ extra_substitutions = [
+ "BUNDLE_ID=$prefpane_bundle_id",
+ "VERSION_FULL=$remoting_version_full",
+ "VERSION_SHORT=$remoting_version_short",
+ "MACOSX_DEPLOYMENT_TARGET=10.7",
]
+ executable_name = "remoting_host_prefpane"
+ }
+
+ bundle_data("remoting_host_prefpane_plist_bundle_data") {
+ sources = get_target_outputs(":remoting_host_prefpane_plist")
outputs = [
"{{bundle_root_dir}}/Info.plist",
]
+ public_deps = [
+ ":remoting_host_prefpane_plist",
+ ]
}
foreach(locale, remoting_locales_with_underscores) {
« no previous file with comments | « build/util/version.gni ('k') | remoting/host/installer/mac/uninstaller/remoting_uninstaller-Info.plist » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698