Move remoting dark/candle/light steps to external script

remoting/remoting.gyp

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     # TODO(dmaclach): can we pick this up some other way? Right now it's
     # duplicated from chrome.gyp
     'chromium_code': 1,
     # Use consistent strings across all platforms. Note that the plugin name
@@ skipping 660 matching lines @@
           ],
           'conditions': [
             ['buildtype == "Official"', {
               'wix_defines': [
                 '-dOfficialBuild=1',
               ],
             }],
           ],
           'rules': [
             {
-              'rule_name': 'candle',
+              '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',
                 '<(platformsdk_path)/redist/x86/sas.dll',
                 'resources/chromoting.ico',
+                'candle_and_light.py',
               ],
               'outputs': [
-                '<(INTERMEDIATE_DIR)/<(RULE_INPUT_ROOT).wixobj',
+                '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).msi',
               ],
-              'process_outputs_as_sources': 1,
               'msvs_cygwin_shell': 0,
               'action': [
-                '<(wix_path)\\candle',
-                '-ext', '<(wix_path)\\WixFirewallExtension.dll',
-                '-ext', '<(wix_path)\\WixUIExtension.dll',
-                '-ext', '<(wix_path)\\WixUtilExtension.dll',
-                '-dVersion=<(version_full)',
-                '-dFileSource=<(PRODUCT_DIR).',
-                '-dIconPath=resources/chromoting.ico',
-                '-dSasDllPath=<(platformsdk_path)/redist/x86/sas.dll',
+                'python', 'candle_and_light.py',
+                '--wix_path', '<(wix_path)',
+                '--version', '<(version_full)',
+                '--product_dir', '<(PRODUCT_DIR).',
+                '--intermediate_dir', '<(INTERMEDIATE_DIR).',
+                '--platformsdk_path', '<(platformsdk_path)',
+                '--input', '<(RULE_INPUT_PATH)',
+                '--output', '<@(_outputs)',
                 '<@(_wix_defines)',
-                '-out', '<@(_outputs)',
-                '<(RULE_INPUT_PATH)',
               ],
               'message': 'Generating <@(_outputs)',
             },
-            {
-              'rule_name': 'light',
-              'extension': 'wixobj',
-              'inputs': [
-                '<(PRODUCT_DIR)/remoting_host_controller.exe',
-                '<(PRODUCT_DIR)/remoting_me2me_host.exe',
-                '<(PRODUCT_DIR)/remoting_service.exe',
-                '<(platformsdk_path)/redist/x86/sas.dll',
-                'resources/chromoting.ico',
-              ],
-              'outputs': [
-                '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).msi',
-                '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).wixpdb',
-              ],
-              'msvs_cygwin_shell': 0,
-              'action': [
-                '<(wix_path)\\light',
-                '-ext', '<(wix_path)\\WixFirewallExtension.dll',
-                '-ext', '<(wix_path)\\WixUIExtension.dll',
-                '-ext', '<(wix_path)\\WixUtilExtension.dll',
-                '-cultures:en-us',
-                '-sw1076',
-                '-dVersion=<(version_full)',
-                '-dFileSource=<(PRODUCT_DIR).',
-                '-dIconPath=resources/chromoting.ico',
-                '-dSasDllPath=<(platformsdk_path)/redist/x86/sas.dll',
-                '<@(_wix_defines)',
-                '-out', '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).msi',
-                '<(RULE_INPUT_PATH)',
-              ],
-              'message': 'Generating <(PRODUCT_DIR)/<(RULE_INPUT_ROOT).msi',
-            },
           ],
         },  # end of target 'remoting_host_installation'
 
         # The 'remoting_host_installation_unittest' target is used to make sure
         # that the code signing job (running outside of Chromium tree) will be
         # able to unpack and re-assemble the installation successfully.
         #
         # *** If this target fails to compile the code signing job will fail
         # too, breaking the official build. ***
         #
         # N.B. The command lines passed to the WiX tools here should be in sync
         # with the code signing script.
         {
           'target_name': 'remoting_host_installation_unittest',
           'type': 'none',
           'dependencies': [
             'remoting_host_installation',
           ],
           'sources': [
             '<(PRODUCT_DIR)/chromoting.msi',
           ],
           'outputs': [
             '<(INTERMEDIATE_DIR)/chromoting-test.msi',
           ],
           'rules': [
             {
-              'rule_name': 'dark',
+              'rule_name': 'dark_and_candle_and_light',
               'extension': 'msi',
+              'inputs': [
+                'dark_and_candle_and_light.py',
+              ],
               'outputs': [
-                '<(INTERMEDIATE_DIR)/<(RULE_INPUT_ROOT).wxs',
-              ],
-              'process_outputs_as_sources': 1,
-              'msvs_cygwin_shell': 0,
-              'action': [
-                '<(wix_path)\\dark',
-                '<(RULE_INPUT_PATH)',
-                '-o', '<@(_outputs)',
-                '-x', '<(INTERMEDIATE_DIR).',
-              ],
-              'message': 'Dark: unpacking <(RULE_INPUT_PATH)',
-            },
-            {
-              'rule_name': 'candle',
-              'extension': 'wxs',
-              'outputs': [
-                '<(INTERMEDIATE_DIR)/<(RULE_INPUT_ROOT).wixobj',
-              ],
-              'process_outputs_as_sources': 1,
-              'msvs_cygwin_shell': 0,
-              'action': [
-                '<(wix_path)\\candle',
-                '<(RULE_INPUT_PATH)',
-                '-o', '<@(_outputs)',
-                '-ext', '<(wix_path)\\WixFirewallExtension.dll',
-              ],
-              'message': 'Candle: compiling <(RULE_INPUT_PATH)',
-            },
-            {
-              'rule_name': 'light',
-              'extension': 'wixobj',
-              'outputs': [
-                '<(INTERMEDIATE_DIR)/<(RULE_INPUT_ROOT)-test.msi',
+                '<(INTERMEDIATE_DIR)/chromoting-test.msi',
               ],
               'msvs_cygwin_shell': 0,
               'action': [
-                '<(wix_path)\\light',
-                '<(RULE_INPUT_PATH)',
-                '-o', '<@(_outputs)',
-                '-ext', '<(wix_path)\\WixFirewallExtension.dll',
-                '-sw1076',
+                'python',
+                'dark_and_candle_and_light.py',
+                '--wix_path', '<(wix_path)',
+                '--input', '<(RULE_INPUT_PATH)',
+                '--intermediate_dir', '<(INTERMEDIATE_DIR).',
+                '--output', '<@(_outputs)',
               ],
-              'message': 'Light: linking <(RULE_INPUT_PATH)',
+              'message': 'Unpacking and repacking to <@(_outputs)',
             },
           ],
         },  # end of target 'remoting_host_installation_unittest'
       ],  # end of 'targets'
     }],  # '<(wix_path) != ""'
 
   ],  # end of 'conditions'
 
   'targets': [
     {
@@ skipping 945 matching lines @@
                   '../base/allocator/allocator.gyp:allocator',
                 ],
               },
             ],
           ],
         }],
       ],  # end of 'conditions'
     },  # end of target 'remoting_unittests'
   ],  # end of targets
 }