Fix antrun for openwire-generate and allow (require) JDK8 for now#1418
Fix antrun for openwire-generate and allow (require) JDK8 for now#1418jbonofre wants to merge 1 commit intoapache:mainfrom
Conversation
|
@kenliao94 as discussed on the mailing list. |
Hey @jbonofre it might be due to my local environment but since I configured Java 17 to work on |
|
As you can see in my change, jdk8 is still required here. I will update to jdk17+ in a separate PR. |
|
As mentioned on the dev list and on the Jira Issue, there is a new project for future OpenWire versions that is being modernized. It's fine to fix this to require JDK 8, but I don't think it's necessary or make sense to bother with fixing the current version for JDK 17. I think we should probably just remove the old generator entirely from the 6.x build as it doesn't work on newer JDKs and we plan to migrate to the new project for v13 and future updates. |
|
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
0fa6d4d to
6c284df
Compare
|
I rebased. This is a simple change just to set JDK 8 for OpenWire generator (just in case someone would like to build it). |
| <configuration> | ||
| <rules> | ||
| <requireJavaVersion> | ||
| <version>[8,)</version> |
There was a problem hiding this comment.
This looks like it will allow java 8 or higher, is that what you want? Doesn't this break when running with newer JDKs which was the problem?
There was a problem hiding this comment.
That's a good point. We should probably be more "restrictive" here. Let me update.
No description provided.