Skip to content

Conversation

@Rossmaxx
Copy link
Contributor

@Rossmaxx Rossmaxx commented Nov 19, 2024

Fixes a build fail on our MSVC CI due to outdated version of vcpkg used.

Also bumps the version of the msvc-dev-cmd github action.

@Rossmaxx Rossmaxx marked this pull request as ready for review November 19, 2024 08:28
@Rossmaxx Rossmaxx changed the title [testing] attempt at fixing the msvc ci fails recently. Fix the MSVC CI fail happening due to outdated vcpkg + slight bump of versions for CI Nov 19, 2024
@Rossmaxx Rossmaxx changed the title Fix the MSVC CI fail happening due to outdated vcpkg + slight bump of versions for CI Fix the MSVC CI fail happening due to outdated vcpkg + bump msvc-dev-ci version Nov 19, 2024
@Rossmaxx Rossmaxx changed the title Fix the MSVC CI fail happening due to outdated vcpkg + bump msvc-dev-ci version Fix the MSVC CI fail happening due to outdated vcpkg + bump msvc-dev-cmd version Nov 19, 2024
@Rossmaxx Rossmaxx changed the title Fix the MSVC CI fail happening due to outdated vcpkg + bump msvc-dev-cmd version Fix the MSVC CI fail happening due to outdated vcpkg Nov 20, 2024
@messmerd messmerd merged commit 0d7749d into LMMS:master Nov 20, 2024
11 checks passed
@Rossmaxx
Copy link
Contributor Author

Thanks messmerd 😊

@Rossmaxx Rossmaxx deleted the fix-msvc-ci branch November 21, 2024 04:19
rubiefawn pushed a commit to rubiefawn/lmms that referenced this pull request Nov 28, 2024
* switch to version 1 for msvc-dev-cmd github action

* bump msvc build year

* try refreshing vcpkg cache msvc

* Revert "try refreshing vcpkg cache msvc"

This reverts commit e8814f8.

* try updating vcpkg manually

* Revert "bump msvc build year"

This reverts commit a95c75e.

* messmerd's review comments

* revert the version bump for msvc-dev-cmd

* Fix `cd` command

---------

Co-authored-by: Dalton Messmer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants