Robot で動作する workspace update スクリプトの挙動についての議論

[sktometometo]

https://github.com/jsk-ros-pkg/jsk_robot/pull/1808 での議論を受けて、各ロボットで用意するといい workspace update スクリプトの仕様について議論したいです。

参考:

• TODO

[sktometometo]

https://github.com/jsk-ros-pkg/jsk_robot/issues/1821#issuecomment-1600523034

についてなんだけど、これは僕の想定が甘くてcommitはしたけど push していない branch が体内にあることは考えていませんでした。commitしていない変更については stash するようにしているはず。

[sktometometo]

@mqcmd196

[sktometometo]

update_workspace.sh のデバッグが結構大変で (特に rebase で divergeした際に勝手に修正する機能) あんまりいろいろなケースは想定していませんでした、他に問題になっているor使いづらい部分はなにかありますか？ @mqcmd196 @tkmtnt7000 @nakane11

[mqcmd196]

https://github.com/jsk-ros-pkg/jsk_robot/blob/1cddbe050e2e7832f9ae0fa5fd6e4ed37a0c8bcf/jsk_robot_common/jsk_robot_startup/scripts/update_workspace_main.sh#L83 確かに大丈夫な気もする．ちなみにこれだと未追跡ファイルがあるとstash失敗する気がするので，git stash -u が正解ですね．

@tkmtnt7000 だとするとなんで変更が完全に消えちゃうんだっけ？

[tkmtnt7000]

先ほどのオフラインでの僕の伝え方が悪かった気もするのですが、 少し特殊ケースで、変更が消えると考えられるのは誤ってdevelop/fetchなどのrosinstallで設定されたブランチにロボット体内でローカルコミットだけ残した場合です。コミットしていないdiffはstashされて残っていると思います。