Skip to content

Conversation

@pazeshun
Copy link
Collaborator

@pazeshun pazeshun commented Nov 14, 2019

Updated version of #240
Includes #241 and #237
#241 で失敗するtravisが、このPRでの修正により成功するようになります。

@YoheiKakiuchi #219 と同じ問題がhrpsys_gazebo_atlasで起きたのですが、修正方法を #221 ではなく、fkanehiro/hrpsys-base#842 と同様 (io/iob.h -> hrpsys/io/iob.h) とし、hrpsys_gazebo_generalにおいても #220 をrevertして同様の変更を行いました。これで大丈夫でしょうか。

@iory @Naoki-Hiraoka #237 をindigoでもビルドできるように修正しました。これでindigo, kineticのユースケースに対応できているか、ご確認ください。

@pazeshun
Copy link
Collaborator Author

昨日までアクセスできていた
http://packages.osrfoundation.org/drc
にアクセスできず、travisが落ちています。復活するまで待機します。

@YoheiKakiuchi
Copy link
Member

@YoheiKakiuchi #219 と同じ問題がhrpsys_gazebo_atlasで起きたのですが、修正方法を #221 ではなく、fkanehiro/hrpsys-base#842 と同様 (io/iob.h -> hrpsys/io/iob.h) とし、hrpsys_gazebo_generalにおいても #220 をrevertして同様の変更を行いました。これで大丈夫でしょうか。

このやり方で問題ないと思います。

@pazeshun pazeshun force-pushed the fix-indigo-kinetic-build branch from ab56c3e to 11de69e Compare November 21, 2019 03:33
@pazeshun
Copy link
Collaborator Author

昨日までアクセスできていた
http://packages.osrfoundation.org/drc
にアクセスできず、travisが落ちています。復活するまで待機します。

Open Roboticsにメールで聞いたところ、以下のURLに移動したようです。
http://packages.osrfoundation.org/gazebo/ubuntu-stable
#241 , #242 ともに、新しいURLに変えてテスト中です。

@pazeshun
Copy link
Collaborator Author

#241 で失敗するtravisがこのPRで成功するのを確認しました。

@pazeshun
Copy link
Collaborator Author

@ban-masa のindigo環境で、ビルドができ、HRP2が歩くことを確認しました。

@Naoki-Hiraoka
Copy link
Contributor

Naoki-Hiraoka commented Dec 3, 2019

kineticのユースケースに対応できていることを確認しました。
https://github.com/Naoki-Hiraoka/rtmros_gazebo/tree/gazebo-hrp2jsknts-p

ビルドができ、HRP2が歩くことを確認しました。

@k-okada k-okada merged commit 6ba0c1e into start-jsk:master Dec 11, 2019
k-okada added a commit that referenced this pull request Dec 11, 2019
…trollers

[on #242] add options to gazebo_robot_no_controllers.launch
@pazeshun pazeshun deleted the fix-indigo-kinetic-build branch December 11, 2019 11:53
k-okada added a commit that referenced this pull request Jul 10, 2020
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.

5 participants