Cronet Dynamic Framework for iOS

components/cronet/tools/package_ios.py

 #!/usr/bin/python
 # Copyright 2016 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.
 
 """
 package_ios.py - Build and Package Release and Rebug fat libraries for iOS.
 """
 
 import argparse
 import os
 import shutil
 import sys
 
 def run(command, extra_options=''):
   command = command + ' ' + extra_options
   print command
   return os.system(command)
 
+def eprint(message):

[mef, 2016/05/11 12:55:02]

only used once, inline?

[kapishnikov, 2016/05/11 19:48:15]

Done.

+  print >>sys.stderr, message

[mef, 2016/05/11 12:55:02]

nit: 2 empty lines between functions. Here and elsewhere.

[kapishnikov, 2016/05/11 19:48:15]

Done.

+
+def copy_directory(src, dst):
+  """Copies |src| directory into |dst| directory.
+  """
+  dir_name = os.path.basename(src)
+  os.mkdir(os.path.join(dst, dir_name))
+  for item in os.listdir(src):

[mef, 2016/05/11 12:55:02]

Hrm, I still find it weird that we need a custom function to copy directory.

[kapishnikov, 2016/05/11 19:48:15]

Done.

+    s = os.path.join(src, item)
+    d = os.path.join(dst, dir_name, item)
+    if os.path.isdir(s):
+      shutil.copytree(s, d)
+    else:
+      shutil.copy2(s, d)
 
 def build(out_dir, test_target, extra_options=''):
   return run('ninja -C ' + out_dir + ' ' + test_target,
              extra_options)
 
 
 def lipo_libraries(out_dir, input_dirs, out_lib, input_lib):
   lipo = "lipo -create "
   for input_dir in input_dirs:
     lipo += input_dir + "/" + input_lib + " "
@@ skipping 14 matching lines @@
   build_target = 'cronet_package'
   target_dir = out_dir + "/Cronet"
   return build(build_dir_sim, build_target) or \
          build(build_dir_dev, build_target) or \
          copy_build_dir(target_dir, build_dir_dev + "/cronet") or \
          lipo_libraries(target_dir, [build_dir_sim, build_dir_dev], \
                         "libcronet_" + build_config + ".a", \
                         "cronet/libcronet_standalone.a")
 
 
+def package_ios_framework(out_dir='out/Framework', extra_options=''):
+  print 'Building Cronet Dynamic Framework...'
+
+  # Use Ninja to build all possible combinations.
+  build_dirs = ['Debug-iphonesimulator',
+                'Debug-iphoneos',
+                'Release-iphonesimulator',
+                'Release-iphoneos']
+  for build_dir in build_dirs:
+    print 'Building ' + build_dir
+    build_result = run('ninja -C out/' + build_dir + ' cronet_framework',
+                       extra_options)
+    if build_result != 0:
+      return build_result
+
+  # Package all builds in the output directory
+  os.makedirs(out_dir)
+  for build_dir in build_dirs:
+    os.makedirs(os.path.join(out_dir, build_dir))
+    copy_directory(os.path.join('out', build_dir, 'Cronet.framework'),
+                   os.path.join(out_dir, build_dir))
+    if 'Release' in build_dir:
+      copy_directory(os.path.join('out', build_dir, 'Cronet.framework.dSYM'),
+                     os.path.join(out_dir, build_dir))
+
 def main():
   parser = argparse.ArgumentParser()
   parser.add_argument('out_dir', nargs=1, help='path to output directory')
   parser.add_argument('-g', '--skip_gyp', action='store_true',
                       help='skip gyp')
   parser.add_argument('-d', '--debug', action='store_true',
                       help='use release configuration')
   parser.add_argument('-r', '--release', action='store_true',
                       help='use release configuration')
+  parser.add_argument('--framework', action='store_true',
+                      help='build Cronet dynamic framework')
 
   options, extra_options_list = parser.parse_known_args()
   print options
   print extra_options_list
 
+  out_dir = options.out_dir[0]
+
+  # Make sure that the output directory does not exist
+  if os.path.exists(out_dir):
+    eprint('The output directory already exists: ' + out_dir)
+    return 1
+
   gyp_defines = 'GYP_DEFINES="OS=ios enable_websockets=0 '+ \
       'disable_file_support=1 disable_ftp_support=1 '+ \
       'enable_errorprone=1 use_platform_icu_alternatives=1 ' + \
-      'disable_brotli_filter=1 target_subarch=both"'
+      'disable_brotli_filter=1 chromium_ios_signing=0 ' + \
+      'target_subarch=both"'
   if not options.skip_gyp:
     run (gyp_defines + ' gclient runhooks')
 
-  return package_ios(options.out_dir[0], "out/Release", "opt") or \
-         package_ios(options.out_dir[0], "out/Debug", "dbg")
+  if options.framework:
+    return package_ios_framework(out_dir)
+
+  return package_ios(out_dir, "out/Release", "opt") or \
+         package_ios(out_dir, "out/Debug", "dbg")
 
 
 if __name__ == '__main__':
   sys.exit(main())