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

[vcpkg] Track hashes of cmake script helpers in consuming portfiles #12254

Merged

Conversation

ras0219
Copy link
Contributor

@ras0219 ras0219 commented Jul 3, 2020

This PR adds a heuristic method for determining what script helpers a given portfile is using and includes the hashes of those helpers into the port's package abi.

@JackBoosY JackBoosY self-assigned this Jul 6, 2020
@JackBoosY JackBoosY added category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed info:internal This PR or Issue was filed by the vcpkg team. labels Jul 6, 2020
@BillyONeal
Copy link
Member

ITK is #12298

@ras0219-msft
Copy link
Contributor

The arm64 errors are all due to a failure while downloading winflexbison from sourceforge.

@JackBoosY
Copy link
Contributor

@ras0219-msft I think vcpkg_find_acquire_program also need vcpkg_from_sourceforge.

@ras0219-msft
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY
Copy link
Contributor

rpclib:

CMake Error: Could not open file for write in copy operation D:/buildtrees/rpclib/src/v2.2.1-2bd00f7cbe.clean/include/rpc/version.h.tmp
CMake Error: : System Error: Permission denied

qt5-imageformats:

D:/installed/x86-windows/tools/qt5/debug/mkspecs/features/qt_plugin.prf:59: Cannot write file D:\buildtrees\qt5-imageformats\x86-windows-dbg\mkspecs\modules-inst\qt_plugin_qwbmp.pri: Cannot create parent directory

@Neumann-A
Copy link
Contributor

qt5-imageformats: Cannot create parent directory

If you see that one just retry. I sometimes also see it locally but just retrying seems to be enough to make it build. Probably a parallel build issue?

@PhoebeHui
Copy link
Contributor

PhoebeHui commented Jul 13, 2020

The osx failures:

libb2: I didn't repro this issue locally, it built sucessfully.

Makefile.am:7: error: 'pkgconfig_DATA' is used but 'pkgconfigdir' is undefined
src/Makefile.am: installing './depcomp'
parallel-tests: installing './test-driver'
autoreconf: automake failed with exit status: 1

spaceland: source download failed, rerun the PR might work.
ITk: confict with gdcm should be fixed by PR #11208

@PhoebeHui
Copy link
Contributor

PhoebeHui commented Jul 13, 2020

Fix libb2 issue via PR #12398

@@ -16,6 +16,7 @@ VCPKG_TARGET_IMPORT_LIBRARY_PREFIX import library prefix for target (same
VCPKG_TARGET_IMPORT_LIBRARY_SUFFIX import library suffix for target (same as CMAKE_IMPORT_LIBRARY_SUFFIX)
VCPKG_FIND_LIBRARY_PREFIXES target dependent prefixes used for find_library calls in portfiles
VCPKG_FIND_LIBRARY_SUFFIXES target dependent suffixes used for find_library calls in portfiles
VCPKG_SYSTEM_LIBRARIES list of libraries are provide by the toolchain and are not managed by vcpkg
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
VCPKG_SYSTEM_LIBRARIES list of libraries are provide by the toolchain and are not managed by vcpkg
VCPKG_SYSTEM_LIBRARIES list of libraries that are provided by the toolchain and are not managed by vcpkg

toolsrc/src/vcpkg/vcpkgpaths.cpp Show resolved Hide resolved
toolsrc/src/vcpkg/vcpkgpaths.cpp Show resolved Hide resolved
@ras0219-msft ras0219-msft merged commit 385fb52 into microsoft:master Jul 13, 2020
@ras0219-msft
Copy link
Contributor

Thanks everyone!

@emptyVoid emptyVoid mentioned this pull request Aug 10, 2020
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
…icrosoft#12254)

* [vcpkg] Track hashes of cmake script helpers in consuming portfiles

* [vcpkg] Format

* [vcpkg_find_acquire_program] Use �cpkg_from_sourceforge for FLEX and BISON

Co-authored-by: Robert Schumacher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants