Skip to content

ci(vcpkg): create lib/vcpkg.sh used by several builds#6701

Merged
devjgm merged 1 commit intogoogleapis:mainfrom
devjgm:oldest-deps-not-clone
Jun 5, 2021
Merged

ci(vcpkg): create lib/vcpkg.sh used by several builds#6701
devjgm merged 1 commit intogoogleapis:mainfrom
devjgm:oldest-deps-not-clone

Conversation

@devjgm
Copy link
Copy Markdown
Contributor

@devjgm devjgm commented Jun 5, 2021

Fixes: #6699

This PR factors the code to fetch the vcpkg repo into a library that's
reused in a couple places. It also clones the repo at a recent release
tag rather than cloning at HEAD which may break our builds and is not
hermetic, or downloading a tarball, which lacks git history.


This change is Reviewable

Fixes: googleapis#6699

This PR factors the code to fetch the vcpkg repo into a library that's
reused in a couple places. It also clones the repo at a recent release
tag rather than cloning at HEAD which may break our builds and is not
hermetic, or downloading a tarball, which lacks git history.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 5, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 5, 2021
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 8f0c9891176ddb9ad04b7e18d94f006432953d7c

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 5, 2021

Codecov Report

Merging #6701 (8f0c989) into main (4032b07) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #6701   +/-   ##
=======================================
  Coverage   95.69%   95.69%           
=======================================
  Files        1215     1215           
  Lines      112632   112632           
=======================================
  Hits       107784   107784           
  Misses       4848     4848           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4032b07...8f0c989. Read the comment docs.

@devjgm devjgm marked this pull request as ready for review June 5, 2021 13:58
@devjgm devjgm requested a review from a team June 5, 2021 13:58
@devjgm devjgm merged commit 9f0989a into googleapis:main Jun 5, 2021
@devjgm devjgm deleted the oldest-deps-not-clone branch June 5, 2021 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cmake-oldest-deps build is broken

4 participants