Add android_library template

build/android/gyp/javac.py

 #!/usr/bin/env python
 #
 # Copyright 2013 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.
 
 import fnmatch
 import optparse
 import os
 import shutil
@@ skipping 34 matching lines @@
     if warning_re.match(line):
       line = Colorize(line, warning_re, warning_color)
     elif error_re.match(line):
       line = Colorize(line, error_re, error_color)
     elif marker_re.match(line):
       line = Colorize(line, marker_re, marker_color)
     return line
 
   return '\n'.join(map(ApplyColor, output.split('\n')))
 
+
 def DoJavac(
-    classpath, javac_includes, classes_dir, chromium_code, java_files):
-  if javac_includes:
-    javac_includes = build_utils.ParseGypList(javac_includes)
-    filtered_java_files = []
-    for f in java_files:
-      for include in javac_includes:
-        if fnmatch.fnmatch(f, include):
-          filtered_java_files.append(f)
-          break
-    java_files = filtered_java_files
+    classpath, classes_dir, chromium_code, java_files):
+  """Runs javac.
+
+  Builds |java_files| with the provided |classpath| and puts the generated
+  .class files into |classes_dir|. If |chromium_code| is true, extra lint
+  checking will be enabled.
+  """
 
   # Compiling guava with certain orderings of input files causes a compiler
   # crash... Sorted order works, so use that.
   # See https://code.google.com/p/guava-libraries/issues/detail?id=950
+  # TODO(cjhopman): Remove this when we have update guava or the compiler to a
+  # version without this problem.
   java_files.sort()
-  classpath = build_utils.ParseGypList(classpath)
 
   jar_inputs = []
   for path in classpath:
     if os.path.exists(path + '.TOC'):
       jar_inputs.append(path + '.TOC')
     else:
       jar_inputs.append(path)
 
   javac_args = [
       '-g',
@@ skipping 18 matching lines @@
         stderr_filter=ColorJavacOutput)
 
   record_path = os.path.join(classes_dir, 'javac.md5.stamp')
   md5_check.CallAndRecordIfStale(
       Compile,
       record_path=record_path,
       input_paths=java_files + jar_inputs,
       input_strings=javac_cmd)
 
 
-def main():
+def main(argv):
   colorama.init()
 
+  argv = build_utils.ExpandFileArgs(argv)
+
   parser = optparse.OptionParser()
   build_utils.AddDepfileOption(parser)
 
   parser.add_option(
       '--src-gendirs',
       help='Directories containing generated java files.')
-  parser.add_option('--classpath', help='Classpath for javac.')
+  parser.add_option(
+      '--java-srcjars',
+      help='List of srcjars to include in compilation.')
+  classpath_args = []
+  parser.add_option(
+      '--classpath',
+      type='string',
+      action='callback',
+      callback=lambda _1, _2, v, _3 : classpath_args.append(v),

[Nico, 2014/07/01 18:24:51]

Isn't this just `action='append'`?
<Option at 0x10ee566c8: --classpath>
<Values at 0x10ee567e8: {'classpath': ['foo', 'bar']}>
[]

[cjhopman, 2014/07/01 22:14:01]

Done.

+      help='Classpath for javac. If this is specified multiple times, they '
+      'will all be appended to construct the classpath.')
   parser.add_option(
       '--javac-includes',
-      help='A list of file patterns. If provided, only java files that match' +
+      help='A list of file patterns. If provided, only java files that match'
       'one of the patterns will be compiled.')
   parser.add_option(
       '--jar-excluded-classes',
+      default='',
       help='List of .class file patterns to exclude from the jar.')
 
   parser.add_option(
       '--chromium-code',
       type='int',
       help='Whether code being compiled should be built with stricter '
       'warnings for chromium code.')
 
   parser.add_option(
       '--classes-dir',
       help='Directory for compiled .class files.')
   parser.add_option('--jar-path', help='Jar output path.')
 
   parser.add_option('--stamp', help='Path to touch on success.')
 
-  options, args = parser.parse_args()
+  options, args = parser.parse_args(argv)
+
+  classpath = []
+  for arg in classpath_args:
+    classpath += build_utils.ParseGypList(arg)
 
   java_files = args
   if options.src_gendirs:
     src_gendirs = build_utils.ParseGypList(options.src_gendirs)
     java_files += build_utils.FindInDirectories(src_gendirs, '*.java')
 
-  with build_utils.TempDir() as classes_dir:
+  with build_utils.TempDir() as temp_dir:
+    classes_dir = os.path.join(temp_dir, 'classes')
+    os.makedirs(classes_dir)
+    if options.java_srcjars:
+      java_dir = os.path.join(temp_dir, 'java')
+      os.makedirs(java_dir)
+      for srcjar in build_utils.ParseGypList(options.java_srcjars):
+        build_utils.ExtractAll(srcjar, path=java_dir)
+      java_files += build_utils.FindInDirectory(java_dir, '*.java')
+
+    if options.javac_includes:
+      javac_includes = build_utils.ParseGypList(options.javac_includes)
+      filtered_java_files = []
+      for f in java_files:
+        for include in javac_includes:
+          if fnmatch.fnmatch(f, include):
+            filtered_java_files.append(f)
+            break
+      java_files = filtered_java_files

[Nico, 2014/07/01 18:24:51]

This changes how gyp does things too, right? If so, mention that somewhere early
in the cl description.

[cjhopman, 2014/07/01 22:14:01]

the javac_includes is just moved here from the beginning of DoJavac. 

the java_srcjars thing is new, but is not yet used in gyp. It is preferred to
use that over src_gendirs and I'd like to change everything to just use srcjars,
but it is difficult in some cases in gyp (see my earlier comment here
https://codereview.chromium.org/269943005/diff/20001/build/android/gn/javac.p...)

+
     DoJavac(
-        options.classpath,
-        options.javac_includes,
+        classpath,
         classes_dir,
         options.chromium_code,
         java_files)
 
     if options.jar_path:
       jar.JarDirectory(classes_dir,
                        build_utils.ParseGypList(options.jar_excluded_classes),
                        options.jar_path)
 
     if options.classes_dir:
       # Delete the old classes directory. This ensures that all .class files in
       # the output are actually from the input .java files. For example, if a
       # .java file is deleted or an inner class is removed, the classes
       # directory should not contain the corresponding old .class file after
       # running this action.
       build_utils.DeleteDirectory(options.classes_dir)
       shutil.copytree(classes_dir, options.classes_dir)
 
   if options.depfile:
     build_utils.WriteDepfile(
         options.depfile,
-        build_utils.GetPythonDependencies())
+        classpath + build_utils.GetPythonDependencies())
 
   if options.stamp:
     build_utils.Touch(options.stamp)
 
 
 if __name__ == '__main__':
-  sys.exit(main())
+  sys.exit(main(sys.argv[1:]))