Skip to content

Fix local use of msrustup#1456

Merged
andyleejordan merged 3 commits intoPowerShell:mainfrom
andyleejordan:ignore-msrustup
Apr 1, 2026
Merged

Fix local use of msrustup#1456
andyleejordan merged 3 commits intoPowerShell:mainfrom
andyleejordan:ignore-msrustup

Conversation

@andyleejordan
Copy link
Copy Markdown
Member

@andyleejordan andyleejordan commented Apr 1, 2026

In several !$useADO conditions, we were unfortunately not actually setting the specified Rust channel to use, and instead reverting back to default. This meant that e.g. Mikey was building with 1.94, while I was building with 1.93 (having manually set my toolchain).

Copilot AI review requested due to automatic review settings April 1, 2026 19:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@andyleejordan andyleejordan changed the title Git ignore for mssrustup needs wildcard for Windows Fix local use of msrustup Apr 1, 2026
@andyleejordan andyleejordan force-pushed the ignore-msrustup branch 2 times, most recently from 5446306 to 937c512 Compare April 1, 2026 21:02
The ADO pipeline was working as expected,
but local builds were behaving differently because of some old incorrect logic.
This is confirmed fixed in 1.94+. I tested ms-beta
which is 1.95, Mikey tested public 1.94. However,
even with ms-prod-1.93-20260401 (today's release),
this fails compilation (due to warning being
treated as error).

More unfortunately, the ignore must be at the
file-level due to how the macro expands. I tested
both on the field-level and enum-level, to no
avail.
@andyleejordan andyleejordan added this pull request to the merge queue Apr 1, 2026
Merged via the queue into PowerShell:main with commit 1ee1022 Apr 1, 2026
13 of 15 checks passed
@andyleejordan andyleejordan deleted the ignore-msrustup branch April 1, 2026 21:14
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