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

Catkin support #4

Merged
merged 3 commits into from
Aug 28, 2014
Merged

Catkin support #4

merged 3 commits into from
Aug 28, 2014

Conversation

k-okada
Copy link
Contributor

@k-okada k-okada commented Aug 19, 2014

#2 (comment)
に従って,catkin_make 出来るようにしました.思ったより色々問題がでてきました.
という事でマージはチョット待って下さい.

k-okada@2fa0703
で,とりあえずcatkin_make でいますが,

$ ls -al /opt/ros/hydro/bin/rtm-config 
lrwxrwxrwx 1 root root 46  8月 19 20:55 /opt/ros/hydro/bin/rtm-config -> opt/ros/hydro/lib/openrtm_aist/bin/rtm-config

となるように,openrtm_aist で変更しなければいけません.まだやっていないので手動でお願いします.

これで,

mkdir ws/src
cd ws/src
git clone https://github.com/fkanehiro/openhrp3
cd ..
catkin_make

でコンパイル出来るようになりました.ただこれだけだと,start-jsk/openhrp3と等価にならずに,
k-okada@10ac2310d3c7f901c3e766771be039956060e91d,
k-okada@9e8d0d3
が必要です.というのもrospack find openhrp3/share/OpenHRP-3.1/sample (https://github.com/start-jsk/openhrp3/blob/master/test/test_openhrp3.py#L63)が動くように,とか,`rosrun openhrp3 openhrp3-model-loader`が動くように,とおもうと,catkinなファイル構造(http://www.ros.org/reps/rep-0122.html)でないといけないです.
ということで,かなり汚くなっています.

また,さらに,

catkin_make test

すると,devel以下に対するテストして,installeいかが対象では無いので,
https://github.com/k-okada/openhrp3-1/blob/catkin_support/CMakeLists.txt#L564
などとしてinstalledディレクトリを汚くしたりしています.ただ,これは何とかなるかもしれないですが,,,
(devel/lib にライブラリがあるけど,binaryはbuild/bin にあるのは何かの環境変数?これを,devel/bin に
書き出させるのは可能そう,でもそれでも,rosrun openhrp3 openhrp-model-loaderにならないので,
なにか問題あると思う.

start-jsk/openrtm_aist_core#1 で色々議論したけど,結局サードパーティの
物はサードパーティとしてROS-FHSでない方法でインストールしてライブラリとしてつかって,rosrun xxは
諦める,というのがいいのかなぁ.どうでしょうか.ただ,そうすると,この話の目的のrostestを使う,もデキなく
なってしまう.moveit/opencvなどでもう一度チェックが必要ですね.

@k-okada k-okada force-pushed the catkin_support branch 3 times, most recently from e5dc95b to d7f3516 Compare August 26, 2014 11:03
@k-okada
Copy link
Contributor Author

k-okada commented Aug 27, 2014

すこしなおしました.
https://github.com/k-okada/openhrp3-1/blob/c9bd2780b0d07401c5c252b4a97347537c51fc8c/CMakeLists.txt#L557
で,20行ぐらい追加するはめになっていますが,一応これで,テストができるようになっています.
k-okada@2af6b89
に書いてある手順でできるとおもいます.
openrtmは自分でビルドしていればそれを使えるはずで,
ros-hydro-catkin
をインストールすればいいはずです.ソースからやりたければ,
https://github.com/catkin/catkin_tools
でしょうか.

ただ,テストを通すには
#9
が必要です.

fkanehiro added a commit that referenced this pull request Aug 28, 2014
@fkanehiro fkanehiro merged commit 25dd3a3 into fkanehiro:master Aug 28, 2014
@k-okada k-okada deleted the catkin_support branch September 6, 2014 06:05
k-okada pushed a commit to k-okada/openhrp3 that referenced this pull request Feb 6, 2015
k-okada pushed a commit to k-okada/openhrp3 that referenced this pull request Feb 6, 2015
k-okada pushed a commit to k-okada/openhrp3 that referenced this pull request Feb 6, 2015
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.

2 participants