Skip to content

doc: Ssteiner/update docs#5

Merged
StefanSteiner merged 7 commits into
tableau:mainfrom
StefanSteiner:ssteiner/update-docs
May 16, 2026
Merged

doc: Ssteiner/update docs#5
StefanSteiner merged 7 commits into
tableau:mainfrom
StefanSteiner:ssteiner/update-docs

Conversation

@StefanSteiner
Copy link
Copy Markdown
Contributor

testing

TEMPORARY diagnostic step added before actions-rust-lang/setup-rust-toolchain
in the test job to collect evidence about the post-2026-05-12 macos-14
image regression where the action fails with "rustc: command not found".

Logs PATH, contents of ~/.cargo/bin and ~/.rustup, which rustup/rustc/cargo,
rustup show output, bash version+location, and the rust-toolchain.toml
contents. To be removed once root cause is identified and fixed.
…new image

Diagnostic output from a CI run on the post-2026-05-12 macos-14 image
showed the actual root cause of the "rustc: command not found" failure:

  === which rustup / rustc / cargo ===
  /opt/homebrew/bin/rustup
  rustc: not found
  cargo: not found

  === rustup show ===
  installed toolchains: stable-aarch64-apple-darwin (active, default)

The new image provisions Rust correctly via Homebrew at /opt/homebrew/bin/
with rustup-managed toolchains. There is no symlink shadowing of cargo
to rustup-init anymore — the original problem the brew-uninstall step was
defending against has been fixed upstream.

But our defensive `rm -f /opt/homebrew/bin/{cargo,rustc,rustup-init}`
deletes the working cargo and rustc symlinks while leaving rustup intact.
The setup-rust-toolchain action then runs `rustc --version` for its
cachekey output and fails because we just removed rustc.

Remove the brew-uninstall steps from ci.yml and npm-build-publish.yml.
Also removes the temporary debug step added in 0187d8e.
@StefanSteiner StefanSteiner merged commit 49bd97b into tableau:main May 16, 2026
9 checks passed
@StefanSteiner StefanSteiner deleted the ssteiner/update-docs branch May 16, 2026 02:28
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