-
Notifications
You must be signed in to change notification settings - Fork 88
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
Fix include dir for QNX build #971
Conversation
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
今QNX@jenkinsで調べています。 @k-okada さん 今hrpsys/CMakeLists.txtに
と加えてみると、
とたくさんでてきます。 |
多分ですがlinuxでapt-get install ros-indigo-hrpsys して /opt にある iob.h ◉ Kei Okada 2016-04-08 12:27 GMT+09:00 Shunichi Nozawa notifications@github.com:
|
みたいにどこかに入れたらいいんだと思う. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
/opt/jsk/xxx以下は、QNXのhironxなどでのバイナリインストールしたhrpsys置き場でしたっけ? 方針としては、/opt/jsk以下のファイルでなくて、 |
2016-04-08 13:48 GMT+09:00 Shunichi Nozawa notifications@github.com:
そうですね.rosだと/opt/rosに対応します.
何をやっていたかちゃんとは覚えていないけど,これはQNXやテストの問題ではなくて,/optや/usrに古いファイルが合った時にそれを見に行ってしまう問題だと理解しているんだけど. ◉ Kei Okada |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
はい、そうなのですが、
のどちらを意図したののをお伺いできればと思ってました。 |
/opt/jskに置いた理由はHiroなどで/opt/grxにファイルがインストールされ、さらにpkgsrcにより/usr/pkgの下にバイナリがインストールされます。 2016年4月8日 14:04 Shunichi Nozawa notifications@github.com:
|
Refer to this link for build results (access rights to CI server needed): Build Log
Test FAILed. |
2016-04-08 14:11 GMT+09:00 Hajime SAITO notifications@github.com:
同じパッケージにソースとインクルードがある状況でそんなトリッキーな事が必要な状況はさすがに無い気がしますが,,, ◉ Kei Okada |
そうですよね、了解です。 @emijahさん、情報ありがとうございます。 ビルドの通し方がわかったのでコミットを直します。 |
Refer to this link for build results (access rights to CI server needed): |
… Hrpsys.h to pass QNX build.
原因と大暑をまとめます。 もともとinclude順番は、一番優先度が高いものはローカルファイルがあり、次に
のようになっていました。 原因としては、 直し方としては、CMakeLists.txtの中で最新iob.hのあるパスを |
https://github.com/fkanehiro/hrpsys-base/pull/842/files#diff-af3b638bc2a3e6c650974192a53c7291R50 ◉ Kei Okada 2016-04-08 15:13 GMT+09:00 Shunichi Nozawa notifications@github.com:
|
Refer to this link for build results (access rights to CI server needed): |
hrpsys-baseのソースディレクトリでは、
となっていて、インストールの時に、
へインストールされます。 pkg-configに従うと、以上のhrpsysディレクトリのあるincludeディレクトリを指しているので、(-I/foo/bar/include となる)
ソースディレクトリにhrpsysが無いので、どうしたものかとなって、シンボリックリンクにしています。 |
通りました |
うーん,とりあえずは今の方法しか無いというかんじですかね.LGTM ◉ Kei Okada On Fri, Apr 8, 2016 at 4:00 PM, Shunichi Nozawa notifications@github.com
|
travisとjenkinsで通りましたので、お手すきのときにmergeしていただけますと助かります。 |
ありがとうございます。 |
QNXでビルドを通すためのテストです。
まだマージなさらないようよろしくお願いいたします。
github.com//pull/842#issuecomment-204658811
#966 (comment)