Make javac and jar a single build action

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
 import re
 import sys
 
 from util import build_utils
 from util import md5_check
 
+import jar
+
 sys.path.append(build_utils.COLORAMA_ROOT)
 import colorama
 
 
 def ColorJavacOutput(output):
   fileline_prefix = r'(?P<fileline>(?P<file>[-.\w/\\]+.java):(?P<line>[0-9]+):)'
   warning_re = re.compile(
       fileline_prefix + r'(?P<full_message> warning: (?P<message>.*))$')
   error_re = re.compile(
       fileline_prefix + r'(?P<full_message> (?P<message>.*))$')
@@ skipping 16 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(options, args):
-  output_dir = options.output_dir
-
-  src_gendirs = build_utils.ParseGypList(options.src_gendirs)
-  java_files = args + build_utils.FindInDirectories(src_gendirs, '*.java')
-  if options.javac_includes:
-    javac_includes = build_utils.ParseGypList(options.javac_includes)
+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
 
   # 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
   java_files.sort()
-  classpath = build_utils.ParseGypList(options.classpath)
+  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',
       '-source', '1.5',
       '-target', '1.5',
       '-classpath', ':'.join(classpath),
-      '-d', output_dir]
-  if options.chromium_code:
+      '-d', classes_dir]
+  if chromium_code:
     javac_args.extend(['-Xlint:unchecked', '-Xlint:deprecation'])
   else:
     # XDignore.symbol.file makes javac compile against rt.jar instead of
     # ct.sym. This means that using a java internal package/class will not
     # trigger a compile warning or error.
     javac_args.extend(['-XDignore.symbol.file'])
 
   javac_cmd = ['javac'] + javac_args + java_files
 
   def Compile():
-    # Delete the 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(output_dir)
-    build_utils.MakeDirectory(output_dir)
     build_utils.CheckOutput(
         javac_cmd,
-        print_stdout=options.chromium_code,
+        print_stdout=chromium_code,
         stderr_filter=ColorJavacOutput)
 
-
-  record_path = '%s/javac.md5.stamp' % options.output_dir
+  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():
   colorama.init()
 
   parser = optparse.OptionParser()
-  parser.add_option('--src-gendirs',
+  parser.add_option(
+      '--src-gendirs',
       help='Directories containing generated java files.')
-  parser.add_option('--javac-includes',
+  parser.add_option('--classpath', help='Classpath for javac.')
+  parser.add_option(
+      '--javac-includes',
       help='A list of file patterns. If provided, only java files that match' +
-        'one of the patterns will be compiled.')
-  parser.add_option('--classpath', help='Classpath for javac.')
-  parser.add_option('--output-dir', help='Directory for javac output.')
+      'one of the patterns will be compiled.')
+  parser.add_option(
+      '--jar-excluded-classes',
+      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.')
-  parser.add_option('--chromium-code', type='int', help='Whether code being '
-                    'compiled should be built with stricter warnings for '
-                    'chromium code.')
 
   options, args = parser.parse_args()
 
-  DoJavac(options, args)
+  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:
+    DoJavac(
+        options.classpath,
+        options.javac_includes,
+        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.stamp:
     build_utils.Touch(options.stamp)
 
 
 if __name__ == '__main__':
   sys.exit(main())