-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fixed pkgconfig files (mostly on Windows) #37
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge-admin, please rerender |
Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like there was nothing to do. This message was generated by GitHub actions workflow run https://github.com/conda-forge/libzip-feedstock/actions/runs/6060841097. |
Thanks! Just for curiosity, which downstream project finds libzlib with pkg-config ? Some gz-* library? |
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
Note that in theory as we are building shared library the Libs.private could be ignored, unfortunately this is not the case, see conda-forge/conda-forge.github.io#1880 for a related discussion. |
It got pulled by gz-fuel_tools9, which in turn just uses the macros given by gz-cmake. And these macros prefer pkgconfig over cmake modules. |
I agree. It seems to me Libs.private are a much different case than Requires.private. I actually wondered why are all these added to the output of |
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)This PR imports the fix for issues nih-at/libzip#399 and nih-at/libzip#404 . It is essential for this library working on Windows. This patch should be removed when new upstream release is done, as it will be included.
The current bad pkgconfig file on Windows looks like this:
Notice
Libs.private
contains garbage, making it impossible to build anything against libzip.