android: Pass (non-generated) .java files via a gyp filelist to javac.py

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 495 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)',
       ],
     },
     {
       'action_name': 'javac_<(_target_name)',
       'message': 'Compiling java for <(_target_name)',
       'variables': {
-        'all_src_dirs': [
-          '<(java_in_dir)/src',
+        'gen_src_dirs': [
           '<(intermediate_dir)/gen',

[cjhopman, 2014/03/11 01:35:57]

Ugh, this path is another hidden dependency (it is constructed from
<(intermediate_dir) in apk-codegen.xml).

-          '>@(additional_src_dirs)',
           '>@(generated_src_dirs)',
         ],
+        # If there is a separate find for additional_src_dirs, it will find the
+        # wrong .java files when additional_src_dirs is empty.
+        'java_source_list': '>|(javasources.<(_target_name).gypcmd >!@(find >(java_in_dir)/src >(additional_src_dirs) -name "*.java"))',

[Nico, 2014/03/11 01:41:59]

I changed the >(java_in_dir) on the lhs to >(java_in_dir)/src, since that's what
was in all_src_dirs. Turns out this caused the red trybot: Both memconsumer and
protobuf.gyp use java_in_dir = 'java' , and gyp caches commands with identical
spelling. So both would evaluate to `find java -name "*.java"`. The reason why
it worked before this patch was that memconsumer included java_apk.gypi and
protobuf included java.gypi, and the former ran find in >(java_in_dir)/src and
the latter in >(java_in_dir) (thus, the two commands weren't completely
identical) (!)

I changed this back to not tack on "/src" for now, but urgh.

…I suppose you don't just list all .java files because of historical reasons
(ant)? Maybe we should move to listing all java files explicitly? (The
alternative is to add some cache-busting piece to the command, but that's urgh
too.)

+
       },
       'inputs': [
         '<(DEPTH)/build/android/gyp/util/build_utils.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")',
+        '>(java_source_list)',
         '>@(input_jars_paths)',
         '<(codegen_stamp)',
         '>@(compile_input_paths)',
       ],
       'outputs': [
         '<(compile_stamp)',
       ],
       'action': [
         'python', '<(DEPTH)/build/android/gyp/javac.py',
         '--output-dir=<(classes_dir)',
         '--classpath=>(input_jars_paths) <(android_sdk_jar)',
-        '--src-dirs=>(all_src_dirs)',
+        '--src-filelist=>(java_source_list)',
+        '--src-gendirs=>(gen_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)',
       ],
     },
     {
       'variables': {
@@ skipping 202 matching lines @@
         '<(DEPTH)/build/android/ant/apk-package.xml',
 
         # 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)',
       ]
     },
   ],
 }