-
Notifications
You must be signed in to change notification settings - Fork 40
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
3 changed files
with
7 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
de13268
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.
@szegedi
Unfortunately, this release appears to be broken when loaded on the Velocity server software: https://papermc.io/downloads/velocity
Would you prefer we continue at bugs.openjdk.com or can a hotfix be made, please? We've been waiting for the previous log4j debug log fix for months in hopes this release will boot. Thanks.
de13268
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 can reproduce this easily, will definitely fix it today.
de13268
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.
@kangarko can you perhaps try to substitute the Nashorn's core JAR with this one I just built from an intermediate fix: https://www.dropbox.com/scl/fi/h93urg1zo6j5a9fm2dgtl/nashorn-core-15.5.jar?rlkey=vjs7p8yi3lnarivad7pj7irmv&dl=0
I'd like to validate it first. (If you're skeptical about getting a binary from a Dropbox link, you can also build from source at https://github.com/szegedi/nashorn/tree/fix-logging-2).
de13268
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.
@szegedi Works now!! You're a champion thank you!