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

Unified Diff: remoting/remoting.gyp

Issue 10910075: [Chromoting] Adding the desktop integration binary to the host installation on Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix preprocessor conditions in .WXS & rebased Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/host/win/remoting_daemon.ver ('k') | remoting/version.rc.version » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/remoting.gyp
diff --git a/remoting/remoting.gyp b/remoting/remoting.gyp
index 82381c006a0a9fdb9afa5b232044e039f95a4c74..4457c2c110c2fb223d44279953a16bdb41e9a4e3 100644
--- a/remoting/remoting.gyp
+++ b/remoting/remoting.gyp
@@ -29,7 +29,7 @@
# Use consistent strings across all platforms.
# These values must match host/plugin/constants.h
'host_plugin_mime_type': 'application/vnd.chromium.remoting-host',
- 'host_plugin_description': '<!(python <(version_py_path) -f <(branding_path) -t "@HOST_PLUGIN_FILE_DESCRIPTION@")',
+ 'host_plugin_description': '<!(python <(version_py_path) -f <(branding_path) -t "@HOST_PLUGIN_DESCRIPTION@")',
'host_plugin_name': '<!(python <(version_py_path) -f <(branding_path) -t "@HOST_PLUGIN_FILE_NAME@")',
'conditions': [
@@ -207,7 +207,7 @@
'ENABLE_REMOTING_AUDIO',
],
}],
- ['remoting_multi_process == 1', {
+ ['remoting_multi_process != 0', {
'defines': [
'REMOTING_MULTI_PROCESS',
],
@@ -310,7 +310,7 @@
],
'variables': {
'host_name': '<!(python <(version_py_path) -f <(branding_path) -t "@HOST_PLUGIN_FILE_NAME@")',
- 'host_service_name': '<!(python <(version_py_path) -f <(branding_path) -t "@HOST_SERVICE_FILE_NAME@")',
+ 'host_service_name': '<!(python <(version_py_path) -f <(branding_path) -t "@DAEMON_FILE_NAME@")',
'host_uninstaller_name': '<!(python <(version_py_path) -f <(branding_path) -t "@MAC_UNINSTALLER_NAME@")',
'bundle_prefix': '<!(python <(version_py_path) -f <(branding_path) -t "@MAC_UNINSTALLER_BUNDLE_PREFIX@")',
},
@@ -540,7 +540,7 @@
'remoting_version_resources',
],
'sources': [
- '<(SHARED_INTERMEDIATE_DIR)/remoting/elevated_controller_version.rc',
+ '<(SHARED_INTERMEDIATE_DIR)/remoting/remoting_controller_version.rc',
'host/branding.cc',
'host/branding.h',
'host/pin_hash.cc',
@@ -587,7 +587,7 @@
'remoting_version_resources',
],
'sources': [
- '<(SHARED_INTERMEDIATE_DIR)/remoting/host_service_version.rc',
+ '<(SHARED_INTERMEDIATE_DIR)/remoting/remoting_daemon_version.rc',
'base/scoped_sc_handle_win.h',
'host/branding.cc',
'host/branding.h',
@@ -653,10 +653,11 @@
],
},
'sources': [
- 'host/plugin/host_plugin.ver',
- 'host/remoting_me2me_host.ver',
- 'host/win/elevated_controller.ver',
- 'host/win/host_service.ver',
+ 'host/plugin/remoting_host_plugin.ver',
+ 'host/remoting_desktop.ver',
+ 'host/remoting_host_me2me.ver',
+ 'host/win/remoting_controller.ver',
+ 'host/win/remoting_daemon.ver',
],
'rules': [
{
@@ -717,8 +718,24 @@
],
'wix_defines' : [
'-dBranding=<(branding)',
+ '-dRemotingMultiProcess=<(remoting_multi_process)',
+ ],
+ 'wix_inputs' : [
+ '<(PRODUCT_DIR)/remoting_host_controller.exe',
+ '<(PRODUCT_DIR)/remoting_me2me_host.exe',
+ '<(PRODUCT_DIR)/remoting_service.exe',
+ '<(sas_dll_path)/sas.dll',
+ 'resources/chromoting.ico',
],
'conditions': [
+ ['remoting_multi_process != 0', {
+ 'dependencies': [
+ 'remoting_desktop',
+ ],
+ 'wix_inputs' : [
+ '<(PRODUCT_DIR)/remoting_desktop.exe',
+ ],
+ }],
['buildtype == "Official"', {
'wix_defines': [
'-dOfficialBuild=1',
@@ -730,11 +747,7 @@
'rule_name': 'candle_and_light',
'extension': 'wxs',
'inputs': [
- '<(PRODUCT_DIR)/remoting_host_controller.exe',
- '<(PRODUCT_DIR)/remoting_me2me_host.exe',
- '<(PRODUCT_DIR)/remoting_service.exe',
- '<(sas_dll_path)/sas.dll',
- 'resources/chromoting.ico',
+ '<@(_wix_inputs)',
'tools/candle_and_light.py',
],
'outputs': [
@@ -805,6 +818,60 @@
], # end of 'targets'
}], # '<(wix_path) != ""'
+ ['remoting_multi_process != 0', {
+ 'targets': [
+ {
+ 'target_name': 'remoting_desktop',
+ 'type': 'executable',
+ 'variables': { 'enable_wexit_time_destructors': 1, },
+ 'dependencies': [
+ 'remoting_base',
+ 'remoting_breakpad',
+ 'remoting_host',
+ 'remoting_version_resources',
+ '../base/base.gyp:base',
+ '../ipc/ipc.gyp:ipc',
+ ],
+ 'sources': [
+ 'host/branding.cc',
+ 'host/branding.h',
+ 'host/desktop_process.cc',
+ 'host/desktop_process.h',
+ 'host/host_ui.rc',
+ 'host/usage_stats_consent.h',
+ 'host/usage_stats_consent_win.cc',
+ '<(SHARED_INTERMEDIATE_DIR)/remoting/remoting_desktop_version.rc',
+ ],
+ 'link_settings': {
+ 'libraries': [
+ '-lcomctl32.lib',
+ ],
+ },
+ 'msvs_settings': {
+ 'VCLinkerTool': {
+ 'AdditionalOptions': [
+ "\"/manifestdependency:type='win32' "
+ "name='Microsoft.Windows.Common-Controls' "
+ "version='6.0.0.0' "
+ "processorArchitecture='*' "
+ "publicKeyToken='6595b64144ccf1df' language='*'\"",
+ ],
+ 'conditions': [
+ ['buildtype == "Official" and remoting_multi_process != 0', {
+ 'AdditionalOptions': [
+ "\"/MANIFESTUAC:level='requireAdministrator' "
+ "uiAccess='true'\"",
+ ],
+ }],
+ ],
+ # 2 == /SUBSYSTEM:WINDOWS
+ 'SubSystem': '2',
+ },
+ },
+ }, # end of target 'remoting_desktop'
+ ],
+ }], # 'remoting_multi_process != 0'
+
], # end of 'conditions'
'targets': [
@@ -1004,7 +1071,7 @@
'<(INTERMEDIATE_DIR)',
],
'sources': [
- '<(SHARED_INTERMEDIATE_DIR)/remoting/host_plugin_version.rc',
+ '<(SHARED_INTERMEDIATE_DIR)/remoting/remoting_host_plugin_version.rc',
'host/host_ui.rc',
'host/plugin/host_plugin.def',
],
@@ -1494,7 +1561,7 @@
],
'sources': [
'<(SHARED_INTERMEDIATE_DIR)/remoting/host/remoting_host_messages.rc',
- '<(SHARED_INTERMEDIATE_DIR)/remoting/remoting_me2me_host_version.rc',
+ '<(SHARED_INTERMEDIATE_DIR)/remoting/remoting_host_me2me_version.rc',
'host/host_ui.rc',
],
'link_settings': {
@@ -1512,7 +1579,7 @@
"publicKeyToken='6595b64144ccf1df' language='*'\"",
],
'conditions': [
- ['buildtype == "Official"', {
+ ['buildtype == "Official" and remoting_multi_process == 0', {
'AdditionalOptions': [
"\"/MANIFESTUAC:level='requireAdministrator' "
"uiAccess='true'\"",
« no previous file with comments | « remoting/host/win/remoting_daemon.ver ('k') | remoting/version.rc.version » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698