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

Update JDK 8 to JDK 11 #632

Merged
merged 1 commit into from
Jan 31, 2022
Merged

Update JDK 8 to JDK 11 #632

merged 1 commit into from
Jan 31, 2022

Conversation

nohe427
Copy link
Collaborator

@nohe427 nohe427 commented Dec 8, 2021

This updates the JDK used to match the JDK that is currently distributed
by Android Studio. This would help keep the building of Android apps
consistent between Android Studio and Bubblewrap. Additionally, this
unlocks the ability for bubblewrap to support keystores that utilize
aes encrypted keys.

This updates the JDK used to match the JDK that is currently distributed
by Android Studio. This would help keep the building of Android apps
consistent between Android Studio and Bubblewrap. Additionally, this
unlocks the ability for bubblewrap to support keystores that utilize
aes encrypted keys.
@nohe427 nohe427 requested a review from andreban December 8, 2021 19:08
@nohe427
Copy link
Collaborator Author

nohe427 commented Dec 8, 2021

Additional notes:

  • tested on linux
  • need to test on mac and windows (lack hardware)
  • This matches the JDK that comes with Android Studio (last I checked)

@nohe427 nohe427 requested a review from PEConn January 26, 2022 16:51
@PEConn
Copy link
Collaborator

PEConn commented Jan 27, 2022

Just to double check, you've testing bubblewrap build specifically?

This LGTM if it works.

@nohe427
Copy link
Collaborator Author

nohe427 commented Jan 27, 2022

@PEConn - Yes, tested building and signing on Linux. Haven't been able to test alternative platforms

@nohe427 nohe427 merged commit 0637928 into main Jan 31, 2022
@andreban andreban added the enhancement New feature or request label Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants