neilcsmith-net commented on PR #6054: URL: https://github.com/apache/netbeans/pull/6054#issuecomment-1585645934
> The problem with flags is that they become cheat codes for running NB on unsupported configs. That is literally what I'm suggesting though! :smile: Bear in mind I was originally in the "raise the default bytecode level across the codebase" camp! But what we agreed was that we'd raise bytecode level as and when necessary. I'm sure @matthiasblaesing made a comment about lower level things like `o.n.bootstrap` being later down the chain? I can't find it right now, but anyway ... We won't support the runtime container on JDK 8 from NB19, and on JDK 11 from NB22. This code should by default error out at those levels with those releases. But, that doesn't mean we will bump the bytecode level of `o.n.bootstrap` with those releases as far as I'm aware. While the bootstrap will technically run, then we should provide an escape hatch to turn the error into a warning. That should be logged, but otherwise allow the launch to proceed. Yes, that might fail later, but that's the point. That's my 2c on approach to this anyway - allows opt in for tests and any other nefarious purpose. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
