isapir commented on PR #731: URL: https://github.com/apache/tomcat/pull/731#issuecomment-2150273321
@dsoumis But now you are completely excluding `/java/org/apache/tomcat/util/net/openssl/panama` and `/java/org/apache/tomcat/util/openssl`? It might "work" on Java 17 but new technology would not be used on Java 22+ `ant` has many different targets and configurations. I disagree with the notion that the IDE needs to be exactly like the build script compiler because that is not possible. Also, I left the commented lines in `res/ide-support/idea/compiler.xml` as an easy to toggle setting by the user, so I am against removing them. I do not see how this patch improves anything. IDEs are very user centric. We provide the tools and documentation and users can and should make adjustments as needed for their own use case. -- 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
