jsk-ros-pkg / jsk_recognition

JSK perception ROS packages
https://github.com/jsk-ros-pkg/jsk_recognition
272 stars 189 forks source link

add groovy/catkin build test #11

Closed garaemon closed 10 years ago

garaemon commented 10 years ago

.travis.ymlにすべて直接書くか、それぞれを.shにしてみるか?

それなりにif文での分岐が必要になる

k-okada commented 10 years ago

後,何処かに.shを置いてwgetして実行,ってパタンもありますよね. ただ,.travisのinstall,before_script, script,みたいなものの使い分けがよくわかっていないです.

garaemon commented 10 years ago

ドキュメントはここですね

:scriptがすべて0で終了するのが大事みたいですね. それ以外は慣習的に用意しているように見えます

garaemon commented 10 years ago

:before_installが走る sudo apt-get update的なもの

:installが走る buildに必要なものをインストールする

:before_script ビルドに備えてやること。DBを設定したり、環境変数をセットしたり.

:script すべて0で終わるのが大事

:after_success:after_failureが走る

:after_scriptが走る

garaemon commented 10 years ago

後,何処かに.shを置いてwgetして実行,ってパタンもありますよね.

できるかぎり、スクリプトは同じレポジトリに入れた方が良いと思っています。 その結果、コピペが増えてしまっても、それぞれで微妙な調整が必要だったり、外部のshスクリプトの バージョンで結果が変わるという方が扱いが面倒なのではないかな、と予想してます。

しばらくしてノウハウがたまったら外部に置いたりして共通化を図ることはできるかなと思っています

garaemon commented 10 years ago

3つのスクリプトに分けてみました

https://github.com/garaemon/jsk_recognition/tree/add-rosbuild-id-11/travis

https://travis-ci.org/garaemon/jsk_recognition/builds/20403086

k-okada commented 10 years ago

I also working catkin/rosbuild check on here -> https://github.com/start-jsk/openrtm_aist_core/pull/29

garaemon commented 10 years ago

Now we are almost ready to write general travis.yml for rosbuild/catkin and wstool/rosws .

the requirements are:

about catkin/rosbuild:

about rosws/wstool

about hydro/groovy

garaemon commented 10 years ago

which is better to use several sh scripts or just travis.yml.

sh scripts strategy

travis.yml strategy

garaemon commented 10 years ago

Build matrix we want to test is:

garaemon commented 10 years ago

done https://github.com/jsk-ros-pkg/jsk_recognition/pull/13