Skip to content
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

[jsk_tools/env-hooks/99.jsk_tools.sh] Fix setting ROS_IP at robot local #1474

Closed
wants to merge 2 commits into from

Conversation

kyawawa
Copy link
Member

@kyawawa kyawawa commented Nov 1, 2016

Fix #1472 at onboard PC

@kyawawa
Copy link
Member Author

kyawawa commented Nov 22, 2016

問題がなければこちら,マージしていただけると助かります

@k-okada
Copy link
Member

k-okada commented Nov 23, 2016

なぜこれ必要かわからないです.
#1472 (comment)
によると、

横から失礼します.

ip -o -4 route get
を実行した際,自身のIPアドレスを指定した場合先頭にlocalが付いてしまうように思えます(なので,体内でしか問題が起こらない).

となっていますが、

$ ip -o -4 route get 192.168.111.7
local 192.168.111.7 dev lo  src 192.168.111.7 \    cache <local> 

と、localがつく環境でも、

$ rossetip
set ROS_IP and ROS_HOSTNAME to 192.168.111.7

と問題は置きません.

@kyawawa
Copy link
Member Author

kyawawa commented Nov 23, 2016

https://github.com/jsk-ros-pkg/jsk_common/blob/master/jsk_tools/env-hooks/99.jsk_tools.sh#L110
master_hostがlocalhostの場合ここの条件を通過するので問題が起きませんが,各ロボットは

$ rossetmaster ${HOSTNAME}

に該当することをしているので,今回の問題が起こってしまいます.
手元のPCでも上のコマンドを打つことで再現します.

@furushchev
Copy link
Member

放置していました。ごめんなさい。
自分自身をmasterにしてrossetipを確かめてませんでした。

#1487

にもう少し整理したものをプルリクエストしました。

@k-okada
Copy link
Member

k-okada commented Nov 28, 2016

closed due to #1487

@k-okada k-okada closed this Nov 28, 2016
@kyawawa kyawawa deleted the fix_rossetip branch November 29, 2016 06:43
@kyawawa
Copy link
Member Author

kyawawa commented Nov 29, 2016

@furushchev ありがとうございます

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants