Remove apktool-cli from Maven publications to stay under 80MB limit - #4190
Remove apktool-cli from Maven publications to stay under 80MB limit#4190iBotPeaches with Copilot wants to merge 5 commits into
Conversation
Good point — kept With the This should bring the total Maven deployment well under 80MB. |
@copilot - even the small yaml library is quite large. Why? https://repo1.maven.org/maven2/org/apktool/apktool-lib/3.0.3/ |
Two issues were causing this:
Combined with the thin JAR fix for |
|
@copilot - thanks, but 2 things. Removing prebuilt will break it, so thats dumb. Is removing the javadoc valid? That seems like it will break the services that render things. I think the angle of fat binaries is probably the fix since that seems moot against maven |
The thin JAR fix (not publishing apktool-cli fat shadow JAR) is sufficient to bring Maven deployment under 80MB. Restore proper javadoc generation and full sources JAR including prebuilt binaries.
Reverted both the empty javadoc stub and the prebuilt exclusion from the sources JAR in The fat binary was the bulk of the 80.16MB — removing it from the Maven publication should comfortably bring the deployment under 80MB without touching javadoc or sources. |
Maven Central deployment was at 80.16MB, just over the 80MB limit. The
apktool-climodule was the culprit — it published a fat shadow JAR bundling all transitive dependencies (smali, baksmali, guava, commons-*), which is inappropriate for Maven consumption anyway. Maven users should depend onapktool-lib; the CLI fat JAR is for end-user distribution via GitHub Releases.Changes
build.gradle.kts— Remove"apktool-cli"frommavenProjects, preventing the vanniktech publish plugin from being applied to it:brut.apktool/apktool-cli/build.gradle.kts— Remove now-deadPublishToMavenRepository,Sign, andGenerateModuleMetadatatask wiring that existed solely to sequence publishing after shadow JAR creation.