ci: Replace `ilammy/msvc-dev-cmd` with manual MSVC setup #1846

pull hebasto wants to merge 1 commits into bitcoin-core:master from hebasto:260413-ci-win changing 1 files +2 −3

hebasto commented at 11:04 AM on April 13, 2026: member

The ilammy/msvc-dev-cmd repository seems abandoned and should be considered unsafe.

This PR updates the workflow to load the MSVC environment variables directly via vcvars64.bat.

For reference, the Bitcoin Core project removed ilammy/msvc-dev-cmd in https://github.com/bitcoin/bitcoin/pull/32513.

Note for Maintainers: Once this PR is merged and other PRs are rebased on top of it, the ilammy/msvc-dev-cmd action should be removed from the "Action permission" settings in this repository.

ci: Replace `ilammy/msvc-dev-cmd` with manual MSVC setup

The `ilammy/msvc-dev-cmd` repository seems abandoned and should be
considered unsafe. This updates the workflow to load the MSVC
environment variables directly via `vcvars64.bat`.

See https://learn.microsoft.com/en-us/cpp/build/building-on-the-command-line.

285cb788e9

real-or-random approved
real-or-random commented at 11:09 AM on April 13, 2026: contributor

utACK https://github.com/bitcoin-core/secp256k1/pull/1846/changes/285cb788e9da2a965178cc8e0769e4821ff0c799

-1 line and a removed dependency. Nice.
real-or-random added the label ci on Apr 13, 2026
real-or-random added the label tweak/refactor on Apr 13, 2026
real-or-random merged this on Apr 13, 2026
real-or-random closed this on Apr 13, 2026
real-or-random commented at 1:13 PM on April 13, 2026: contributor

Note for Maintainers: Once this PR is merged and other PRs are rebased on top of it, the ilammy/msvc-dev-cmd action should be removed from the "Action permission" settings in this repository.

Done
hebasto deleted the branch on Apr 13, 2026

Contributors

hebasto

real-or-random

Labels

ci tweak/refactor