Skip to content

Implement JVM_LoadZipLibrary() - #18055

Merged
tajila merged 2 commits into
eclipse-openj9:masterfrom
keithc-ca:zip
Sep 1, 2023
Merged

Implement JVM_LoadZipLibrary()#18055
tajila merged 2 commits into
eclipse-openj9:masterfrom
keithc-ca:zip

Conversation

@keithc-ca

Copy link
Copy Markdown
Contributor

The stub had the wrong signature: this corrects it and provides an implementation.

Fixes: #17996.

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
wip

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
Implement JVM_LoadZipLibrary()

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
@keithc-ca
keithc-ca requested a review from tajila August 30, 2023 20:55
@tajila

tajila commented Sep 1, 2023

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux jdk21

@tajila

tajila commented Sep 1, 2023

Copy link
Copy Markdown
Contributor

jenkins test sanity win jdk8

@keithc-ca

Copy link
Copy Markdown
Contributor Author

I think you meant "alinux64" instead of "alinux". I'll let you decide whether the Windows build is sufficient or you want that build as well.

@keithc-ca

Copy link
Copy Markdown
Contributor Author

If #18069 is accepted, "alinux" will be equivalent to "alinux64".

@tajila

tajila commented Sep 1, 2023

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux jdk21

@keithc-ca

Copy link
Copy Markdown
Contributor Author

#18069 is not yet merged, so if you want that aarch64 Linux build, you'll have to say "alinux64" for the time being.

@keithc-ca

Copy link
Copy Markdown
Contributor Author

Jenkins test sanity alinux64 jdk21

@tajila

tajila commented Sep 1, 2023

Copy link
Copy Markdown
Contributor

Known failure

23:16:03.320 0x19200j9vmutil(j9vm).15     *   ** ASSERTION FAILED ** at /home/jenkins/workspace/Build_JDK21_aarch64_linux_Personal/openj9/runtime/util/mthutil.c:600: ((((uintptr_t)-1) != methodIndex))
JVMDUMP039I Processing dump event "traceassert", detail "" at 2023/09/01 19:16:03 - please wait.

@tajila
tajila merged commit e57170e into eclipse-openj9:master Sep 1, 2023
@keithc-ca
keithc-ca deleted the zip branch September 5, 2023 12:30
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.

java.exe Crashes when JPackage Bundles it

2 participants