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 build to set the release to Java 8 #3648
Conversation
Signed-off-by: Tyler Van Gorder <tkvangorder@users.noreply.github.com>
I was able to build locally with this change, install it into my maven local and verify things were working when using 4.10.1-SNAPSHOT and running on Java 8. |
@parrt if you haven't released yet, you probably want to include this PR in 4.10 |
Probably it makes sense of publishig RC (release candidate) before real release. |
This seems fine to me. Deferring to @parrt though. |
Damn. what if I release with mvn as 4.10.0-1 ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@KvanTTT @ericvergnaud should i do 4.10-1 or 4.10.1? i suggest 4.10-1 (or 4.10.0-1) with mvn then update jars. don't change version in software. |
4.10.1
… Le 14 avr. 2022 à 22:10, Terence Parr ***@***.***> a écrit :
@KvanTTT <https://github.com/KvanTTT> @ericvergnaud <https://github.com/ericvergnaud> should i do 4.10-1 or 4.10.1?
—
Reply to this email directly, view it on GitHub <#3648 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZNQJE2A7OLU4B4Z7RF6BLVFB3S3ANCNFSM5TE2FLHA>.
You are receiving this because you were mentioned.
|
oh man. now i gotta go thru release process again. haha. |
trust me, I feel your pain. 8) |
* [C++] Remove reference to antlrcpp:s2ws (#3656) Signed-off-by: Justin King <jcking@google.com> * Fix build to set the release to Java 8 (#3648) Signed-off-by: Tyler Van Gorder <tkvangorder@users.noreply.github.com> Co-authored-by: Tyler Van Gorder <tkvangorder@users.noreply.github.com> * prep for 4.10.1 Signed-off-by: Terence Parr <parrt@antlr.org> * fix v numbers Signed-off-by: Terence Parr <parrt@antlr.org> Co-authored-by: Justin King <jcking@google.com> Co-authored-by: Tyler Van Gorder <1878529+tkvangorder@users.noreply.github.com> Co-authored-by: Tyler Van Gorder <tkvangorder@users.noreply.github.com>
The 4.10.0 runtime of antlr correctly sets the
source
andtarget
to 1.8, however, the compiler is still compiling against the Java 11 runtime classpath. This results in the runtime using a different method signature forjava.nio.CharBuffer.flip()
This manifests as the following error when using antlr in a java8-based project:
This PR sets the release (used by the maven compiler plugin) to 8, thus forcing it to use the java 8 runtime when compiling.