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

[WIP][.travis.yml] use travis in all tests instead of jenkins. #974

Closed
wants to merge 1 commit into from

Conversation

mmurooka
Copy link
Member

related to #973 (comment).

@furushchev
Copy link
Member

we cannot use travis directly on kinetic, please consider to use USE_DOCKER=true. (refer: https://github.com/jsk-ros-pkg/jsk_travis/blob/master/.travis.yml#L51)
And I think it is simpler that declare USE_DOCKER=true globally for testing of all distros.

@furushchev
Copy link
Member

furushchev commented Oct 17, 2016

As far as I see what you are discussing on #973 (comment), when testing on docker on travis, we need to configure each environment variables to pass to docker ( https://github.com/jsk-ros-pkg/jsk_travis/blob/master/travis.sh#L78-L91 ) like jenkins unfortunately.

@mmurooka
Copy link
Member Author

travisの結果のメモ

hydroはtravisだと失敗.

indigoでは環境変数IS_EUSLISP_TRAVIS_TESTがわたってroseusのテストが走る.
https://travis-ci.org/start-jsk/rtmros_common/jobs/168202321

[hrpsys_ros_bridge:cmake] Execute rostest for euslisp unittest

jenkinsだとダメでtravisだとOKなので,#973 (comment) で合っていた.
ただ,50分で終わらない.

@mmurooka mmurooka changed the title [DO NOT MERGE YET][.travis.yml] use travis in all tests instead of jenkins. [WIP][.travis.yml] use travis in all tests instead of jenkins. Oct 18, 2016
@pazeshun
Copy link
Collaborator

@mmurooka
こちらについてですが、#1085 において、IS_EUSLISP_TRAVIS_TESTをjenkinsに渡すことで、euslispのテストがjenkinsでも走るようにしました。
もしこれで大丈夫でしたら、こちらのcloseをお願いします。

@mmurooka
Copy link
Member Author

ありがとうございます.

@mmurooka mmurooka closed this Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants