Skip to content

feat(bindings/java): Add musl platform support#7087

Merged
Xuanwo merged 13 commits into
mainfrom
xuanwo/java-alpine-v1
May 14, 2026
Merged

feat(bindings/java): Add musl platform support#7087
Xuanwo merged 13 commits into
mainfrom
xuanwo/java-alpine-v1

Conversation

@Xuanwo

@Xuanwo Xuanwo commented Dec 22, 2025

Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #7086

Rationale for this change

This PR will add musl platform support for binding java.

What changes are included in this PR?

Are there any user-facing changes?

AI Usage Statement

Parts of this PR were drafted with assistance from Codex (with gpt-5.2) and fully reviewed and edited by me. I take full responsibility for all changes.

@Xuanwo Xuanwo requested a review from tisonkun as a code owner December 22, 2025 13:06
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Dec 22, 2025
Comment thread bindings/java/src/main/java/org/apache/opendal/NativeLibrary.java Outdated
Comment thread bindings/java/src/main/java/org/apache/opendal/NativeLibrary.java
@Xuanwo Xuanwo requested a review from tisonkun December 22, 2025 16:03
Comment thread bindings/java/tools/build.py Outdated
if (loader == null) {
return false;
}
return Files.exists(Paths.get("/lib", loader)) || Files.exists(Paths.get("/usr/lib", loader));

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you test this line works?

I suspect that with bundled jar the file doesn't exist on /lib but in the JAR package.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 12, 2026
@Xuanwo Xuanwo merged commit 89f943d into main May 14, 2026
94 checks passed
@Xuanwo Xuanwo deleted the xuanwo/java-alpine-v1 branch May 14, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: opendal-java is not available on musl

3 participants