Skip to content

Fix native library path separator on Windows - #383

Closed
gharrma wants to merge 1 commit into
bazelbuild:masterfrom
gharrma:fix-java-native-lib-handling-on-windows
Closed

gharrma wants to merge 1 commit into
bazelbuild:masterfrom
gharrma:fix-java-native-lib-handling-on-windows

Conversation

@gharrma

@gharrma gharrma commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

On Windows, java.library.path is split by ";" instead of ":".

Regression test added.

Note for Googlers: this fix is in service of b/149162146#comment19

@gharrma
gharrma requested review from a team and hvadehra as code owners September 23, 2026 04:28
On Windows, java.library.path is split by ";" instead of ":".
@gharrma
gharrma force-pushed the fix-java-native-lib-handling-on-windows branch from 68ad9cf to b0e4c3e Compare September 23, 2026 05:45
@gharrma

gharrma commented Sep 23, 2026

Copy link
Copy Markdown
Contributor Author

I had to disable the added test on Bazel 7.x since that version of Bazel still uses the built-in Java rules. After that, CI passes and the change is now ready for review.

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.

1 participant