Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update for 315 15 0 #1214

Merged
merged 2 commits into from
Oct 23, 2017
Merged

Update for 315 15 0 #1214

merged 2 commits into from
Oct 23, 2017

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Oct 22, 2017

315.15.0へのアップデートです。
idlに変更があるため、315.15.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する

@k-okada
Copy link
Contributor

k-okada commented Oct 22, 2017

近日中に#1206
でまたIDLが変わるなら,今のうちにいれておくのがいいのでは?

まだまだかなり検討時間が必要ならよいのですが..

@snozawa
Copy link
Contributor Author

snozawa commented Oct 22, 2017

まだまだかなり検討時間が必要ならよいのですが..

そうですね。別途 @ishiguroJSK とtagづけするけどどうしましょう、と話し合って、
#1206 は次回にすることにしました。

@snozawa snozawa closed this Oct 23, 2017
@snozawa snozawa reopened this Oct 23, 2017
@fkanehiro fkanehiro merged commit ebb7202 into fkanehiro:master Oct 23, 2017
@snozawa
Copy link
Contributor Author

snozawa commented Oct 23, 2017

FYI : @ishiguroJSK

あまり文章でかくのもアレですが、CHANGELOGなどをかくときのメモを少し詳しめに書きました。
openhrp3は、おそらく1の工程での編集するファイルが少し多いかも(HYSTORY-en, jaなど)

  1. CHANGELOG.rstをつくる

    1. git clone

      git clone https://github.com/fkanehiro/hrpsys-base
      cd hrpsys-base
      
    2. CHANGELOG.rstを自動で生成するコマンド

      catkin_generate_changelog
      

      前回tagづけ時からの変更が一覧になって.rstファイルの先頭から追記される

    3. CHANGELOGの中身を編集する
      ii.のCHANGELOR.rstはPRやコミットがdateで並んでるだけなので見やすくする
      hrpsys-baseの場合、

      • Stable RTCとUnstable RTCを分けて書く
      • 個々のRTC(など)ごとに箇条書きして、簡単にサマリをかく
      • 詳しくは過去のtagづけしているPRを参照(Update for 315 13 0 #1138)
    4. CHANGELOG.rstをコミット

      git commit -m "update CHANGELOG" -a CHANGELOG.rst
      
  2. バージョン番号をあげるコミット

    1. package.xml, CHANGELOG.rst
      catkin_prepare_release --no-push --bump major
      # か、もしくは
      catkin_prepare_release --no-push --bump minor
      # か、もしくは
      catkin_prepare_release --no-push --bump patch
      
      これでpackage.xml, CHANGELOG.rstが自動でかわり、バージョン番号が上がる(git commitもされている)。
      patch, minor, majorに関しては後述
    2. CMakeLists.txtを変更
      CMakeLists.txtの番号を手動でコミットしてコミットをまとめる
      set(CPACK_PACKAGE_VERSION_MAJOR 315)
      set(CPACK_PACKAGE_VERSION_MINOR 14)
      set(CPACK_PACKAGE_VERSION_PATCH 0)
      
      などの番号を、package.xml, CHANGELOG.rstに基づき適宜あげ、git commitする
    3. 上記2つのコミットを一個にまとめる (git rebase -i HEAD^^など)
  3. PRする

  4. travis、確認など諸々通ったら、Mergeをお願いする

  5. tagづけをお願いする

    git tag 315.xx.yy && git push --tags # xx, yyは実際の番号
    

    をPRのコメントにつけてお願いする

以下はさらなるメモです

@snozawa
Copy link
Contributor Author

snozawa commented Oct 23, 2017

@fkanehiro さん

ありがとうございます
お手すきの時で構いませんので

git tag 315.15.0 && git push --tags

をお願いいたします。

@snozawa snozawa deleted the update_for_315_15_0 branch October 23, 2017 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants