markt-asf commented on PR #731: URL: https://github.com/apache/tomcat/pull/731#issuecomment-2160396222
> Since IDE support is [configured](https://github.com/apache/tomcat/blob/main/build.xml#L3830) with build.java.version, which currently is [17](https://github.com/apache/tomcat/blob/main/build.xml#L111), then should be aligned with build.xml. Fair point. I think that should probably be release.java.version and was missed when we split build version and release version but that probably needs to be a separate discussion. > (It's noteworthy that I haven't managed to make this work in Intellij even when using Java22 with preview features enabled, which might be a bug in IDEA and could affect other users as well. From my point of view, it's better to have something that works flawlessly out of the box than trying to figure out what is going wrong, which led to the existence of this PR :) ) Personally, I'd always rather fix the root cause than the symptom. Since I'm not an Idea user I don't think my views should hold much weight. That said, what I'd suggest is: - do nothing for now - folks can always tweak their own configuration if they want/need to - figure out the root cause of why this isn't working for you - discuss the next steps once we know what the root cause is -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
