Update for 315 13 0

[snozawa]

315.12.2へのアップデートです。

idlは

diff --git a/idl/AutoBalancerService.idl b/idl/AutoBalancerService.idl
diff --git a/idl/RemoveForceSensorLinkOffsetService.idl b/idl/RemoveForceSensorLinkOffsetService.idl
diff --git a/idl/StabilizerService.idl b/idl/StabilizerService.idl

の３つだけで、unstableだけなので、minorでなくpatchをあげました。

以下自分用めも

1. CHANGELOG.rstをつくる

   git clone https://github.com/fkanehiro/hrpsys-base
   cd hrpsys-base
   catkin_generate_changelog
   

   して、CHANGELOGの中身を編集して、
   git commit -m "update CHANGELOG" -a
2. package.xml, CHANGELOG.rst, CMakeLists.txtを変更
   今回はpatchバージョンを挙げるので

   catkin_prepare_release --no-push --bump patch
   

   これでpackage.xml, CHANGELOG.rstが自動でかわるので、CMakeLists.txtの番号を手動でコミットしてコミットをまとめる
3. PRする

よろしくお願いします。

[k-okada]

idlが変わればminorをあげていた気がします．

[snozawa]

すいません、わからなくなったので質問としましては、

• Stable/unstable idlがあがったらminorをあげる
• Stable idlがあがったらminorをあげる

のどちらでしょうか。

#1122
のPRでは、unstableのIDLがあがるのですがどうしましょう、
#1122 (comment)
という質問にたいし
#1122 (comment)
で315.12.1というお返事をいただいたので、後者なのかと思ってました。
（#1122まではstable/unstable idlがあがったらminorをあげるのだとおもってました）

[k-okada]

#1122 (comment) の返事がまちがえていますね． https://github.com/fkanehiro/hrpsys-base/pull/1122/files でidlを検索しても出てこないから変更ないのかとおもっていたのかな． 基本的には皆に再コンパイルを要求するような変更はminorをあげるのがいいと思います．皆、の定義は難しいわけだけど．

…

-- ◉ Kei Okada 2017年5月19日 14:45 Shunichi Nozawa <[email protected]>:

すいません、わからなくなったので質問としましては、 - Stable/unstable idlがあがったらminorをあげる - Stable idlがあがったらminorをあげる のどちらでしょうか。 #1122 <#1122> のPRでは、unstableのIDLがあがるのですがどうしましょう、 #1122 (comment) <#1122 (comment)> という質問にたいし #1122 (comment) <#1122 (comment)> で315.12.1というお返事をいただいたので、後者なのかと思ってました。 （#1122まではstable/unstable idlがあがったらminorをあげるのだとおもってました） — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1138 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAeG3IQpYySN1HLHFA6U5b2ZACahJBg7ks5r7SxwgaJpZM4NgFK0> .

[snozawa]

なるほど、ありがとうございます。

基本的には皆に再コンパイルを要求するような変更はminorをあげるのがいいと思います．皆、の定義は難しいわけだけど．

そうですね。
定義がムズいですが、idlはけっこう明確に

• 動く動かないを生じさせる
• 変更の有無の管理がみためでわかる

ので、idlがかわったらとにかくminorというのでよさそうですね。

バージョン番号かえて更新します

[snozawa]

idlに変更があるため、315.13.0にminorにしました。

以下自分用めも

1. CHANGELOG.rstをつくる

   git clone https://github.com/fkanehiro/hrpsys-base
   cd hrpsys-base
   catkin_generate_changelog
   

   して、CHANGELOGの中身を編集して、
   git commit -m "update CHANGELOG" -a
2. package.xml, CHANGELOG.rst, CMakeLists.txtを変更
   今回はidlに変更がありminorバージョンを挙げるので

   catkin_prepare_release --no-push --bump minor
   

   これでpackage.xml, CHANGELOG.rstが自動でかわるので、CMakeLists.txtの番号を手動でコミットしてコミットをまとめる
3. PRする

[snozawa]

@fkanehiroさん
travisとおりました。お手すきの時で構いませんので、ご確認のうえ、mergeおよび
git tag 315.13.0 && git push --tags
をお願いできますと幸いです
よろしくお願いします

[snozawa]

ありがとうございました