Commit 955ee6ad authored by Fumitoshi Ukai's avatar Fumitoshi Ukai Committed by Commit Bot

disable use_java_goma

when enabled java, errorprone for NoRedundantFieldInitCheck fails.

disable use_java_goma as it is not a priority for now.

Bug: b/167942918, 726475
Change-Id: I1e864ae881225bca4879b7aa3f47af5ddc8ac54c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2398864Reviewed-by: default avatarTakuto Ikuta <tikuta@chromium.org>
Reviewed-by: default avatarYoshisato Yanagisawa <yyanagisawa@google.com>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
Cr-Commit-Position: refs/heads/master@{#805241}
parent 3a3feb45
...@@ -401,7 +401,8 @@ ...@@ -401,7 +401,8 @@
'Chromium Win Goma RBE Staging (clobber)': 'release_bot_x86_minimal_symbols', 'Chromium Win Goma RBE Staging (clobber)': 'release_bot_x86_minimal_symbols',
'Chromium Android ARM 32-bit Goma RBE ToT': 'android_release_bot_minimal_symbols', 'Chromium Android ARM 32-bit Goma RBE ToT': 'android_release_bot_minimal_symbols',
'Chromium Android ARM 32-bit Goma RBE ToT (ATS)': 'android_release_bot_minimal_symbols_java_goma', # TODO(b/167942918): re-enable java goma?
'Chromium Android ARM 32-bit Goma RBE ToT (ATS)': 'android_release_bot_minimal_symbols',
'Chromium Android ARM 32-bit Goma RBE Staging': 'android_release_bot_minimal_symbols', 'Chromium Android ARM 32-bit Goma RBE Staging': 'android_release_bot_minimal_symbols',
'chromeos-amd64-generic-rel-goma-rbe-tot': 'chromeos_amd64-generic', 'chromeos-amd64-generic-rel-goma-rbe-tot': 'chromeos_amd64-generic',
...@@ -1262,12 +1263,6 @@ ...@@ -1262,12 +1263,6 @@
'android', 'release_bot', 'minimal_symbols', 'strip_debug_info', 'android', 'release_bot', 'minimal_symbols', 'strip_debug_info',
], ],
# TODO(crbug.com/726475): Remove this when we use goma for javac by default.
'android_release_bot_minimal_symbols_java_goma': [
'android', 'release_bot', 'minimal_symbols', 'strip_debug_info',
'use_java_goma',
],
'android_release_bot_minimal_symbols_arm64': [ 'android_release_bot_minimal_symbols_arm64': [
'android', 'release_bot', 'minimal_symbols', 'arm64', 'android', 'release_bot', 'minimal_symbols', 'arm64',
'strip_debug_info', 'strip_debug_info',
...@@ -3080,11 +3075,6 @@ ...@@ -3080,11 +3075,6 @@
'gn_args': 'use_jacoco_coverage=true', 'gn_args': 'use_jacoco_coverage=true',
}, },
# TODO(crbug.com/726475): Remove this when we use goma for javac by default.
'use_java_goma': {
'gn_args': 'use_java_goma=true',
},
'v8_simulate_arm': { 'v8_simulate_arm': {
'gn_args': 'target_cpu="x86" v8_target_cpu="arm"', 'gn_args': 'target_cpu="x86" v8_target_cpu="arm"',
}, },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment