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

[zlib] Add pkgconfig file #12133

Merged

Conversation

c72578
Copy link
Contributor

@c72578 c72578 commented Jun 26, 2020

Describe the pull request

  • What does your PR fix? Fixes #
    Install the zlib.pc file, which is generated during the CMake build.
    Fix the pkgconfig file under Windows, where the lib is called zlib
    instead of libz and add the postfix 'd' in case of a Windows debug
    build.

  • Which triplets are supported/not supported? Have you updated the CI baseline?
    All triplets are supported

  • Does your PR follow the maintainer guide?
    Yes

Install the zlib.pc file, which is generated during the CMake build.
Fix the pkgconfig file under Windows, where the lib is called zlib
instead of libz and add the postfix 'd' in case of a Windows debug
build.
@c72578 c72578 marked this pull request as ready for review June 26, 2020 20:37
ports/zlib/portfile.cmake Outdated Show resolved Hide resolved
ports/zlib/portfile.cmake Show resolved Hide resolved
ports/zlib/portfile.cmake Outdated Show resolved Hide resolved
ports/zlib/portfile.cmake Show resolved Hide resolved
@NancyLi1013 NancyLi1013 added requires:author-response category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist labels Jun 28, 2020
c72578 and others added 2 commits June 28, 2020 09:59
- Remove include(vcpkg_common_functions)
- Comment out vcpkg_test_cmake(PACKAGE_NAME ZLIB MODULE)
@c72578 c72578 force-pushed the 2020-06-26_zlib_Add_pkgconfig_file branch from 040319e to 76015db Compare June 28, 2020 08:06
@c72578 c72578 requested a review from NancyLi1013 June 28, 2020 08:10
@c72578
Copy link
Contributor Author

c72578 commented Jul 1, 2020

@NancyLi1013, is the PR OK now from your side?

@NancyLi1013
Copy link
Contributor

@c72578
LGTM now. Thanks for your PR.
Since there is a regression on osx for itk that is not related with this PR. But it sill needs to resolve.
Once all CI checks have passed, this PR can be ready to merge.

@PhoebeHui
Copy link
Contributor

The itk failures will be fixed by #11208

ports/zlib/CONTROL Outdated Show resolved Hide resolved
@PhoebeHui PhoebeHui self-assigned this Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants