Skip to content
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

[JDK24] Find and consume --illegal-native-access=<VALUE> #20755

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

babsingh
Copy link
Contributor

@babsingh babsingh commented Dec 4, 2024

After reading the last --illegal-native-access=<VALUE> option,
the system property jdk.module.illegal.native.access is assigned
the specified <VALUE>.

Related: #19680
Related: #20354

After reading the last --illegal-native-access=<VALUE> option,
the system property jdk.module.illegal.native.access is assigned
the specified <VALUE>.

Related: eclipse-openj9#19680
Related: eclipse-openj9#20354

Signed-off-by: Babneet Singh <[email protected]>
@babsingh babsingh requested a review from tajila December 4, 2024 19:56
@babsingh
Copy link
Contributor Author

babsingh commented Dec 4, 2024

jenkins test sanity.functional amac jdknext

@babsingh
Copy link
Contributor Author

babsingh commented Dec 5, 2024

The failures seem unrelated to the proposed changes. @JasonFengJ9 Have you seen the sanity.functional failures in recent builds?

@JasonFengJ9
Copy link
Member

17:25:41 NonExistJarTests.CheckForError1
17:25:41 SCURLClassLoaderNPTests_0_FAILED

17:30:47 ---SUMMARY OF FAILED TESTS---
17:30:47 simple caching of a resource, with security on
17:30:47 simple caching of a resource, with security on but readonly policy
17:30:47 simple caching of a resource, with security on but writeonly policy
17:30:47 caching of multiple resources, security on
17:30:47 -----------------------------------
17:30:47 cmdLineTester_DataHelperTests_0_FAILED

These failures and a few others seem related to JDK24: Permanently Disable the Security Manager, and not caused by this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants