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 gradle #33

Draft
wants to merge 15 commits into
base: develop
Choose a base branch
from
Draft

Update gradle #33

wants to merge 15 commits into from

Conversation

ninovanhooff
Copy link
Collaborator

No description provided.

Copy link
Collaborator Author

@ninovanhooff ninovanhooff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WARNING Make sure CI can build this.

JitCi's dropdown does not seem to provide jdk 21
https://jitci.com/gh/Q42/Q42Stats.Android

But a yml file may be the solution... jitpack/jitpack.io#6479

If we want to stay on JitPack instead of Maven in the first place
#29

UPDATE
None of the methods I tried in this branch to get jitpack to use jdk 17 or 21 worked

jitpack.yml Outdated
jdk:
- openjdk17
before_install:
- echo "Hello jitpack.yml before_install"
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file seems to not be picked up at all; I don't see this being printed to the logs

@@ -63,6 +68,24 @@ afterEvaluate {
// Applies the component for the release build variant.
from components.release

pom {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems not to work, Jitpack still builds using java 11

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.

1 participant