Add input content checking to some build scripts

build/java_apk.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 Android APKs in a consistent manner.
 #
 # To use this, create a gyp target with the following form:
 # {
 #   'target_name': 'my_package_apk',
@@ skipping 212 matching lines @@
           },
           'dependencies': [
             '<(DEPTH)/tools/android/md5sum/md5sum.gyp:md5sum',
           ],
           'actions': [
             {
               'action_name': 'push_libraries_<(_target_name)',
               'message': 'Pushing libraries to device for <(_target_name)',
               'inputs': [
                 '<(DEPTH)/build/android/pylib/build_utils.py',
+                '<(DEPTH)/build/android/gyp/util/md5_check.py',

[Yaron, 2013/04/03 22:05:35]

I take it you couldn't find a more generic way to add this dependency (i.e.
depend on all python in build/android/gyp/?)

[cjhopman, 2013/04/03 22:17:56]

Once build_utils.py is moved, I plan to introduce a variable
android_gyp_utils='<!(find <(DEPTH)/build/android/gyp/util)' and expand that in
the inputs to all these actions. I.e. all the python actions will depend on all
the python build utilities.

The actions would still explicitly list there own script (e.g. push_libraries.py
here).

                 '<(DEPTH)/build/android/gyp/push_libraries.py',
                 '<(strip_stamp)',
               ],
               'outputs': [
                 '<(push_stamp)'
               ],
               'action': [
                 'python', '<(DEPTH)/build/android/gyp/push_libraries.py',
                 '--libraries-dir=<(apk_libraries_dir)',
                 '--device-dir=<(device_library_dir)',
@@ skipping 127 matching lines @@
       'variables': {
         'all_src_dirs': [
           '<(java_in_dir)/src',
           '<(intermediate_dir)/gen',
           '>@(additional_src_dirs)',
           '>@(generated_src_dirs)',
         ],
       },
       'inputs': [
         '<(DEPTH)/build/android/pylib/build_utils.py',
-        '<(DEPTH)/build/android/javac.py',
+        '<(DEPTH)/build/android/gyp/javac.py',
         # If there is a separate find for additional_src_dirs, it will find the
         # wrong .java files when additional_src_dirs is empty.
         '>!@(find >(java_in_dir) >(additional_src_dirs) -name "*.java")',
         '>@(input_jars_paths)',
         '<(codegen_stamp)',
         '>@(compile_input_paths)',
       ],
       'outputs': [
         '<(compile_stamp)',
       ],
       'action': [
-        'python', '<(DEPTH)/build/android/javac.py',
+        'python', '<(DEPTH)/build/android/gyp/javac.py',
         '--output-dir=<(classes_dir)',
         '--classpath=>(input_jars_paths) <(android_sdk_jar)',
         '--src-dirs=>(all_src_dirs)',
         '--javac-includes=<(javac_includes)',
         '--chromium-code=<(chromium_code)',
         '--stamp=<(compile_stamp)',
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
       ],
     },
     {
       'action_name': 'jar_<(_target_name)',
       'message': 'Creating <(_target_name) jar',
       'inputs': [
         '<(DEPTH)/build/android/pylib/build_utils.py',
-        '<(DEPTH)/build/android/jar.py',
+        '<(DEPTH)/build/android/gyp/util/md5_check.py',
+        '<(DEPTH)/build/android/gyp/jar.py',
         '<(compile_stamp)',
       ],
       'outputs': [
         '<(jar_stamp)',
       ],
       'action': [
-        'python', '<(DEPTH)/build/android/jar.py',
+        'python', '<(DEPTH)/build/android/gyp/jar.py',
         '--classes-dir=<(classes_dir)',
         '--jar-path=<(jar_path)',
         '--excluded-classes=<(jar_excluded_classes)',
         '--stamp=<(jar_stamp)',
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
       ]
     },
     {
@@ skipping 51 matching lines @@
               '>@(library_dexed_jars_paths)',
             ],
             'dex_generated_inputs': [
               '<(classes_dir)',
             ],
           }],
         ],
       },
       'inputs': [
         '<(DEPTH)/build/android/pylib/build_utils.py',
-        '<(DEPTH)/build/android/dex.py',
+        '<(DEPTH)/build/android/gyp/util/md5_check.py',
+        '<(DEPTH)/build/android/gyp/dex.py',
         '<(compile_stamp)',
         '>@(dex_inputs)',
       ],
       'outputs': [
         '<(dex_path)',
       ],
       'action': [
-        'python', '<(DEPTH)/build/android/dex.py',
+        'python', '<(DEPTH)/build/android/gyp/dex.py',
         '--dex-path=<(dex_path)',
         '--android-sdk-root=<(android_sdk_root)',
 
         # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja.
         '--ignore=>!(echo \'>(_inputs)\' | md5sum)',
 
         '>@(dex_inputs)',
         '>@(dex_generated_inputs)',
       ]
     },
@@ skipping 43 matching lines @@
 
         # Add list of inputs to the command line, so if inputs change
         # (e.g. if a Java file is removed), the command will be re-run.
         # TODO(newt): remove this once crbug.com/177552 is fixed in ninja.
         '-DTHIS_IS_IGNORED=>!(echo \'>(_inputs)\' | md5sum)',
 
       ]
     },
   ],
 }