Generate tests/Android.mk from gyp

platform_tools/android/bin/gyp_to_android.py

 #!/usr/bin/python
 
 # Copyright 2014 Google Inc.
 #
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """
 Script for generating the Android framework's version of Skia from gyp
 files.
 """
 
-import android_framework_gyp
 import os
 import shutil
 import sys
 import tempfile
 
 # Find the top of trunk
 SCRIPT_DIR = os.path.abspath(os.path.dirname(__file__))
 SKIA_DIR = os.path.normpath(os.path.join(SCRIPT_DIR, os.pardir, os.pardir,
                                          os.pardir))
 
 # Find the directory with our helper files, and add it to the path.
 GYP_GEN_DIR = os.path.join(SKIA_DIR, 'platform_tools', 'android', 'gyp_gen')
 sys.path.append(GYP_GEN_DIR)
 
+import android_framework_gyp
 import gypd_parser
 import generate_user_config
 import makefile_writer
+import tool_makefile_writer
 import vars_dict_lib
 
 # Folder containing all gyp files and generated gypd files.
 GYP_FOLDER = 'gyp'
 
-# TODO(scroggo): Update the docstrings to match the style guide:
-# http://google-styleguide.googlecode.com/svn/trunk/pyguide.html#Comments
-def clean_gypd_files(folder):
-  """
-  Remove the gypd files generated by android_framework_gyp.main().
-  @param folder Folder in which to delete all files ending with 'gypd'.
-  """
-  assert os.path.isdir(folder)
-  files = os.listdir(folder)
-  for f in files:
-    if f.endswith('gypd'):
-      os.remove(os.path.join(folder, f))
 
 def generate_var_dict(target_dir, target_file, skia_arch_type, have_neon):
-  """
-  Create a VarsDict for a particular arch type. Each paramater is passed
-  directly to android_framework_gyp.main().
-  @param target_dir Directory containing gyp files.
-  @param target_file Target gyp file.
-  @param skia_arch_type Target architecture.
-  @param have_neon Whether the target should build for neon.
-  @return a VarsDict containing the variable definitions determined by gyp.
+  """Create a VarsDict for a particular arch type.
+
+  Each paramater is passed directly to android_framework_gyp.main().
+
+  Args:
+    target_dir: Directory containing gyp files.
+    target_file: Target gyp file.
+    skia_arch_type: Target architecture.
+    have_neon: Whether the target should build for neon.
+  Returns:
+    A VarsDict containing the variable definitions determined by gyp.
   """
   result_file = android_framework_gyp.main(target_dir, target_file,
                                            skia_arch_type, have_neon)
   var_dict = vars_dict_lib.VarsDict()
-  gypd_parser.parse_gypd(var_dict, result_file)
-  clean_gypd_files(target_dir)
+  gypd_parser.parse_gypd(var_dict, result_file, '.')
+  android_framework_gyp.clean_gypd_files(target_dir)
   print '.',
   return var_dict
 
 def main(target_dir=None, require_sk_user_config=False):
-  """
-  Read gyp files and create Android.mk for the Android framework's
-  external/skia.
-  @param target_dir Directory in which to place 'Android.mk'. If None, the file
-                    will be placed in skia's root directory.
-  @param require_sk_user_config If True, raise an AssertionError if
-                                SkUserConfig.h does not exist.
+  """Create Android.mk for the Android framework's external/skia.
+
+  Builds Android.mk using Skia's gyp files.
+
+  Args:
+    target_dir: Directory in which to place 'Android.mk'. If None, the file
+      will be placed in skia's root directory.
+    require_sk_user_config: If True, raise an AssertionError if
+      SkUserConfig.h does not exist.
   """
   # Create a temporary folder to hold gyp and gypd files. Create it in SKIA_DIR
   # so that it is a sibling of gyp/, so the relationships between gyp files and
   # other files (e.g. platform_tools/android/gyp/dependencies.gypi, referenced
   # by android_deps.gyp as a relative path) is unchanged.
   # Use mkdtemp to find an unused folder name, but then delete it so copytree
   # can be called with a non-existent directory.
   tmp_folder = tempfile.mkdtemp(dir=SKIA_DIR)
   os.rmdir(tmp_folder)
   shutil.copytree(os.path.join(SKIA_DIR, GYP_FOLDER), tmp_folder)
@@ skipping 27 matching lines @@
     arm64_var_dict = generate_var_dict(tmp_folder, main_gyp_file, 'arm64',
                                        False)
 
     # Compute the intersection of all targets. All the files in the intersection
     # should be part of the makefile always. Each dict will now contain trimmed
     # lists containing only variable definitions specific to that configuration.
     var_dict_list = [default_var_dict, arm_var_dict, arm_neon_var_dict,
                      x86_var_dict, mips_var_dict, arm64_var_dict]
     common = vars_dict_lib.intersect(var_dict_list)
 
+    common.LOCAL_MODULE.add('libskia')
+
     # Create SkUserConfig
     user_config = os.path.join(SKIA_DIR, 'include', 'config', 'SkUserConfig.h')
     if target_dir:
       dst_dir = target_dir
     else:
       dst_dir = os.path.join(SKIA_DIR, 'include', 'core')
 
     generate_user_config.generate_user_config(
         original_sk_user_config=user_config,
         require_sk_user_config=require_sk_user_config, target_dir=dst_dir,
         ordered_set=common.DEFINES)
 
-    # Now that the defines have been written to SkUserConfig, they are not
-    # needed in Android.mk.
+    tool_makefile_writer.generate_tool(gyp_dir=tmp_folder,
+                                       target_file='tests.gyp',
+                                       skia_trunk=target_dir,
+                                       dest_dir='tests',
+                                       skia_lib_var_dict=common,
+                                       local_module_name='skia_test',
+                                       local_module_tags=['eng', 'tests'])
+
+    # TODO (scroggo): Generate bench/Android.mk. See skbug.com/2448
+    #tool_makefile_writer.generate_tool(gyp_dir=tmp_folder,
+    #                                   target_file='bench.gyp',
+    #                                   skia_trunk=target_dir,
+    #                                   dest_dir='bench',
+    #                                   skia_lib_var_dict=common,
+    #                                   local_module_name='skia_bench',
+    #                                   local_module_tags=['tests'])
+
+    # Now that the defines have been written to SkUserConfig and they've been
+    # used to skip adding them to the tools makefiles, they are not needed in
+    # Android.mk. Reset DEFINES.
     common.DEFINES.reset()
 
     # Further trim arm_neon_var_dict with arm_var_dict. After this call,
     # arm_var_dict (which will now be the intersection) includes all definitions
     # used by both arm and arm + neon, and arm_neon_var_dict will only contain
     # those specific to arm + neon.
     arm_var_dict = vars_dict_lib.intersect([arm_var_dict, arm_neon_var_dict])
 
     # Now create a list of VarsDictData holding everything but common.
     deviations_from_common = []
@@ skipping 14 matching lines @@
                                                                'arm64'))
 
     makefile_writer.write_android_mk(target_dir=target_dir,
         common=common, deviations_from_common=deviations_from_common)
 
   finally:
     shutil.rmtree(tmp_folder)
 
 if __name__ == '__main__':
   main()