Skip to content

Set Java version to 1.8 to match Maven build.#93

Merged
leonardehrenfried merged 1 commit intoopenstreetmap:masterfrom
sebastic:java-version
Mar 11, 2026
Merged

Set Java version to 1.8 to match Maven build.#93
leonardehrenfried merged 1 commit intoopenstreetmap:masterfrom
sebastic:java-version

Conversation

@sebastic
Copy link
Contributor

Not setting the version results in much too high bytecode version to be used which complicates compatibility with reverse dependencies like osmosis.

@leonardehrenfried
Copy link
Collaborator

In 15 years of writing Java I have never used ant so I have no idea if this is correct, but it looks fine to me.

Question: do you need a deployment to Maven Central for this? Presumably not, because ant doesn't use Maven Central, correct?

@leonardehrenfried leonardehrenfried merged commit 28f76c6 into openstreetmap:master Mar 11, 2026
10 checks passed
@sebastic
Copy link
Contributor Author

In 15 years of writing Java I have never used ant so I have no idea if this is correct, but it looks fine to me.

It's correct per the documentation: https://ant.apache.org/manual/Tasks/javac.html

And also fixes the issue with the Debian package that blocked testing migration:

https://qa.debian.org/excuses.php?package=osmpbf

Question: do you need a deployment to Maven Central for this?

No, the osmpbf Debian package uses the ant build because it's so much simpler than Maven.

Presumably not, because ant doesn't use Maven Central, correct?

Indeed.

@sebastic sebastic deleted the java-version branch March 11, 2026 09:41
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