Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Closed
sktometometo opened this issue Jun 21, 2023 · 5 comments

Comments

@sktometometo
Copy link
Contributor

#1808 での議論を受けて、各ロボットで用意するといい workspace update スクリプトの仕様について議論したいです。

参考:

  • TODO
@sktometometo
Copy link
Contributor Author

#1821 (comment)

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

@sktometometo
Copy link
Contributor Author

@mqcmd196

@sktometometo
Copy link
Contributor Author

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

@mqcmd196
Copy link
Member

mqcmd196 commented Jun 21, 2023

wstool foreach -t $WORKSPACE/src --git 'git stash'

確かに大丈夫な気もする.ちなみにこれだと未追跡ファイルがあるとstash失敗する気がするので,git stash -u が正解ですね.

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

@tkmtnt7000
Copy link
Member

tkmtnt7000 commented Jun 21, 2023

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

@sktometometo sktometometo pinned this issue Jun 21, 2023
@jsk-ros-pkg jsk-ros-pkg locked and limited conversation to collaborators Jun 21, 2023
@sktometometo sktometometo converted this issue into discussion #1823 Jun 21, 2023
@sktometometo sktometometo unpinned this issue Jun 21, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Projects
None yet
Development

No branches or pull requests

3 participants