search

rp-rs / rp2040-project-template

A basic rp2040-hal project with blinky and rtt logging example code. With this you can quickly get started on a new rp2040 project
480 stars 98 forks source link

Update probe-rs tools to 0.19 #64

Closed 9names closed 1 year ago

9names commented 1 year ago

The latest version of probe-rs moved all of the tools behind a single crate - update instructions to use this.

Also updated probe-rs-debugger instructions, since you can now install the plugin through the regular extensions marketplace instead of having to manually install.

Lastly, removed a stray </details> tag - not sure when that got introduced

c-h-johnson commented 1 year ago

Needs to be changed here too: https://github.com/rp-rs/rp2040-project-template/blob/4db7b897e6e22498c9c42920d0e4fbd57bf784f3/.cargo/config.toml#L5-L8

Dicklessgreat commented 1 year ago

Needs to change launch.json also. "runtimeExecutable": from "probe-rs-debugger" to "probe-rs" "runtimeArgs": from "debugger" to "dap-server"

9names commented 1 year ago

Thanks @Dicklessgreat, I didn't notice the debugger config wasn't correct because I still had an older copy of probe-rs-debugger installed that it was using.

jannic commented 1 year ago

I added a note regarding removal of cargo-embed. The option "create separate branch", which I wanted to use first, didn't work. So I opted to directly push to your branch, @9names. I hope that's fine with you, otherwise feel free to remove the commit.

9names commented 1 year ago

I thought you meant putting that notice at the cargo-embed section. I did attempt to find a way to say it there, but it just didn't work well and got in the way. Up where you've put it is okay.

jannic commented 1 year ago

I thought you meant putting that notice at the cargo-embed section. I did attempt to find a way to say it there, but it just didn't work well and got in the way. Up where you've put it is okay.

Yes, I also struggled to find a proper way to include that information. In fact, I'd remove it again in a few months, because it clutters the instructions. But for now I expect it to be a common issue for people upgrading from older versions, so it's better to mention the solution somewhere.

9names commented 1 year ago

I think this is good to merge now, thanks everyone.