Skip to content
This repository has been archived by the owner on Mar 15, 2023. It is now read-only.

fix bzip2 #65

Merged
merged 1 commit into from
Dec 7, 2022
Merged

fix bzip2 #65

merged 1 commit into from
Dec 7, 2022

Conversation

jhheider
Copy link
Contributor

@jhheider jhheider commented Dec 7, 2022

This is confusing. What were we doing with the third-party makefile (implemented by me) and when did it stop working? I'm mystified.

@what-the-diff
Copy link

what-the-diff bot commented Dec 7, 2022

  • The build script was changed to make the symlinks only if they don't already exist.
  • This is a bug fix for an issue where the package would fail to install on systems that had previously installed bzip2, because it tried to create duplicate links and failed with "File exists".

@jhheider
Copy link
Contributor Author

jhheider commented Dec 7, 2022

Added testing to the build-script to ensure we get out the dylibs we thought we were getting. I think this is as safe as I can make it.

@jhheider jhheider merged commit 9705e8e into main Dec 7, 2022
@jhheider jhheider deleted the fix-bzip2 branch December 7, 2022 22:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant