Release new version (2017/10)

[ishiguroJSK]

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

[snozawa]

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

fkanehiro/hrpsys-base#1214 (comment)
に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 <[email protected]>:

どうしましょう。 3.1.10なんでしょうか。 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#130 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAeG3KJ65C0jxPtOwWzwJFogQO5NDS5Vks5svgokgaJpZM4QCbpA> .

[ishiguroJSK]

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