Skip to content

fix: Select the toolchain properly - #5

Merged
flub merged 1 commit into
mainfrom
flub/nightly-fix
Sep 28, 2026
Merged

flub merged 1 commit into
mainfrom
flub/nightly-fix

Conversation

@flub

@flub flub commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator

See n0-computer/iroh#4515.

This edited a file that's being deleted in n0-computer/iroh#4505

@flub
flub requested review from Arqu and matheus23 September 28, 2026 14:13
@matheus23

Copy link
Copy Markdown
Member

Oh! I wasn't aware of this work! Good to know.

@flub
flub merged commit 4e050c9 into main Sep 28, 2026
@flub
flub deleted the flub/nightly-fix branch September 28, 2026 14:50
@Arqu

Arqu commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

👍

rustup target add ${{ matrix.target }}
rustup set default-host ${{ matrix.target }}
rustup toolchain install nightly --target "${{ matrix.target }}"
rustup set default-host "${{ matrix.target }}"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

tbh think this part is not needed if you set the RUSTUP_TOOLCHAIN for the job

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

this is the target-tuple though, not the toolchain version?

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