ciniml / rust-dap

CMSIS-DAP Rust implementation
Apache License 2.0
88 stars 10 forks source link

Update to rp2040-hal=0.8, pio=0.2 #53

Closed elfmimi closed 1 year ago

elfmimi commented 1 year ago

This will pave the way for further development. including fixing #40.

elfmimi commented 1 year ago

動作確認はまだです。 edited: 簡単に動作確認しました。

elfmimi commented 1 year ago

CI用のrustのバージョンいくつにするのが良いですか?

ciniml commented 1 year ago

今の最新 (1.71.1) でいいんじゃないですかね。

elfmimi commented 1 year ago

1.71.1 にしました。

elfmimi commented 1 year ago

よく考えると rust-dap-rp2040 のリビジョンも変えるのが筋な気がします。 0.1.0 → 0.2.0 でしょうか?

※変更点が溜まってきてpublishする時にバージョンバンプするというのもよく見ますね。

ciniml commented 1 year ago

たしかに。rust-dapの方も一緒に 0.2.0 に上げてしまいましょうかね。

elfmimi commented 1 year ago

rust-dapの方も一緒に 0.2.0 に上げてしまいましょうかね。

rust-dap の方は 0.1.0 だと思い込んでましたが、 edition が変わった時に 0.2.0 になってました。そうか... https://github.com/ciniml/rust-dap/commit/07720a35ded7b4ebe2b3453c3bd209218358a567

ciniml commented 1 year ago

edition が変わった時に 0.2.0 になってました。

完全に忘れてた…

elfmimi commented 1 year ago

あとは、Cargo.toml の他の部分では 0.8 表記や 0.2 表記など(パッチ番号は指定しない)例が多いです。それに合わせましょうか?

もろもろ決断ヨロシクです。

ciniml commented 1 year ago

rust-dap本体には影響なさそうですし、

にしましょう。

elfmimi commented 1 year ago

これで良いかと思います。

elfmimi commented 1 year ago

rust 1.72 が出ているので更新しました。

ciniml commented 1 year ago

遅くなってすみません。ご対応ありがとうございます。

手元で動作確認して動くを確認しました。マージします。