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

[libarchive] Restore pkgconfig file #19428

Merged

Conversation

c72578
Copy link
Contributor

@c72578 c72578 commented Aug 7, 2021

Describe the pull request

  • What does your PR fix?

    So far, generating libarchive.pc has been disabled by the
    fix-buildsystem.patch, whereas INCLUDE(CreatePkgConfigFile) was removed
    from CMakeLists.txt

    • Update fix-buildsystem.patch and allow creating the pkgconfig file
    • Add vcpkg_fixup_pkgconfig() to portfile
    • Add FEATURES keyword to vcpkg_check_features() to avoid deprecation
      warning
  • Which triplets are supported/not supported? Have you updated the CI baseline?

    all, No

  • Does your PR follow the maintainer guide?

    Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@c72578 c72578 marked this pull request as ready for review August 7, 2021 07:49
@JackBoosY JackBoosY added category:port-bug The issue is with a library, which is something the port should already support category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist and removed category:port-bug The issue is with a library, which is something the port should already support labels Aug 9, 2021
So far, generating libarchive.pc has been disabled by the
fix-buildsystem.patch, whereas INCLUDE(CreatePkgConfigFile) has
been removed from CMakeLists.txt
- Update fix-buildsystem.patch and allow creating the pkgconfig file
- Add vcpkg_fixup_pkgconfig() to portfile
- Add `FEATURES` keyword to vcpkg_check_features() to avoid deprecation
  warning
@c72578 c72578 force-pushed the 2021-08-05_libarchive_restore_pkgconfig_file branch from 171547d to 23aa5e1 Compare August 9, 2021 04:21
@JackBoosY JackBoosY added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Aug 9, 2021
@dan-shaw dan-shaw merged commit 32f6448 into microsoft:master Aug 10, 2021
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 info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants