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

Fix Java 8 compatibility issue when building on Java9+ (https://github.com/castor-software/depclean/issues/25) #29

Merged
merged 2 commits into from
Jan 14, 2021

Conversation

amottier
Copy link
Contributor

#25

Use maven.compiler.release to avoid runtime issue
As maven.compiler.release lead to the usage of -release javac option.
Removed maven.compiler.source and maven.compiler.target that are no longer useful.
Important: project cannot be build anymore with Java 8 because javac version 8 does not support --release option added by Maven property

Use `maven.compiler.release` to avoid runtime issue (see ASSERT-KTH#25).
As `maven.compiler.release` lead to the usage of `-release` `javac` option.
Removed `maven.compiler.source` and `maven.compiler.target` that are no longer useful.
Important: project cannot be build anymore with Java 8 because `javac` version 8 does not support `--release` option added by Maven property `maven.compiler.release`.
To prepare for future development.
@cesarsotovalero cesarsotovalero self-requested a review January 14, 2021 22:07
@cesarsotovalero cesarsotovalero self-assigned this Jan 14, 2021
@cesarsotovalero cesarsotovalero added the bug Something isn't working label Jan 14, 2021
@cesarsotovalero cesarsotovalero changed the title #25 Fix Java 8 compatibility issue when building on Java9+ Fix Java 8 compatibility issue when building on Java9+ (https://github.com/castor-software/depclean/issues/25) Jan 14, 2021
@cesarsotovalero cesarsotovalero merged commit dde6d48 into ASSERT-KTH:master Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants