-
Notifications
You must be signed in to change notification settings - Fork 15
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
[24.1] Backport: Updated JFR substitutions from [GR-54877] #764
Conversation
…p.java failed again with "Expected outer stream to have 3 events"' (cherry picked from commit fbc44e9)
Failing GHAs:
|
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.
LGTM other than the references to JDK 24 source code. @roberttoyonaga could you please adapt them accordingly? Thank you
...tevm/src/com.oracle.svm.core.posix/src/com/oracle/svm/core/posix/PosixPlatformTimeUtils.java
Outdated
Show resolved
Hide resolved
substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/util/PlatformTimeUtils.java
Outdated
Show resolved
Hide resolved
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.
LGTM other than the references to JDK 24 source code. @roberttoyonaga could you please adapt them accordingly? Thank you
yes, thanks for spotting that. Done! |
…p.java failed again with "Expected outer stream to have 3 events"'
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.
Thank you @roberttoyonaga !
This backport was replaced by upstream oracle@27149a9, but we kept e308905 on top of it. |
(cherry picked from commit fbc44e9)
Related issue: #763
A change to OpenJDK JFR was introduced in JDK 24 and later backported to JDK23+29. The Mandrel GHAs are using JDK23+29 so builds that include JFR will fail due to out of date substitutions.