[vcpkg/scripts/pkgconfig] rewrite pkg-config check for libraries #11550
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Maybe we should choose one port to test this PR? |
JackBoosY
suggested changes
May 26, 2020
JackBoosY
changed the title
This reverts commit 26269e9.
|
@JackBoosY: osx has no pkg-config installed? |
|
Yes, please install |
JackBoosY
added
the
depends:vm-update
|
@JackBoosY: Done |
4 tasks
JackBoosY
approved these changes
May 29, 2020
JackBoosY
added
info:reviewed
|
Looks good in my side. |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Closed
|
currently checking why there is no regression in zstd. according to #12407 there should be a regression |
- add additional debug messages for debuging - fix unsetting of cache variables hiding issues with debug libraries - fixed pkgconfig fix in the debug case
…fig_to_fixup_pkgconfig # Conflicts: # ports/zstd/CONTROL # ports/zstd/portfile.cmake
…ed (fixes some regressions) add option NOT_STATIC_PKGCONFIG to run pkg-config without the --static argument
…milar regex to the library removal for ignored flags.
|
@ras0219-msft: Should have 561287c triggered a full rebuild after 5ccacd4? Please rerun failing CIs. This PR should be green now. |
|
@Neumann-A Easiest way to force that is to push a merge with master; in this case there are merge conflicts |
…_fixup_pkgconfig # Conflicts: # ports/zlib/CONTROL # ports/zlib/portfile.cmake
|
The ompl:x86-windows regressions seems unrelated. Worked locally and error in CI is: |
|
@ras0219-msft @ras0219 ping for merge this PR. |
|
LGTM, thanks @Neumann-A for the continued awesome work on this! I think that it will eventually make sense to move the "checking" parts of this into the vcpkg tool itself, to ensure it is always run on any |
hellozee
pushed a commit
to hellozee/vcpkg
that referenced
this pull request
Sep 11, 2020
…rosoft#11550) * [vcpkg/scripts/pkgconfig] rewrite pkgconfig check for libraries. * [vcpkg/scripts/pkgconfig] fix more details. * [vcpkg/scripts/pkgconfig] smaller improvements * ws change to check functionallity of script for testing. * [vcpkg/scripts/pkgconfig] update to latest test version * Revert "ws change to check functionallity of script for testing." This reverts commit 26269e9. * [nettle] bump control for vcpkg_fixup_pkgconfig test * [gmp] rerun CI to create new *.pc files * [gmp/nettle] enable port debug * [vcpkg/script/pkgconfig] fix path conversion regex for CI * [CI|OSX] install pkg-config on osx * [gmp/nettle] disable port debug * [vcpkg/scripts/pkgconfig] update to latest version of x windows pr * [zlib] add pkgconfig file * [zstd] add fixup pkgconfig * [libpng] add pc file installation * [bzip2] install pc file and leave a TODO for somebody else ;) * [bzip2] bump control * [vcpkg/scripts/pkgconfig] skip checks if pkgconfig cannot be found. - small bugfix in ignore flags regex * retry zstd and zlib * [libpng] retry by adding dl and m to system libraries for osx * [libpng] add missing vcpkg_fixup_pkgconfig() * [vcpkg/scripts/pkgconfig] unset var if found. fixes checks for other configurations which will be always succesful otherwise * [libpng] bump control * [libpng] enable port debug to debug osx regression * ws change to retrigger osx ci and hopeing that it works * fix typo * [libpng] remove -lm flag on apple platforms * ws change to rerun some ports in ci for good measure after the merge with master * Re-trigger CI test * [zstd] add pkgconfig * [zstd] correct debug link * [vcpkg/script/pkgconfig] - add more search suffixes for static libs - add additional debug messages for debuging - fix unsetting of cache variables hiding issues with debug libraries - fixed pkgconfig fix in the debug case * [pcre] add pthread dependency to pcre * [protobuf] fix pkgconfig * include quotes around the prefix in the case the prefix path was quoted (fixes some regressions) add option NOT_STATIC_PKGCONFIG to run pkg-config without the --static argument * [ignition-msgs5] get rid of a dev warning * [apr] fix apr regression * [protobuf] fix protobuf regression * [sdl1] fix regression due to variable expansion of pkg-config. Use similar regex to the library removal for ignored flags. * [ignition] fix regression * remove changes of zlib portfile. * reset changes in osx ci pipeline * add DISABLE_PARALLEL_CONFIGURE Co-authored-by: Jack·Boos·Yu <[email protected]> Co-authored-by: JackBoosY <[email protected]> Co-authored-by: Billy Robert O'Neal <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #11103