[Android] Auto-generate API 14 resources from the existing API 17 resources.

build/java.gypi

 # 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.
 
 # This file is meant to be included into a target to provide a rule
 # to build Java in a consistent manner.
 #
 # To use this, create a gyp target with the following form:
 # {
 #   'target_name': 'my-package_java',
@@ skipping 65 matching lines @@
     'variables': {
       'input_jars_paths': ['<(jar_path)'],
       'library_dexed_jars_paths': ['<(dex_path)'],
     },
   },
   'conditions': [
     ['has_java_resources == 1', {
       'variables': {
         'res_dir': '<(java_in_dir)/res',
         'res_crunched_dir': '<(intermediate_dir)/res_crunched',
+        'res_v14_dir': '<(intermediate_dir)/res_v14',
+        'res_v14_stamp': '<(intermediate_dir)/res_v14.stamp',
+        'res_v17_dir': '<(intermediate_dir)/res_v17',
+        'res_v17_stamp': '<(intermediate_dir)/res_v17.stamp',
         'res_input_dirs': ['<(res_dir)', '<@(res_extra_dirs)'],
         'resource_input_paths': ['<!@(find <(res_dir) -type f)'],
         'R_dir': '<(intermediate_dir)/java_R',
         'R_text_file': '<(R_dir)/R.txt',
         'R_stamp': '<(intermediate_dir)/resources.stamp',
         'generated_src_dirs': ['<(R_dir)'],
-        'additional_input_paths': ['<(R_stamp)'],
+        'additional_input_paths': ['<(R_stamp)',
+                                   '<(res_v14_stamp)',
+                                   '<(res_v17_stamp)',],
         'additional_res_dirs': [],
+        'additional_res_input_dirs': [],
         'dependencies_res_files': [],
       },
       'all_dependent_settings': {
         'variables': {
           # Dependent jars include this target's R.java file via
           # generated_R_dirs and include its resources via
           # dependencies_res_files.
           'generated_R_dirs': ['<(R_dir)'],
-          'additional_input_paths': ['<(R_stamp)'],
+          'additional_input_paths': ['<(R_stamp)',
+                                     '<(res_v14_stamp)',
+                                     '<(res_v17_stamp)',],
           'dependencies_res_files': ['<@(resource_input_paths)'],
 
+          'additional_res_input_dirs': ['<@(res_input_dirs)'],

[newt (away), 2013/05/03 20:16:16]

this should be "dependencies_res_input_dirs"

[Kibeom Kim (inactive), 2013/05/03 20:34:57]

Done.

+
           # Dependent APKs include this target's resources via
           # additional_res_dirs, additional_res_packages, and
           # additional_R_text_files.
-          'additional_res_dirs': ['<(res_crunched_dir)', '<@(res_input_dirs)'],
+          'additional_res_dirs': ['<(res_crunched_dir)',
+                                  '<(res_v17_dir)',

[newt (away), 2013/05/03 20:16:16]

put v14 before v17 for consistency

[Kibeom Kim (inactive), 2013/05/03 20:34:57]

Done.

+                                  '<(res_v14_dir)',
+                                  '<@(res_input_dirs)'],
           'additional_res_packages': ['<(R_package)'],
           'additional_R_text_files': ['<(R_text_file)'],
         },
       },
       'conditions': [
         ['java_strings_grd != ""', {
           'variables': {
             'res_grit_dir': '<(intermediate_dir)/res_grit',
             'res_input_dirs': ['<(res_grit_dir)'],
             'grit_grd_file': '<(java_in_dir)/strings/<(java_strings_grd)',
@@ skipping 15 matching lines @@
       ],
       'actions': [
         # Generate R.java and crunch image resources.
         {
           'action_name': 'process_resources',
           'message': 'processing resources for <(_target_name)',
           'variables': {
             'android_manifest': '<(DEPTH)/build/android/AndroidManifest.xml',
             # Include the dependencies' res dirs so that references to
             # resources in dependencies can be resolved.
-            'all_res_dirs': ['<@(res_input_dirs)', '>@(additional_res_dirs)'],
+            'all_res_dirs': ['<@(res_input_dirs)', '>@(additional_res_input_dirs)'],

[Kibeom Kim (inactive), 2013/05/03 18:46:03]

line too long.

[newt (away), 2013/05/03 20:16:16]

gyp doesn't have strict limits on line length, so go for what's most readable

[Kibeom Kim (inactive), 2013/05/03 20:34:57]

Done.

           },
           'inputs': [
             '<(DEPTH)/build/android/gyp/util/build_utils.py',
             '<(DEPTH)/build/android/gyp/process_resources.py',
             '>@(resource_input_paths)',
             '>@(dependencies_res_files)',
           ],
           'outputs': [
             '<(R_stamp)',
           ],
           'action': [
             'python', '<(DEPTH)/build/android/gyp/process_resources.py',
             '--android-sdk', '<(android_sdk)',
             '--android-sdk-tools', '<(android_sdk_tools)',
             '--R-dir', '<(R_dir)',
             '--res-dirs', '>(all_res_dirs)',
             '--crunch-input-dir', '>(res_dir)',
             '--crunch-output-dir', '<(res_crunched_dir)',
             '--android-manifest', '<(android_manifest)',
             '--non-constant-id',
             '--custom-package', '<(R_package)',
             '--stamp', '<(R_stamp)',
 
             # Add hash of inputs to the command line, so if inputs change
             # (e.g. if a resource if removed), the command will be re-run.
             # TODO(newt): remove this once crbug.com/177552 is fixed in ninja.
             '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
           ],
         },
+        # Copy API 17 resources.
+        {
+          'action_name': 'copy_v17_resources_<(_target_name)',
+          'message': 'Copying Android API 17 resources <(_target_name)',
+          'inputs': [
+            '<(DEPTH)/build/android/gyp/util/build_utils.py',
+            '<(DEPTH)/build/android/gyp/copy_v17_resources.py',
+            '>@(resource_input_paths)',
+          ],
+          'outputs': [
+            '<(res_v17_stamp)',
+          ],
+          'action': [
+            'python', '<(DEPTH)/build/android/gyp/copy_v17_resources.py',
+            '--res-dir=<(res_dir)',
+            '--res-v17-dir=<(res_v17_dir)',
+            '--stamp', '<(res_v17_stamp)',
+          ]
+        },
+        # Generate API 14 resources.
+        {
+          'action_name': 'generate_api_14_resources_<(_target_name)',
+          'message': 'Generating Android API 14 resources <(_target_name)',
+          'inputs': [
+            '<(DEPTH)/build/android/gyp/util/build_utils.py',
+            '<(DEPTH)/build/android/gyp/generate_v14_resources.py',
+            '>@(resource_input_paths)',
+          ],
+          'outputs': [
+            '<(res_v14_stamp)',
+          ],
+          'action': [
+            'python', '<(DEPTH)/build/android/gyp/generate_v14_resources.py',
+            '--res-dir=<(res_dir)',
+            '--res-v14-dir=<(res_v14_dir)',
+            '--stamp', '<(res_v14_stamp)',
+          ]
+        },
       ],
     }],
   ],
   'actions': [
     {
       'action_name': 'javac_<(_target_name)',
       'message': 'Compiling <(_target_name) java sources',
       'variables': {
         'all_src_dirs': [
           '>(java_in_dir)/src',
@@ skipping 86 matching lines @@
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
 
         '<(jar_path)',
       ]
     },
 
   ],
 }