Skip to content

Conversation

hasufell
Copy link
Member

No description provided.

@hasufell hasufell requested a review from Bodigrim March 10, 2024 10:33
Comment on lines +27 to +31
flag system-bzlib
description:
Use system bzlib rather than the bundled sources
default: True
manual: True
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the right default, imo.

Using bundled C sources should be the last resort. Even under windows you can build against bzip2 just fine: https://packages.msys2.org/base/mingw-w64-bzip2

Unfortunately, upstream does not provide a pkg-config file, so we can't use the auto fallback mechanism.

Copy link
Contributor

Choose a reason for hiding this comment

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

I understand your position, but my experience with other packages is that practically it would force Windows users to stay on the older, vulnerable version of bzlib.

Copy link
Member Author

Choose a reason for hiding this comment

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

Well, feel free to cherry-pick the first commit then, which keeps the old behavior.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks, I'll merge and revert the second commit then.

@Bodigrim Bodigrim merged commit 1debc6f into haskell:master Mar 10, 2024
@Bodigrim
Copy link
Contributor

Thanks!

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