Skip to content

Commit a1e745b

Browse files
ajsinclaircopybara-github
authored andcommitted
Remove unused branch in generate_main_dex_list.
The legacy_main_dex_list_generator is always provided. PiperOrigin-RevId: 705205900 Change-Id: Idb9ea45736892220de0336ce026cb676b5d23d5e
1 parent bd6d0e3 commit a1e745b

File tree

5 files changed

+19
-72
lines changed

5 files changed

+19
-72
lines changed

rules/android_binary/impl.bzl

-1
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,6 @@ def _process_dex(ctx, validation_ctx, packaged_resources_ctx, deploy_ctx, bp_ctx
301301
main_dex_list_opts = ctx.attr.main_dex_list_opts,
302302
main_dex_proguard_spec = packaged_resources_ctx.main_dex_proguard_config,
303303
proguard_specs = list(ctx.files.main_dex_proguard_specs),
304-
shrinked_android_jar = get_android_sdk(ctx).shrinked_android_jar,
305304
main_dex_list_creator = get_android_sdk(ctx).main_dex_list_creator,
306305
legacy_main_dex_list_generator =
307306
ctx.attr._legacy_main_dex_list_generator.files_to_run if ctx.attr._legacy_main_dex_list_generator else get_android_sdk(ctx).legacy_main_dex_list_generator,

rules/android_sdk.bzl

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ def _impl(ctx):
2727
aidl_lib = None,
2828
android_jar = ctx.file.android_jar,
2929
source_properties = ctx.file.source_properties,
30-
shrinked_android_jar = ctx.file.shrinked_android_jar,
30+
shrinked_android_jar = None,
3131
main_dex_classes = ctx.file.main_dex_classes,
3232
adb = ctx.attr.adb.files_to_run,
3333
dx = ctx.attr.dx.files_to_run,

rules/android_sdk_repository/helper.bzl

-2
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,6 @@ def create_android_sdk_rules(
226226
":disallow_proguard": ":fail",
227227
"//conditions:default": "@bazel_tools//tools/jdk:proguard",
228228
}),
229-
shrinked_android_jar = "platforms/android-%d/android.jar" % api_level,
230229
# See https://github.com/bazelbuild/bazel/issues/8757
231230
tags = ["__ANDROID_RULES_MIGRATION__"],
232231
zipalign = select({
@@ -583,7 +582,6 @@ def create_dummy_sdk_toolchain():
583582
main_dex_classes = "dummy.jar",
584583
main_dex_list_creator = ":empty-binary",
585584
proguard = ":empty-binary",
586-
shrinked_android_jar = "dummy.jar",
587585
tags = ["__ANDROID_RULES_MIGRATION__"],
588586
zipalign = ":empty-binary",
589587
)

rules/attrs.bzl

-4
Original file line numberDiff line numberDiff line change
@@ -303,10 +303,6 @@ ANDROID_SDK_ATTRS = dict(
303303
executable = True,
304304
mandatory = True,
305305
),
306-
shrinked_android_jar = attr.label(
307-
allow_single_file = True,
308-
cfg = "exec",
309-
),
310306
source_properties = attr.label(
311307
allow_single_file = True,
312308
cfg = "exec",

rules/dex.bzl

+18-64
Original file line numberDiff line numberDiff line change
@@ -726,7 +726,6 @@ def _generate_main_dex_list(
726726
main_dex_proguard_spec = None,
727727
proguard_specs = [],
728728
legacy_apis = [],
729-
shrinked_android_jar = None,
730729
toolchain_type = None,
731730
main_dex_list_creator = None,
732731
legacy_main_dex_list_generator = None,
@@ -737,70 +736,25 @@ def _generate_main_dex_list(
737736
if main_dex_proguard_spec:
738737
proguard_specs.append(main_dex_proguard_spec)
739738

740-
# If legacy_main_dex_list_generator is not set by either the SDK or the flag, use ProGuard and
741-
# the main dext list creator specified by the android_sdk rule. If
742-
# legacy_main_dex_list_generator is provided, use that tool instead.
743-
# TODO(b/147692286): Remove the old main-dex list generation that relied on ProGuard.
744-
if not legacy_main_dex_list_generator:
745-
if not shrinked_android_jar:
746-
fail("In \"legacy\" multidex mode, either legacy_main_dex_list_generator or " +
747-
"shrinked_android_jar must be set in the android_sdk.")
748-
749-
# Process the input jar through Proguard into an intermediate, streamlined jar.
750-
stripped_jar = _get_dx_artifact(ctx, "main_dex_intermediate.jar")
751-
args = ctx.actions.args()
752-
args.add("-forceprocessing")
753-
args.add("-injars", jar)
754-
args.add("-libraryjars", shrinked_android_jar)
755-
args.add("-outjars", stripped_jar)
756-
args.add("-dontwarn")
757-
args.add("-dontnote")
758-
args.add("-dontoptimize")
759-
args.add("-dontobfuscate")
760-
ctx.actions.run(
761-
outputs = [stripped_jar],
762-
executable = proguard_tool,
763-
args = [args],
764-
inputs = [jar, shrinked_android_jar],
765-
mnemonic = "MainDexClassesIntermediate",
766-
progress_message = "Generating streamlined input jar for main dex classes list",
767-
use_default_shell_dev = True,
768-
toolchain = toolchain_type,
769-
)
770-
771-
args = ctx.actions.args()
772-
args.add_all([main_dex_list, stripped_jar, jar])
773-
args.add_all(main_dex_list_opts)
774-
775-
ctx.actions.run(
776-
outputs = [main_dex_list],
777-
executable = main_dex_list_creator,
778-
arguments = [args],
779-
inputs = [jar, stripped_jar],
780-
mnemonic = "MainDexClasses",
781-
progress_message = "Generating main dex classes list",
782-
toolchain = toolchain_type,
783-
)
784-
else:
785-
inputs = [jar, android_jar] + proguard_specs
739+
inputs = [jar, android_jar] + proguard_specs
786740

787-
args = ctx.actions.args()
788-
args.add("--main-dex-list-output", main_dex_list)
789-
args.add("--lib", android_jar)
790-
if desugar_java8_libs:
791-
args.add_all(legacy_apis, before_each = "--lib")
792-
inputs += legacy_apis
793-
args.add_all(proguard_specs, before_each = "--main-dex-rules")
794-
args.add(jar)
795-
ctx.actions.run(
796-
executable = legacy_main_dex_list_generator,
797-
arguments = [args],
798-
outputs = [main_dex_list],
799-
inputs = inputs,
800-
mnemonic = "MainDexClasses",
801-
progress_message = "Generating main dex classes list",
802-
toolchain = toolchain_type,
803-
)
741+
args = ctx.actions.args()
742+
args.add("--main-dex-list-output", main_dex_list)
743+
args.add("--lib", android_jar)
744+
if desugar_java8_libs:
745+
args.add_all(legacy_apis, before_each = "--lib")
746+
inputs += legacy_apis
747+
args.add_all(proguard_specs, before_each = "--main-dex-rules")
748+
args.add(jar)
749+
ctx.actions.run(
750+
executable = legacy_main_dex_list_generator,
751+
arguments = [args],
752+
outputs = [main_dex_list],
753+
inputs = inputs,
754+
mnemonic = "MainDexClasses",
755+
progress_message = "Generating main dex classes list",
756+
toolchain = toolchain_type,
757+
)
804758
return main_dex_list
805759

806760
def _transform_dex_list_through_proguard_map(

0 commit comments

Comments
 (0)