Skip to content

Do not set LIBRARY_PATH on Windows - #53

Merged
carbolymer merged 1 commit into
IntersectMBO:mainfrom
jasagredo:js/library-path
Oct 27, 2026
Merged

carbolymer merged 1 commit into
IntersectMBO:mainfrom
jasagredo:js/library-path

Conversation

@jasagredo

Copy link
Copy Markdown
Contributor

This causes GHC to find the libc++ that was installed by the protobuf package and results in TH segfaults. Snappy can be linked with the cabal flag +pkg-config and for protobuf we only need the protoc compiler which is accessible via $PATH.

@jasagredo
jasagredo requested a review from Jimbo4350 as a code owner October 26, 2026 10:49
@jasagredo
jasagredo force-pushed the js/library-path branch 2 times, most recently from 9115770 to 568cdab Compare October 26, 2026 10:53
run: |
/usr/bin/pacman --noconfirm -S mingw-w64-clang-x86_64-snappy mingw-w64-clang-x86_64-protobuf
cat <<EOF >> $GITHUB_ENV
LIBRARY_PATH=C:/msys64/clang64/lib

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can this fix be extended to macos as well?

@carbolymer

Copy link
Copy Markdown
Contributor

LGTM

This causes GHC to find the libc++ that was installed by the protobuf package
and results in TH segfaults. Snappy can be linked with the cabal flag `+pkg-config`
and for protobuf we only need the protoc compiler which is accessible via `$PATH`.
@carbolymer
carbolymer merged commit 78bc06d into IntersectMBO:main Oct 27, 2026
8 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