Release new version (2017/10)

[ishiguroJSK]

@k-okada さん， 最新のopenhrp3の機能を必要とするhrpsysのTravis(aptのopenhrp3との組み合わせ)が軒並み通らないので， ros-indigo-openhrp3のreleaseの手続きを始めてもよろしいでしょうか？

[snozawa]

（横からすいません、参考までに）

https://github.com/fkanehiro/hrpsys-base/pull/1214#issuecomment-338589368 にhrpsys-baseのCHANGELOG修正とtagづけまでの流れを少し細かくかきました。 openhrp3の場合も基本は過去のPR https://github.com/fkanehiro/openhrp3/pull/121/files などを参考にしながらつくると良さそうです。 （hrpsys-baseと大きく違うのは、HISTORYファイルもアップデートが必要な点？）

pkgconfigまわりの重要な変更もあったとおもうので、releaseは行うのが良さそうに思います。

一方、hrpsys-baseではhydroのテストも行っており、hydroはdebが更新されないので、 テストを通すという目的に対してはhrpsys-base側で何かopenhrp3バージョンを見て判断する if文的なものもいるのではないかと想像しています。

[k-okada]

新しいPRについて#ifdef はできないかな．

diff --git a/rtc/AutoBalancer/CMakeLists.txt b/rtc/AutoBalancer/CMakeLists.txt
index 3625bd8..c18dd28 100644
--- a/rtc/AutoBalancer/CMakeLists.txt
+++ b/rtc/AutoBalancer/CMakeLists.txt
@@ -1,3 +1,10 @@
+add_definitions(-DOPENHRP_PACKAGE_VERSION=\"\\"${OPENHRP_VERSION}\\"\")
+message(STATUS "OpenHRP3 Version : ${OPENHRP_VERSION}")
+if ("${OPENHRP_VERSION}" VERSION_GREATER "3.1.9")
+  message(STATUS "enable OpenHRP3 3.2")
+  add_definitions(-DOPENHRP_PACKAGE_VERSION_320)
+endif()
+

[ishiguroJSK]

ifdefでhrpsysのTravisを通るようにしました

(ちなみにadd_definitions(-DOPENHRP_PACKAGE_VERSION=\"\\"${OPENHRP_VERSION}\\"\")は今回つかわないと思いますが今後必要ということでしょうか？)

そして結局今回openhrp3の3.2.0のタグを切るということでよろしいでしょうか？

[snozawa]

どうしましょう。 3.1.10なんでしょうか。

[k-okada]

3.2.0でいいとおもいます．APIが変わっているので，でも，3.2.0の次にすぐ3.3.0にくるとかっこわるいので，APIが変わるものはこの際入れたいところです．

-- ◉ Kei Okada

2017-10-25 1:02 GMT+09:00 Shunichi Nozawa notifications@github.com:

どうしましょう。 3.1.10なんでしょうか。

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/fkanehiro/openhrp3/issues/130#issuecomment-339041897, or mute the thread https://github.com/notifications/unsubscribe-auth/AAeG3KJ65C0jxPtOwWzwJFogQO5NDS5Vks5svgokgaJpZM4QCbpA .

[ishiguroJSK]

今のところ，これはどうしてもOpenHRP側に…という実装は特に無いですね． 3.2.0承知しました．