[Chromoting] Add target to build archive for remoting_me2me_host files.

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 148 matching lines @@
       'webapp/xhr.js',
       'resources/chromoting16.png',
       'resources/chromoting48.png',
       'resources/chromoting128.png',
       'resources/disclosure_arrow_down.png',
       'resources/disclosure_arrow_right.png',
       'resources/infographic_my_computers.png',
       'resources/infographic_remote_assistance.png',
       'resources/tick.png',
     ],
+    'remoting_host_installer_mac_root': 'host/installer/mac/',
+    'remoting_host_installer_mac_files': [
+      'host/installer/mac/ChromeRemoteDesktop.packproj',
+      'host/installer/mac/Chromoting.packproj',
+      'host/installer/mac/LaunchAgents/org.chromium.chromoting.plist',
+      'host/installer/mac/PrivilegedHelperTools/org.chromium.chromoting.json',
+      'host/installer/mac/PrivilegedHelperTools/org.chromium.chromoting.me2me.sh',
+      'host/installer/mac/Scripts/keystone_install.sh',
+      'host/installer/mac/Scripts/remoting_postflight.sh',
+      'host/installer/mac/Scripts/uninstall.sh',
+      'host/installer/mac/Keystone/*',
+    ],
   },
 
   'target_defaults': {
     'defines': [
     ],
     'include_dirs': [
       '..',  # Root of Chrome checkout
     ],
   },
 
@@ skipping 890 matching lines @@
         'remoting_base',
         '../base/base.gyp:base',
         '../base/base.gyp:base_i18n',
         '../crypto/crypto.gyp:crypto',
       ],
       'sources': [
         'host/keygen_main.cc',
       ],
     },  # end of target 'remoting_host_keygen'
 
+    # This packages up the files needed for the remoting host installer so
+    # they can be sent off to be signed.
+    # We don't build an installer here because we don't have signed binaries.
+    {
+      'target_name': 'remoting_me2me_host_archive',
+      'type': 'none',
+      'dependencies': [
+        'remoting_me2me_host',
+      ],
+      'sources': [
+        'host/installer/build-installer-archive.py',
+      ],
+      'conditions': [
+        ['OS=="mac"', {
+          'sources': [
+            '<@(remoting_host_installer_mac_files)',
+          ],
+        }],  # OS=="mac"
+        ['OS=="win"', {
+          'dependencies': [
+            # TODO(garykac)
+          ],
+        }],  # OS=="win"
+      ],  # conditions
+      'actions': [
+        {
+          'action_name': 'Zip installer files for signing',
+          'temp_dir': '<(SHARED_INTERMEDIATE_DIR)/remoting/remoting-me2me-host',
+          'zip_path': '<(PRODUCT_DIR)/remoting-me2me-host-<(OS).zip',
+          'binaries_src': [],
+          'binaries_dst': [],
+          'other_files_root': '',

[alexeypa (please no reviews), 2012/04/13 17:27:13]

Naming is confusing. How about:

binaries_src -> generated_files
binaries_dst -> generated_destinations
other_files -> source_files
other_files_root -> source_root

The names in the script should be changed similarly.

[garykac, 2012/04/13 18:06:03]

Done.

+          'other_files': [],
+          'conditions': [
+            ['OS=="mac"', {
+              'binaries_src': [
+                '<(PRODUCT_DIR)/remoting_me2me_host',
+              ],
+              'binaries_dst': [
+                'PrivilegedHelperTools/org.chromium.chromoting.me2me_host',
+              ],
+              'other_files_root': '<(remoting_host_installer_mac_root)',
+              'other_files': [
+                '<@(remoting_host_installer_mac_files)',
+              ],
+            }],  # OS=="mac"
+          ],  # conditions
+          'inputs': [
+            'host/installer/build-installer-archive.py',

[alexeypa (please no reviews), 2012/04/13 17:27:13]

Are you sure that the script should be listed here? I thought it is only for the
input files.

[garykac, 2012/04/13 18:06:03]

The script name is an input to python.

+            '<@(_other_files)',
+          ],
+          'outputs': [
+            '<(_zip_path)',
+          ],
+          'action': [
+            'python',
+            'host/installer/build-installer-archive.py',
+            '<(_temp_dir)',
+            '<(_zip_path)',
+            '<(_other_files_root)',
+            '--files',
+            '<@(_other_files)',
+            '--binaries-src',
+            '<@(_binaries_src)',
+            '--binaries-dst',
+            '<@(_binaries_dst)',
+          ],
+        },
+      ],  # actions
+    }, # end of target 'remoting_me2me_host_archive'
+
     {
       'target_name': 'remoting_jingle_glue',
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
       'dependencies': [
         '../base/base.gyp:base',
         '../jingle/jingle.gyp:jingle_glue',
         '../jingle/jingle.gyp:notifier',
         '../third_party/libjingle/libjingle.gyp:libjingle',
         '../third_party/libjingle/libjingle.gyp:libjingle_p2p',
@@ skipping 296 matching lines @@
                   '../base/allocator/allocator.gyp:allocator',
                 ],
               },
             ],
           ],
         }],
       ],  # end of 'conditions'
     },  # end of target 'remoting_unittests'
   ],  # end of targets
 }