-
Notifications
You must be signed in to change notification settings - Fork 915
Don't load the windows clipboard agent on JDK 25 and later #8608
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
Conversation
matthiasblaesing
left a comment
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.
I don't see the need for this rush, but looks fine (eyeballed only).
|
NB 27 will be able to run on JDK 25, so its just in time so to speak. |
I ran NB 26 on JDK25 with JDK-8353950 in with and without agent. The copy-cycle test with notepad worked fine. |
|
I did run it on JDK 26 too, I even changed JDK's class version value to cause an ASM failure to check if it cleanly exits. JDK is a moving target which receives updates while NB releases are frozen in time. I don't like to wait for things to break when it is easy to turn them off for JDK versions on which they aren't needed anyway. |
|
also while testing I noticed that the JDK warning window can't pop up |
|
fixed that too since it was easy to do |
- JDK 25+ contains JDK-8353950 already - no patching needed - avoids trouble with ASM or other unexpected interactions downgraded classfile version of the agent to 8, so that the JDK warning msg popup can show.
matthiasblaesing
left a comment
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.
Still looks sane.
downgraded classfile version of the agent to 8, so that the JDK warning msg popup can show (#8608 (comment)).