Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(257)

Side by Side Diff: components/cronet/tools/generate_javadoc.py

Issue 2425563002: Revert of Upgrade to new Android SDK. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « components/cronet/android/api/build.xml ('k') | docs/android_test_instructions.md » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # 2 #
3 # Copyright 2015 The Chromium Authors. All rights reserved. 3 # Copyright 2015 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 import optparse 7 import optparse
8 import os 8 import os
9 import sys 9 import sys
10 import tempfile 10 import tempfile
(...skipping 25 matching lines...) Expand all
36 working_dir = os.path.join(options.input_dir, 'android/api') 36 working_dir = os.path.join(options.input_dir, 'android/api')
37 overview_file = os.path.abspath(options.overview_file) 37 overview_file = os.path.abspath(options.overview_file)
38 lib_java_dir = os.path.abspath(options.lib_java_dir) 38 lib_java_dir = os.path.abspath(options.lib_java_dir)
39 39
40 build_utils.DeleteDirectory(output_dir) 40 build_utils.DeleteDirectory(output_dir)
41 build_utils.MakeDirectory(output_dir) 41 build_utils.MakeDirectory(output_dir)
42 javadoc_cmd = ['ant', '-Dsource.dir=' + src_dir , '-Ddoc.dir=' + output_dir, 42 javadoc_cmd = ['ant', '-Dsource.dir=' + src_dir , '-Ddoc.dir=' + output_dir,
43 '-Dlib.java.dir=' + lib_java_dir, '-Doverview=' + overview_file, 43 '-Dlib.java.dir=' + lib_java_dir, '-Doverview=' + overview_file,
44 'doc'] 44 'doc']
45 stdout = build_utils.CheckOutput(javadoc_cmd, cwd=working_dir) 45 stdout = build_utils.CheckOutput(javadoc_cmd, cwd=working_dir)
46 # TODO(crbug.com/655666): add 'or "warning"' back once compiler 46 if " error: " in stdout or "warning" in stdout or "javadoc: error " in stdout:
47 # situation is resolved.
48 if " error: " in stdout or "javadoc: error " in stdout:
49 build_utils.DeleteDirectory(output_dir) 47 build_utils.DeleteDirectory(output_dir)
50 raise build_utils.CalledProcessError(working_dir, javadoc_cmd, stdout) 48 raise build_utils.CalledProcessError(working_dir, javadoc_cmd, stdout)
51 49
52 50
53 def main(): 51 def main():
54 parser = optparse.OptionParser() 52 parser = optparse.OptionParser()
55 build_utils.AddDepfileOption(parser) 53 build_utils.AddDepfileOption(parser)
56 parser.add_option('--output-dir', help='Directory to put javadoc') 54 parser.add_option('--output-dir', help='Directory to put javadoc')
57 parser.add_option('--input-dir', help='Root of cronet source') 55 parser.add_option('--input-dir', help='Root of cronet source')
58 parser.add_option('--input-src-jar', help='Cronet api source jar') 56 parser.add_option('--input-src-jar', help='Cronet api source jar')
(...skipping 22 matching lines...) Expand all
81 assert options.stamp 79 assert options.stamp
82 deps = [] 80 deps = []
83 for root, _, filenames in os.walk(options.input_dir): 81 for root, _, filenames in os.walk(options.input_dir):
84 deps.extend(os.path.join(root, f) for f in filenames) 82 deps.extend(os.path.join(root, f) for f in filenames)
85 build_utils.WriteDepfile(options.depfile, options.stamp, deps) 83 build_utils.WriteDepfile(options.depfile, options.stamp, deps)
86 # Clean up temporary output directory. 84 # Clean up temporary output directory.
87 build_utils.DeleteDirectory(unzipped_jar_path) 85 build_utils.DeleteDirectory(unzipped_jar_path)
88 86
89 if __name__ == '__main__': 87 if __name__ == '__main__':
90 sys.exit(main()) 88 sys.exit(main())
OLDNEW
« no previous file with comments | « components/cronet/android/api/build.xml ('k') | docs/android_test_instructions.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698