diff options
| author | yfriedman@chromium.org <yfriedman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-01 23:52:40 +0000 |
|---|---|---|
| committer | yfriedman@chromium.org <yfriedman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-01 23:52:40 +0000 |
| commit | 9ac32ff8cdf76aa1bb2e4a27f2def96d8e1ba594 (patch) | |
| tree | b326461ccf7a8cfa518f16d48d680f573f4a00e0 | |
| parent | e9f0ca8b565cfba2bce76e3be60a8d19913b8174 (diff) | |
| download | chromium_src-9ac32ff8cdf76aa1bb2e4a27f2def96d8e1ba594.zip chromium_src-9ac32ff8cdf76aa1bb2e4a27f2def96d8e1ba594.tar.gz chromium_src-9ac32ff8cdf76aa1bb2e4a27f2def96d8e1ba594.tar.bz2 | |
[Android] Make build output a little quieter.
Suppress warnings from jsr305 by only enabling some warnings for chromium_code.
Also fixes the fact that components code wasn't setting chromium_code
Review URL: https://chromiumcodereview.appspot.com/12702017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191706 0039d316-1c4b-4281-b951-d872f2087c98
| -rwxr-xr-x | build/android/javac.py | 17 | ||||
| -rw-r--r-- | build/java.gypi | 1 | ||||
| -rw-r--r-- | build/java_apk.gypi | 1 |
3 files changed, 15 insertions, 4 deletions
diff --git a/build/android/javac.py b/build/android/javac.py index 9adf566..2b19185 100755 --- a/build/android/javac.py +++ b/build/android/javac.py @@ -40,16 +40,22 @@ def DoJavac(options): build_utils.DeleteDirectory(output_dir) build_utils.MakeDirectory(output_dir) - build_utils.CheckCallDie([ + cmd = [ 'javac', '-g', - '-Xlint:unchecked', '-source', '1.5', '-target', '1.5', '-classpath', ':'.join(classpath), - '-d', output_dir] + - java_files) + '-d', output_dir] + # Only output Java warnings for chromium code + if options.chromium_code: + cmd += ['-Xlint:unchecked'] + else: + cmd += [# Suppress "Sun proprietary API" warnings. See: goo.gl/OYxUM + '-XDignore.symbol.file'] + + build_utils.CheckCallDie(cmd + java_files) def main(argv): parser = optparse.OptionParser() @@ -60,6 +66,9 @@ def main(argv): parser.add_option('--classpath', help='Classpath for javac.') parser.add_option('--output-dir', help='Directory for javac output.') 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.') # TODO(newt): remove this once http://crbug.com/177552 is fixed in ninja. parser.add_option('--ignore', help='Ignored.') diff --git a/build/java.gypi b/build/java.gypi index 967b444..1bc952a 100644 --- a/build/java.gypi +++ b/build/java.gypi @@ -201,6 +201,7 @@ '--classpath=>(input_jars_paths)', '--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. diff --git a/build/java_apk.gypi b/build/java_apk.gypi index 8d57be8..d1ea1cd 100644 --- a/build/java_apk.gypi +++ b/build/java_apk.gypi @@ -309,6 +309,7 @@ '--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. |