Skip to content

Add cargo bin to PATH after rustup-init fallback - #321

Merged
Finesssee merged 1 commit into
mainfrom
fix/circleci-rustup-init-path
Aug 15, 2026
Merged

Add cargo bin to PATH after rustup-init fallback#321
Finesssee merged 1 commit into
mainfrom
fix/circleci-rustup-init-path

Conversation

@Finesssee

@Finesssee Finesssee commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • after rustup-init.exe installs rustup to ~/.cargo/bin, explicitly add that directory to PATH via Add-PrerequisitePath before Refresh-PrerequisitePath
  • --no-modify-path flag (added in PR Fix rustup-init fallback: wrap in ErrorActionPreference Continue #320) prevents rustup-init from modifying PATH env vars, so the install directory was never on PATH when Require-PrerequisiteCommand checked for rustup

Root cause

Pipeline #36: choco install rustup.install failed 3x (504 Gateway Timeout from Chocolatey community feed). The rustup-init.exe fallback ran but rustup was still not found on PATH because --no-modify-path prevented the install directory from being added to PATH, and Refresh-PrerequisitePath only refreshes from existing Machine+User PATH env vars.

Validation

  • PowerShell parser passed for all release scripts
  • scripts/release-pipeline.tests.ps1 passed
  • circleci config validate .circleci/config.yml passed
  • no version files changed

View with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is enabled.

@Finesssee
Finesssee merged commit 3313bd4 into main Aug 15, 2026
3 checks passed
@Finesssee
Finesssee deleted the fix/circleci-rustup-init-path branch August 15, 2026 06:52
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.

1 participant