Skip to content
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

Don't install msys2 in CI #136588

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ChrisDenton
Copy link
Member

@ChrisDenton ChrisDenton commented Feb 5, 2025

windows-msvc doesn't need it and windows-gnu installs its own version

This might still fail on window-msvc if it turns out we need to setup Visual Studio in PATH. I don't know how the default bash is configured. This will fail on the one remaining Makefile test but let's see if it gets that far.

try-job: x86_64-msvc-1
try-job: i686-msvc-1
try-job: i686-mingw
try-job: x86_64-mingw-1

r? ghost

windows-msvc doesn't need it and windows-gnu has its own version
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 5, 2025
@ChrisDenton
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 5, 2025
Don't install msys2 in CI

windows-msvc doesn't need it and windows-gnu installs its own version

This might still fail on window-msvc if it turns out we need to setup Visual Studio in `PATH`. I don't know how the default `bash` is configured. This will fail on the one remaining run-make test but let's see if it gets that far.

try-job: x86_64-msvc-1
try-job: i686-msvc-1
try-job: i686-mingw
try-job: x86_64-mingw-1

r? ghost
@bors
Copy link
Contributor

bors commented Feb 5, 2025

⌛ Trying commit c94ab20 with merge 6896d6e...

@bors
Copy link
Contributor

bors commented Feb 5, 2025

☀️ Try build successful - checks-actions
Build commit: 6896d6e (6896d6e9adfe1dd2de222e2a597a970ef8064fc0)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants