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? 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

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