include directoryの表記をソースファイルとpkg-configを使った場合で同じにする

[YoheiKakiuchi]

#803
の変更にともなって、
ソースファイルでのinclude の表記とpkg-configを使った場合のincludeの表記が
同じになるように修正しました。

(io/iob.h -> hrpsys/io/iob.h,
xxx.hh -> hrpsys/idl/xxx.hh,
util/xxx.h -> hrpsys/util/xxx.h)

[k-okada]

Refer to this link for build results (access rights to CI server needed):
https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2384/

Build Log
last 10 lines

[...truncated 7560 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:201)
    at hudson.model.Run.execute(Run.java:1786)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:408)

Test FAILed.

[k-okada]

結構な変化なのでバージョン切りたいですが，この前で切って，この変更入れて切って，という感じでしょうか？作業中のものは
#839
#837
でしょうか．

[k-okada]

コンパイル中にsymbolic link貼りまくるのは正解だろうか？
他にやりかたがあるかというと難しいけど、
libにかんしてはinclude/hrpsys->libとはるeusスタイルもあるきがする
Idlはどうだろうかコンパイル時に出力さきをかえるとか、インストール時に書き換えるとか、#include <$EXPORT_PATH/ion.h> みたいな感じとか

[k-okada]

#867 で別実装をつくってみましたが，あまりかわらないかなぁ．

最終的にどういう風になるのがいいのかよくわかっていませんが，
１）ソースツリーは hrpsys-base/lib/io/iob.h に置かれている．
２）インストールするときは include/hrpsys/io/iob.hに置かれるようになる．
３）たのソース，たとえばec/hrpEC/hrpEC-common.cpp も，コンパイル時はhrpsys-base/ecにあるが，インストールするときはshare/hrpsys/src/ec/ に置かれる．ここに置かれたコードは，外部のプログラムが参照する可能性がある．なので，インストール後のhrpEC-common.cppは

@@ -1,5 +1,5 @@
 #include "hrpEC.h"
-#include "io/iob.h"
+#include "hrpsys/io/iob.h"

と変更する必要が有る

とすると，解法としては
A) そもそものソースツリーをhrpsys-base/include/hrpsys/io/iob.h とする
B)　インストールしたファイルのヘッダファイルを書き直す #867
なんではないかともうけどどうなんだろうか．

普通はどうしているんですかね？

[fkanehiro]

A)のような気がするけれども、具体的にはどんな問題が起きているのでしたっけ？
外部のプログラムからインクルードファイルが見つからなくなったということ？

2015年10月27日 12:05 Kei Okada [email protected]:

#867 #867
で別実装をつくってみましたが，あまりかわらないかなぁ．

最終的にどういう風になるのがいいのかよくわかっていませんが，
１）ソースツリーは hrpsys-base/lib/io/iob.h に置かれている．
２）インストールするときは include/hrpsys/io/iob.hに置かれるようになる．
３）たのソース，たとえばec/hrpEC/hrpEC-common.cpp も，コンパイル時はhrpsys-base/ec
にあるが，インストールするときはshare/hrpsys/src/ec/
に置かれる．ここに置かれたコードは，外部のプログラムが参照する可能性がある．なので，インストール後のhrpEC-common.cppは

@@ -1,5 +1,5 @@
#include "hrpEC.h"
-#include "io/iob.h"
+#include "hrpsys/io/iob.h"

と変更する必要が有る

とすると，解法としては
A) そもそものソースツリーをhrpsys-base/include/hrpsys/io/iob.h とする
B) インストールしたファイルのヘッダファイルを書き直す #867
#867
なんではないかともうけどどうなんだろうか．

普通はどうしているんですかね？

—
Reply to this email directly or view it on GitHub
#842 (comment)
.

[k-okada]

@YoheiKakiuchi could you create this PR with latest source tree?

[k-okada]

Refer to this link for build results (access rights to CI server needed):
https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2573/

Build Log
last 10 lines

[...truncated 8544 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:202)
    at hudson.model.Run.execute(Run.java:1783)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:410)

Test FAILed.

[YoheiKakiuchi]

最新ソースからPRつくり直しました。

外部のプログラムからインクルードファイルが見つからなくなったということ？

そうです。外部のiobとリンクするためにRobotHardwareをソースツリー外でコンパイルするときがあります。
（これ自体は、iobのリンク先を変える方法が思いつくのですが、そうとう格闘した結果うまくいっていない状況です）
また、hrpsys-baseのソースツリー外で作るrtcと内部のrtcでインクルードが違ってしまうということがあります。