ci: switch to cvcpkg-install for per-component dep downloads#81
Open
transfix wants to merge 11 commits into
Open
ci: switch to cvcpkg-install for per-component dep downloads#81transfix wants to merge 11 commits into
transfix wants to merge 11 commits into
Conversation
…loads Replace the monolithic archive download in the fetch-libcvc-deps composite action with the new cvcpkg-install action from transfix/[email protected]. This downloads only the components listed in cvc-requirements.yaml instead of a single ~2 GB archive. The output interface (steps.<id>.outputs.path) is unchanged, so all 30+ references across ci.yml and release.yml work without modification. - Add cvc-requirements.yaml listing all 22 components needed by libcvc and volrover3 - Update fetch-libcvc-deps action to delegate to cvcpkg-install - Default version bumped from 1.0.2 to 1.2.0 See transfix/libcvc-deps#34 and the split-distribution roadmap.
# Conflicts: # cvc-requirements.yaml
- Reference cvcpkg-install@master instead of non-existent @v1.2.0 tag - Remove version input (no longer downloading from GitHub releases) - Add qt6 and vtk to cvc-requirements.yaml (needed for volrover3)
…rd-party provider Windows package jobs in ci.yml and release.yml no longer install boost/hdf5/fftw/gsl/cgal/imagemagick/log4cplus via vcpkg, no longer route buildtrees to D:, no longer pass -DCMAKE_TOOLCHAIN_FILE=vcpkg.cmake or -DVCPKG_TARGET_TRIPLET. cvcpkg (via .github/actions/fetch-libcvc-deps) is the single source of truth on Linux, macOS, and Windows for Qt6, VTK, Boost, HDF5, FFTW, GSL, ImageMagick, log4cplus, and CGAL. Deletes vcpkg-overlay/ (imagemagick port + custom triplets) now that vcpkg is unused. Also adds publish-cvcpkg.yml + cvcpkg/recipes/libcvc/recipe.yaml (stubbed on 'cvcpkg pack-from-prefix' pending that feature).
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Switches the
fetch-libcvc-depscomposite action from downloading a monolithic ~2 GB archive to usingcvcpkg-installfromtransfix/[email protected], which downloads only the individual component bundles listed incvc-requirements.yaml.Changes
.github/actions/fetch-libcvc-deps/action.yml: Replaced the curl+tar/unzip/7z logic with a single delegation totransfix/libcvc-deps/.github/actions/[email protected]. The output interface (path) is identical, so all 30+ workflow references inci.ymlandrelease.ymlwork without modification.cvc-requirements.yaml(new): Declares the 22 components libcvc and volrover3 need from libcvc-deps.How it works
fetch-libcvc-depsaction delegates tocvcpkg-installcvcpkg, resolvescvc-requirements.yamlagainst the published catalog, and downloads only the listed componentsCMAKE_PREFIX_PATHusage is unchangedBackwards compatibility
The action still falls back gracefully (empty
pathoutput) if the catalog or bundles are unavailable, preserving the existing behavior where callers gate from-source builds onsteps.<id>.outputs.path != ''.Related