Revert "Use R8 for resource shrinking"
Revert submission 2816534-r8resourceshrinking
Reason for revert: b/309075997
Reverted changes: /q/submissionid:2816534-r8resourceshrinking
Change-Id: Ic173491eaf22967fbad14fbcc85f6ddb2cf54d23
diff --git a/java/dex.go b/java/dex.go
index cf0a94c..9ce5053 100644
--- a/java/dex.go
+++ b/java/dex.go
@@ -95,8 +95,6 @@
proguardDictionary android.OptionalPath
proguardConfiguration android.OptionalPath
proguardUsageZip android.OptionalPath
- resourcesInput android.OptionalPath
- resourcesOutput android.OptionalPath
providesTransitiveHeaderJars
}
@@ -163,7 +161,7 @@
"$r8Template": &remoteexec.REParams{
Labels: map[string]string{"type": "compile", "compiler": "r8"},
Inputs: []string{"$implicits", "${config.R8Jar}"},
- OutputFiles: []string{"${outUsage}", "${outConfig}", "${outDict}", "${resourcesOutput}"},
+ OutputFiles: []string{"${outUsage}", "${outConfig}", "${outDict}"},
ExecStrategy: "${config.RER8ExecStrategy}",
ToolchainInputs: []string{"${config.JavaCmd}"},
Platform: map[string]string{remoteexec.PoolKey: "${config.REJavaPool}"},
@@ -183,7 +181,7 @@
Platform: map[string]string{remoteexec.PoolKey: "${config.REJavaPool}"},
},
}, []string{"outDir", "outDict", "outConfig", "outUsage", "outUsageZip", "outUsageDir",
- "r8Flags", "zipFlags", "mergeZipsFlags", "resourcesOutput"}, []string{"implicits"})
+ "r8Flags", "zipFlags", "mergeZipsFlags"}, []string{"implicits"})
func (d *dexer) dexCommonFlags(ctx android.ModuleContext,
dexParams *compileDexParams) (flags []string, deps android.Paths) {
@@ -350,12 +348,6 @@
r8Flags = append(r8Flags, "-ignorewarnings")
}
- if d.resourcesInput.Valid() {
- r8Flags = append(r8Flags, "--resource-input", d.resourcesInput.Path().String())
- r8Deps = append(r8Deps, d.resourcesInput.Path())
- r8Flags = append(r8Flags, "--resource-output", d.resourcesOutput.Path().String())
- }
-
return r8Flags, r8Deps
}
@@ -397,8 +389,6 @@
android.ModuleNameWithPossibleOverride(ctx), "unused.txt")
proguardUsageZip := android.PathForModuleOut(ctx, "proguard_usage.zip")
d.proguardUsageZip = android.OptionalPathForPath(proguardUsageZip)
- resourcesOutput := android.PathForModuleOut(ctx, "package-res-shrunken.apk")
- d.resourcesOutput = android.OptionalPathForPath(resourcesOutput)
r8Flags, r8Deps := d.r8Flags(ctx, dexParams.flags)
r8Deps = append(r8Deps, commonDeps...)
rule := r8
@@ -417,22 +407,17 @@
rule = r8RE
args["implicits"] = strings.Join(r8Deps.Strings(), ",")
}
- implicitOutputs := android.WritablePaths{
- proguardDictionary,
- proguardUsageZip,
- proguardConfiguration}
- if d.resourcesInput.Valid() {
- implicitOutputs = append(implicitOutputs, resourcesOutput)
- args["resourcesOutput"] = resourcesOutput.String()
- }
ctx.Build(pctx, android.BuildParams{
- Rule: rule,
- Description: "r8",
- Output: javalibJar,
- ImplicitOutputs: implicitOutputs,
- Input: dexParams.classesJar,
- Implicits: r8Deps,
- Args: args,
+ Rule: rule,
+ Description: "r8",
+ Output: javalibJar,
+ ImplicitOutputs: android.WritablePaths{
+ proguardDictionary,
+ proguardUsageZip,
+ proguardConfiguration},
+ Input: dexParams.classesJar,
+ Implicits: r8Deps,
+ Args: args,
})
} else {
d8Flags, d8Deps := d8Flags(dexParams.flags)