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

Unified Diff: remoting/host/BUILD.gn

Issue 2229673002: Fixing the broken CRD Windows MSI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 | « no previous file | no next file » | 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 beaa52907ba5f0dbc10ee52c2b4cf6058ac392f2..c86b50e8650477d07dedd8004d0c8d337e9b74d3 100644
--- a/remoting/host/BUILD.gn
+++ b/remoting/host/BUILD.gn
@@ -1298,7 +1298,24 @@ if (enable_me2me_host) {
]
_generated_files = rebase_path(inputs, root_build_dir)
- _extra_files = []
+ _generated_files += [ rebase_path("//remoting/resources/chromoting.ico") ]
+
+ # _generated_dst_files must contain the same files in the same order as
+ # _generated_files, otherwise the Windows MSI will not be built correctly.
+ _generated_dst_files = [
+ "files/remote_assistance_host.exe",
+ "files/remote_security_key.exe",
+ "files/remoting_core.dll",
+ "files/remoting_desktop.exe",
+ "files/remoting_host.exe",
+ "files/remoting_native_messaging_host.exe",
+ "files/remoting_start_host.exe",
+ "files/CREDITS.txt",
+ "files/com.google.chrome.remote_assistance.json",
+ "files/com.google.chrome.remote_desktop.json",
+ "files/icudtl.dat",
+ "files/chromoting.ico",
+ ]
args =
[
@@ -1314,24 +1331,11 @@ if (enable_me2me_host) {
"--generated-files",
] + _generated_files +
[
- rebase_path("//remoting/resources/chromoting.ico"),
-
# Position of files in zip file
"--generated-files-dst",
- "files/remote_assistance_host.exe",
- "files/remote_security_key.exe",
- "files/remoting_core.dll",
- "files/remoting_desktop.exe",
- "files/remoting_host.exe",
- "files/remoting_native_messaging_host.exe",
- "files/remoting_start_host.exe",
- "files/CREDITS.txt",
- "files/com.google.chrome.remote_assistance.json",
- "files/com.google.chrome.remote_desktop.json",
- "files/chromoting.ico",
- "files/icudtl.dat",
- ] + _extra_files + # Defs
+ ] + _generated_dst_files +
[
+ # Defs
"--defs",
"BRANDING=$_branding",
"DAEMON_CONTROLLER_CLSID={$daemon_controller_clsid}",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698