-
Notifications
You must be signed in to change notification settings - Fork 5.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Windows Server 2022 with Visual Studio 2022 #4775
Conversation
9dcdb6c
to
3747c6a
Compare
acb9cb0
to
890114b
Compare
echo VCVARS="C:\Program Files\Microsoft Visual Studio\2022\Preview\VC\Auxiliary\Build\vcvars64.bat" >> %GITHUB_ENV% | ||
) ELSE IF ${{ matrix.os }}==windows-2019 ( | ||
echo VCVARS="C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Auxiliary\Build\vcvars64.bat" >> %GITHUB_ENV% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes %VCVARS%
to include double quotes.
It feels weird a little.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PATH without double quotes causes an error.
So I add double quotes to echo
again, and remove from call
.
https://github.com/ruby/ruby/runs/3418598073?check_suite_focus=true#step:4:11
Run IF windows-2022==windows-2022 (
IF windows-2022==windows-2022 (
echo VCVARS=C:\Program Files\Microsoft Visual Studio\2022\Preview\VC\Auxiliary\Build\vcvars64.bat >> %GITHUB_ENV%
) ELSE IF windows-2022==windows-2019 (
echo VCVARS=C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Auxiliary\Build\vcvars64.bat >> %GITHUB_ENV%
)
shell: C:\Windows\system32\cmd.EXE /D /E:ON /V:OFF /S /C "CALL "{0}""
env:
GITPULLOPTIONS: --no-tags origin refs/pull/4775/merge
PATCH: C:\msys64\usr\bin\patch.exe
\Microsoft was unexpected at this time.
Error: Process completed with exit code 255.
402a314
to
8644262
Compare
Because patch does not installed on windows-2022. actions/runner-images#3949 > Only pure MSYS2 is installed. No additional packages
2b43c2a
to
019e476
Compare
https://github.blog/changelog/2021-08-23-github-actions-windows-server-2022-with-visual-studio-2022-is-now-available-on-github-hosted-runners-public-beta/