check version in CMakeLists.txt against package.xml

[YoheiKakiuchi]

#990
と直接は関係ないのですが、バージョン番号が正しくrtcに反映されていないように思います。

ここのdefineでバージョン番号を設定しているのですが、それが、package.xmlのものとと違っています。
https://github.com/fkanehiro/hrpsys-base/blob/master/CMakeLists.txt#L176

このPRでは、違う場合に落ちるようになっています。

値を直接設定するために、find_package(catkin)をすると、問題がありました。

• docディレクトリでのtargetが重複する (catkinでロードしたときに先に設定されると思われる）
• subdirectoryにCMakeLists.txtがないとエラーになる (find_package(catkin)の有無で違いがある）
  　　こちらは、空のCMakeLists.txtを置けば解決するとは思う。

バージョン番号の付け方について、理解が足りていないのですが、現在は315.9.0がリリースされていますが、
それ以降にPRがあってソースコードに変更があってもバージョン番号は315.9.0に固定でしょうか？
リリース(deb)のあとに追加したソースコードの機能をテストしたい場合は、バージョンが上がっていてほしいと思うのですが、どうでしょうか？

[k-okada]

すいません．ただこれは，
https://github.com/fkanehiro/hrpsys-base/blob/master/CMakeLists.txt#L108
を更新するのを忘れていた，というだけでした．
次のリリースでは直します．

リリースしたらバージョンが上がります．IDLが変わると9.0が10.0，それ以外の変更は9.0から9.1というふうにしているつもりです．

一応いまのところは，hrpsys-baesはrosに依存せずにコンパイル出来るように，というふうにしてcatkinは入れていないです．が，
それでも一応rosrun hrpssy xxx 出来るようにするために，かなり変なことをしているので，
https://github.com/fkanehiro/hrpsys-base/blob/master/CMakeLists.txt#L212
if(caktin_FOUND) みたになのをこっそり入れよううかな，と思い始めているところです．

◉ Kei Okada

2016-05-13 17:47 GMT+09:00 Yohei Kakiuchi [email protected]:

#990 #990
と直接は関係ないのですが、バージョン番号が正しくrtcに反映されていないように思います。

ここのdefineでバージョン番号を設定しているのですが、それが、package.xmlのものとと違っています。
https://github.com/fkanehiro/hrpsys-base/blob/master/CMakeLists.txt#L176

このPRでは、違う場合に落ちるようになっています。

値を直接設定するために、find_package(catkin)をすると、問題がありました。

• docディレクトリでのtargetが重複する (catkinでロードしたときに先に設定されると思われる）
• subdirectoryにCMakeLists.txtがないとエラーになる
  (find_package(catkin)の有無で違いがある） こちらは、空のCMakeLists.txtを置けば解決するとは思う。

バージョン番号の付け方について、理解が足りていないのですが、現在は315.9.0がリリースされていますが、
それ以降にPRがあってソースコードに変更があってもバージョン番号は315.9.0に固定でしょうか？

リリース(deb)のあとに追加したソースコードの機能をテストしたい場合は、バージョンが上がっていてほしいと思うのですが、どうでしょうか？

You can view, comment on, or merge this pull request online at:

#991
Commit Summary

• check version in CMakeLists.txt against package.xml

File Changes

• M CMakeLists.txt
  https://github.com/fkanehiro/hrpsys-base/pull/991/files#diff-0 (7)

Patch Links:

• https://github.com/fkanehiro/hrpsys-base/pull/991.patch
• https://github.com/fkanehiro/hrpsys-base/pull/991.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#991

[k-okada]

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

[k-okada]

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