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 probe-run list-probes command in README #22

Merged
merged 1 commit into from
May 6, 2023

Conversation

nmarley
Copy link
Contributor

@nmarley nmarley commented May 1, 2023

Hi, I noticed when I followed the current instructions in the README to use probe-run, I got an error:

probe-run --chip RP2040 --list-probes
error: the argument '--chip <CHIP>' cannot be used with '--list-probes'

Usage: probe-run --chip <CHIP> <ELF>

For more information, try '--help'.

I'm on macOS Monterey 12.6.5 (21G531) and currently here's my version from probe-run:

probe-run --version
0.3.8
supported defmt version: 4

Once I removed the --chip argument I got this:

probe-run --list-probes
the following probes were found:
[0]: Picoprobe CMSIS-DAP (VID: 2e8a, PID: 000c, Serial: E6616408438E3C21, CmsisDap)

I am guessing this is the intended behaviour, and that the --chip and --list-probes arguments no longer work together.

Hi, I noticed when I followed the [current instructions](https://github.com/rp-rs/rp-hal-boards/blob/2f8d0f4404c32cf8877c57812dd3649a80431380/README.md?plain=1#L413) in the README to use probe-run, I got an error:

```sh
probe-run --chip RP2040 --list-probes
error: the argument '--chip <CHIP>' cannot be used with '--list-probes'

Usage: probe-run --chip <CHIP> <ELF>

For more information, try '--help'.
```

I'm on macOS Monterey 12.6.5 (21G531) and currently here's my version from probe-run:

```sh
probe-run --version
0.3.8
supported defmt version: 4
```

Once I removed the `--chip` argument I got this:

```sh
probe-run --list-probes
the following probes were found:
[0]: Picoprobe CMSIS-DAP (VID: 2e8a, PID: 000c, Serial: E6616408438E3C21, CmsisDap)
```

I am guessing this is the intended behaviour, and that the `--chip` and `--list-probes` arguments no longer work together.
Copy link
Member

@jannic jannic left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@jannic
Copy link
Member

jannic commented May 2, 2023

The pipeline error is unrelated. I hope it'll be fixed by #23.

@jannic jannic merged commit 53ee09e into rp-rs:main May 6, 2023
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.

2 participants