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

JDK17+ JVM_LoadLibrary() opens shared library via J9PORT_SLOPEN_DECORATE #19861

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

JasonFengJ9
Copy link
Member

@JasonFengJ9 JasonFengJ9 commented Jul 16, 2024

JDK17+ JVM_LoadLibrary() opens shared library via J9PORT_SLOPEN_DECORATE

j9sl_open_shared_library() with the flag J9PORT_SLOPEN_DECORATE expects the incoming library name to be platform independent, i.e., it must not contain any prefix or file extension. A path to the library is supported.

Required by

closes #19344

Signed-off-by: Jason Feng fengj@ca.ibm.com

runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
@JasonFengJ9 JasonFengJ9 force-pushed the aixlib branch 2 times, most recently from b0c2289 to ce0a760 Compare July 23, 2024 17:17
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
runtime/j9vm/j9scar.tdf Outdated Show resolved Hide resolved
runtime/j9vm/jvm.c Outdated Show resolved Hide resolved
j9sl_open_shared_library() with the flag J9PORT_SLOPEN_DECORATE expects
the incoming library name to be platform independent, i.e., it must not
contain any prefix or file extension. A path to the library is
supported.

Signed-off-by: Jason Feng <fengj@ca.ibm.com>
@keithc-ca
Copy link
Contributor

Jenkins test sanity aix,amac,xlinux,win jdk11,jdk17

@keithc-ca
Copy link
Contributor

I keep forgetting we have no Windows build systems (online). I'll rely on the test results for the other platforms.

@keithc-ca keithc-ca merged commit 9142f7e into eclipse-openj9:master Jul 25, 2024
14 of 17 checks passed
@JasonFengJ9 JasonFengJ9 deleted the aixlib branch July 25, 2024 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JVM_LoadLibrary should handle .a files on AIX
2 participants