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

.travis.yml: re-enable jade and hydro with released deb #375

Closed
wants to merge 2 commits into from

Conversation

k-okada
Copy link
Member

@k-okada k-okada commented Sep 27, 2015

#372 (comment)

We can say still jade is too early for us. Should we set jade allow_failures instead of ignoring these error?

Since jade is current "official" release and we already have roseus as "official" client program for jade, I think we have responsibility to keep roseus working on jade. So we should not put jade to allow_failures.

and I expect this would work, if we compare https://travis-ci.org/jsk-ros-pkg/jsk_roseus/jobs/81017815 and https://travis-ci.org/jsk-ros-pkg/jsk_roseus/jobs/81053071, woking version using euslisp, geneus from source, even though that uses USE_DEB=true, so someone broke jsk_travis

@k-okada
Copy link
Member Author

k-okada commented Sep 27, 2015

2d7d46a#commitcomment-13454838 is the problem

k-okada added a commit to k-okada/jsk_roseus that referenced this pull request Sep 27, 2015
@k-okada
Copy link
Member Author

k-okada commented Sep 27, 2015

updated docker image by

jenkins@jenkins:~/docker/ubuntu-14.04$ echo -e "FROM ros-ubuntu:14.04\nRUN apt-get update\nRUN apt-get -y upgrade\nEXPOSE 22" | sudo docker build -t ros-ubuntu:14.04 -

MEMO: on jade,
http://www.ros.org/debbuild/jade.html?q=geneus
release repo is not updated yet.

@garaemon
Copy link
Member

http://www.ros.org/debbuild/jade.html?q=geneus
release repo is not updated yet.

I see, maybe this causes error on jade.

@k-okada k-okada closed this Jun 27, 2018
@k-okada k-okada deleted the enable-jade branch June 27, 2018 12:53
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.

2 participants