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

Update installation instruction for probe-rs #148

Merged
merged 1 commit into from
Jun 9, 2024

Conversation

OrangeTux
Copy link
Contributor

The binary probe-rs tools has been moved to it's own crate: probe-rs-tools. Therefore, the current instructions are outdated and fail:

$ cargo install probe-rs
    Updating crates.io index
  Downloaded probe-rs v0.24.0
  Downloaded 1 crate (1.7 MB) in 0.35s
error: there is nothing to install in `probe-rs v0.24.0`, because it has no binaries
`cargo install` is only for installing programs, and can't be used with libraries.
To use a library crate, add it as a dependency to a Cargo project with `cargo add`.

This commit updates the installation instruction.
For more information, see the [probe-rs' changelog]1.

Footnotes

  1. https://github.com/probe-rs/probe-rs/blob/master/CHANGELOG.md#0240

Copy link
Member

@BartMassey BartMassey left a comment

Choose a reason for hiding this comment

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

Could you please squash these changes and then force-push the result? Thanks much!

@BartMassey
Copy link
Member

Sigh. Thanks much for the timely update. We'll get it merged shortly.

The binary `probe-rs` tools has been moved to it's own crate:
`probe-rs-tools`. For more information, see the [probe-rs' changelog][^1].

[^1]: https://github.com/probe-rs/probe-rs/blob/master/CHANGELOG.md#0240
@OrangeTux OrangeTux force-pushed the fix-installation-instructions branch from 34034b5 to 56059d5 Compare June 8, 2024 08:40
@OrangeTux
Copy link
Contributor Author

Sure, here you go.

@OrangeTux OrangeTux requested a review from BartMassey June 8, 2024 08:41
Copy link
Member

@BartMassey BartMassey left a comment

Choose a reason for hiding this comment

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

Thanks much!

@BartMassey BartMassey added this pull request to the merge queue Jun 9, 2024
Merged via the queue into nrf-rs:main with commit cf93884 Jun 9, 2024
9 checks passed
@BartMassey
Copy link
Member

Appreciate your help with this.

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