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

ISSUE-1064 Bump rdf4j.version from 3.7.4 to 4.0.1 #1065

Closed
wants to merge 1 commit into from

Conversation

lewismc
Copy link
Contributor

@lewismc lewismc commented Jun 3, 2022

Hi Folks, this my a PR to address #1064
I am running OpenJDK 11.0.15 hence the tangential additions to this PR. I was unable to get the project to build otherwise.

Apache Maven 3.8.4 (9b656c72d54e5bacbed989b64718c159fe39b537)
Maven home: /usr/local/Cellar/maven/3.8.4/libexec
Java version: 11.0.15, vendor: Homebrew, runtime: /usr/local/Cellar/openjdk@11/11.0.15/libexec/openjdk.jdk/Contents/Home
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "12.4", arch: "x86_64", family: "mac"

I understand that byte code compliance and general max Java version was pinned at 1.8 however I started using 11 several years ago. I hope that this PR can act as the basis for possibly updating the owlapi project to at atleast JDK11...?

Thanks

@ignazio1977
Copy link
Contributor

Version 6 is targeting Java 17. I'm not sure about changing the required Java version for version 5, I'll try to find out what people are running it on.

@lewismc
Copy link
Contributor Author

lewismc commented Jun 9, 2022

Hi @ignazio1977 thanks for the response. I was unaware that version6 was under development... nice work 👍 Also, yes I agree with targeting JDK17 for that release.
Are you planning on releasing the version5 development branch much more or is most development going into version6? I can probably pretty easily adapt this PR for version6 if you intend to make a release.
Thanks for your consideration.

@ignazio1977
Copy link
Contributor

Bumped to 4.2.0 on version 5, next release to be 5.5.0

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.

2 participants