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

Unified Diff: remoting/remoting.gyp

Issue 9980015: The Chrome Remote Desktop installation for Windows can be build by bots now. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 8 years, 8 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 | « chrome/tools/build/win/FILES.cfg ('k') | no next file » | 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 ef927de279a03d18499efc8167b46e684fd63ae3..b3e3dc056a13eeaf13ba8cf36566f493750a896b 100644
--- a/remoting/remoting.gyp
+++ b/remoting/remoting.gyp
@@ -24,9 +24,8 @@
'version_full':
'<!(python <(version_py_path) -f <(version_path) -t "@MAJOR@.@MINOR@.@BUILD@.@PATCH@")',
- # Windows Installer XML (WiX) path can be set in ~/.gyp/include.gypi to
- # indicate that WiX is available.
- 'wix_path%': '',
+ 'platformsdk_exists': '<!(python <(DEPTH)/build/dir_exists.py ../third_party/platformsdk_win7)',
+ 'wix_exists': '<!(python <(DEPTH)/build/dir_exists.py ../third_party/wix)',
'conditions': [
['OS=="mac"', {
@@ -396,11 +395,11 @@
], # end of 'targets'
}], # 'OS=="win"'
- # The host installation is generated only if WiX location is known and only
- # as part of a non-component build. WiX does not provide a easy way to
- # include all DLLs imported by the installed binaries depend on, so
- # supporting the component build becomes a burden.
- ['"<(wix_path)" != "" and component != "shared_library"', {
+ # The host installation is generated only if WiX is available and when
+ # building a non-component build. WiX does not provide a easy way to
+ # include all DLLs imported by the installed binaries, so supporting
+ # the component build becomes a burden.
+ ['OS == "win" and component != "shared_library" and wix_exists == "True" and platformsdk_exists == "True"', {
'targets': [
{
'target_name': 'remoting_host_installation',
@@ -417,7 +416,8 @@
'<(PRODUCT_DIR)/chromoting.msi',
],
'variables': {
- 'sas_dll_path': '<(DEPTH)/third_party/platformsdk_win7/files/redist/x86/sas.dll'
+ 'sas_dll_path': '<(DEPTH)/third_party/platformsdk_win7/files/redist/x86/sas.dll',
+ 'wix_path': '<(DEPTH)\\third_party\\wix',
},
'conditions': [
['branding == "Chrome"', {
@@ -447,10 +447,10 @@
'msvs_cygwin_shell': 0,
'msvs_quote_cmd': 0,
'action': [
- '"<(wix_path)\\bin\\candle"',
- '-ext "<(wix_path)\\bin\\WixFirewallExtension.dll"',
- '-ext "<(wix_path)\\bin\\WixUIExtension.dll"',
- '-ext "<(wix_path)\\bin\\WixUtilExtension.dll"',
+ '"<(wix_path)\\candle"',
+ '-ext "<(wix_path)\\WixFirewallExtension.dll"',
+ '-ext "<(wix_path)\\WixUIExtension.dll"',
+ '-ext "<(wix_path)\\WixUtilExtension.dll"',
'-dVersion=<(version_full) '
'"-dFileSource=<(PRODUCT_DIR)." '
'"-dSasDllPath=<(sas_dll_path)" '
@@ -476,10 +476,10 @@
'msvs_cygwin_shell': 0,
'msvs_quote_cmd': 0,
'action': [
- '"<(wix_path)\\bin\\light"',
- '-ext "<(wix_path)\\bin\\WixFirewallExtension.dll"',
- '-ext "<(wix_path)\\bin\\WixUIExtension.dll"',
- '-ext "<(wix_path)\\bin\\WixUtilExtension.dll"',
+ '"<(wix_path)\\light"',
+ '-ext "<(wix_path)\\WixFirewallExtension.dll"',
+ '-ext "<(wix_path)\\WixUIExtension.dll"',
+ '-ext "<(wix_path)\\WixUtilExtension.dll"',
'-cultures:en-us',
'-dVersion=<(version_full) '
'"-dFileSource=<(PRODUCT_DIR)." '
« no previous file with comments | « chrome/tools/build/win/FILES.cfg ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698