Skip to content

claude: add Windows support (x86-64/arm64): bundle dlls, search PATH,… - #15

Merged
DRMacIver merged 2 commits into
hegeldev:mainfrom
esauro:claude/hegel-java-windows-support-ad57c6
Aug 13, 2026
Merged

claude: add Windows support (x86-64/arm64): bundle dlls, search PATH,…#15
DRMacIver merged 2 commits into
hegeldev:mainfrom
esauro:claude/hegel-java-windows-support-ad57c6

Conversation

@esauro

@esauro esauro commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

… cache in %LOCALAPPDATA%; add windows CI

Co-Authored-By: Claude Fable 5 noreply@anthropic.com

… cache in %LOCALAPPDATA%; add windows CI

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
if (os.equals("windows")) {
return "PATH";
}
return os.equals("darwin") ? "DYLD_LIBRARY_PATH" : "LD_LIBRARY_PATH";

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.

Not that this really matters, but using the first os.equals follow by a ternary operator is pretty weird, especially given that Java supports string switch statements which would be cleaner all round.

@DRMacIver
DRMacIver enabled auto-merge August 13, 2026 12:40
@DRMacIver
DRMacIver merged commit 876acef into hegeldev:main Aug 13, 2026
7 checks passed
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