Skip to content

Use JDK 8 for release #252

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

Merged
merged 1 commit into from
Apr 8, 2023

Conversation

mdedetrich
Copy link
Contributor

Not sure why the -release:8 scalac flag didn't work when releasing a new version but one way to enforce JDK 8 compatibility is to just use JDK8 when releasing.

Also since we are building against JDK 8, I added it to the normal CI to make sure we don't break JDK 8 compatiblity

@mdedetrich
Copy link
Contributor Author

mdedetrich commented Apr 2, 2023

@ruippeixotog So if you look at the CI run in this PR against JDK 8 I think I figured out the issue, it appears that the problem is in a dependency , i.e. org/eclipse/jgit/storage/file/FileRepositoryBuilder. This has been solved in a separate PR made at #253.

This means that the -release:8 argument technically speaking is working because its making sure that this project emits JDK 8 compatible bytecode but of course it cannot magically convert the dependencies of this project to also target JDK 8.

@ruippeixotog
Copy link
Contributor

Thanks for the quick fix, @mdedetrich! Agree that the best way to guarantee it works with JDK 8 (including dependencies) is to have CI use it.

@rolandtritsch rolandtritsch merged commit 9f7e93b into scoverage:main Apr 8, 2023
@rolandtritsch
Copy link
Member

Also since we are building against JDK 8, I added it to the normal CI to make sure we don't break JDK 8 compatiblity

Agreed. Merged. Thanks for the fix.

@mdedetrich mdedetrich deleted the use-jdk8-for-release branch April 8, 2023 08:12
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