Skip to content

Commit

Permalink
fix: MSVC build broke because packages aged out of mirrors
Browse files Browse the repository at this point in the history
  Remove the package list pin on vcpkg - no other platform
  is pinning package versions, and this removes the need
  for people to remember to update the commit id every
  year.

Signed-off-by: David Li <[email protected]>
  • Loading branch information
randombk committed Jan 9, 2025
1 parent 8c88415 commit bcb58ba
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 9 deletions.
5 changes: 0 additions & 5 deletions .github/workflows/manual-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -202,11 +202,6 @@ jobs:
appendedCacheKey: ${{ hashFiles( 'msvc-full-features/vcpkg.json', '.github/vcpkg_triplets/**' ) }}-${{ matrix.arch }}-1
setupOnly: true
vcpkgDirectory: "${{ runner.workspace }}/b/vcpkg"
# We have to use at least this version of vcpkg to include fixes for
# various issues we've encountered over time. Keep it in sync with the builtin-baseline
# field in vcpkg.json. Caching happens as a post-action which runs at the end of
# the whole workflow, after vcpkg install happens during msbuild run.
vcpkgGitCommitId: "66444e13a86da7087ee24c342f91801cc6eb9877"
- name: Install dependencies (windows msvc) (3/3)
if: runner.os == 'Windows'
run: |
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/msvc-full-features-cmake.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,11 +75,10 @@ jobs:
uses: lukka/get-cmake@latest

- name: Install vcpkg
uses: lukka/run-vcpkg@v11
uses: lukka/run-vcpkg@main
id: runvcpkg
with:
vcpkgDirectory: '${{ runner.workspace }}/b/vcpkg'
vcpkgGitCommitId: "66444e13a86da7087ee24c342f91801cc6eb9877"

- name: Integrate vcpkg
run: |
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,6 @@ jobs:
id: runvcpkg
with:
vcpkgDirectory: '${{ runner.workspace }}/b/vcpkg'
vcpkgGitCommitId: "66444e13a86da7087ee24c342f91801cc6eb9877"

- name: Integrate vcpkg
if: runner.os == 'Windows'
Expand Down
1 change: 0 additions & 1 deletion msvc-full-features/vcpkg.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@
},
"sdl2-ttf"
],
"builtin-baseline": "c9aba300923c8ec0ab190e2bff23085209925c97",
"vcpkg-configuration": {
"overlay-ports": [
"../.github/vcpkg_ports"
Expand Down

0 comments on commit bcb58ba

Please sign in to comment.