Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: fix libdeflate find module never finding anything #4968

Merged
merged 1 commit into from
Feb 20, 2023

Conversation

mikedld
Copy link
Member

@mikedld mikedld commented Feb 20, 2023

DEFLATE_VERSION was never set but was listed as required. Move it from REQUIRED_VARS to VERSION_VAR and populate properly.

Notes: Fixed build issue that prevented preinstalled copies of libdeflate from being used

@mikedld mikedld added this to the 4.0.1 milestone Feb 20, 2023
@mikedld mikedld added scope:3rdparty type:build Changes that affect the build system labels Feb 20, 2023
@mikedld mikedld mentioned this pull request Feb 20, 2023
`DEFLATE_VERSION` was never set but was listed as required. Move it from
`REQUIRED_VARS` to `VERSION_VAR` and populate properly.
@ckerr ckerr changed the title Fix libdeflate find module never finding anything build: fix libdeflate find module never finding anything Feb 20, 2023
@mikedld
Copy link
Member Author

mikedld commented Feb 20, 2023

FreeBSD build on TeamCity finally found it:

  -- Found DEFLATE: /usr/local/include (found suitable version "1.10", minimum required is "1.10")

@mikedld mikedld merged commit 88556bd into main Feb 20, 2023
@mikedld mikedld deleted the bugfix/deflate-find-module branch February 20, 2023 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope:3rdparty type:build Changes that affect the build system
Development

Successfully merging this pull request may close these issues.

1 participant