Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change APK base name depending on the branch name for debug builds #3265

Merged
merged 1 commit into from
Apr 2, 2020

Conversation

TobiGr
Copy link
Member

@TobiGr TobiGr commented Mar 25, 2020

What is it?

  • Bug fix
  • Feature
  • Development improvement

Long description of the changes in your PR

Debug APKs are now named after their branch name as suggested by @opusforlife2 e.g. the debug APK for this branch is named NewPipe_apk_base_name-debug.apk
Release builds are not touched, as well as debug builds on dev and master branch.

Testing apk

not needed

Agreement

app/build.gradle Outdated Show resolved Hide resolved
Stypox
Stypox previously approved these changes Mar 26, 2020
@Stypox
Copy link
Member

Stypox commented Apr 2, 2020

@TobiGr am I supposed to re-review this for some reason?

@TobiGr
Copy link
Member Author

TobiGr commented Apr 2, 2020

@Stypox no.

@TobiGr TobiGr merged commit dc6604b into dev Apr 2, 2020
@TobiGr TobiGr deleted the apk_base_name branch April 2, 2020 21:39
@Bubu
Copy link

Bubu commented Apr 10, 2020

This is somehow not working as intended (as it affects release builds). Even when doing a release build the apk is now called:
NewPipe_head-release-unsigned.apk. Also a file in the META-INF dir now ends up being called NewPipe_head_release.kotlin_module instead of app_release.kotlin_module. This is somehow not the case in you 0.19.2 github release build, so this breaks any chance for reproducibility unfortunately.

And how do you generate the github release build? How does this not end up containing the kotlin_module rename?

I run ./gradlew clean assembleRelease.

Reverting this change, everything returns to normal.

@TobiGr
Copy link
Member Author

TobiGr commented Apr 10, 2020

@B0pol @Bubu Thanks for notifying us! I tested my changes locally when creating the PR and did not notice and changes in the release builds. Tbh. I have no idea why release builds are effected by this change. I can try to force the old naming of the release by adding archivesBaseName = app to the release part of the Gradle config.

And how do you generate the github release build? How does this not end up containing the kotlin_module rename?

I build the app through Android studio by first doing clean project which is the equivalent of ./gradlew clean and then build the unsigned app Build > Build Bundle(s) / APK(s) > Build APK which runs ./gradlew assembleRelease. Signing is done in another step.

@Stypox
Copy link
Member

Stypox commented Apr 10, 2020

I think you meant @Bubu, not @B0pol ;-)

TobiGr added a commit that referenced this pull request Apr 10, 2020
Fixes the behaviour described in #3265 (comment)
@TobiGr TobiGr mentioned this pull request Apr 10, 2020
5 tasks
TobiGr added a commit that referenced this pull request Apr 11, 2020
Fixes the behaviour described in #3265 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants