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

Use merged devel space on testing #288

Closed
wants to merge 1 commit into from

Conversation

wkentaro
Copy link
Member

@wkentaro wkentaro commented Aug 1, 2016

This is required to run this script on Travis.

This is required to run this script on Travis.
@wkentaro
Copy link
Member Author

wkentaro commented Aug 1, 2016

@k-okada What do you think about this?

@k-okada
Copy link
Member

k-okada commented Aug 1, 2016

is there any test case which did not pass with isolated/linked devel space?
if so, since merged devel space is not a default settings, people using default setting may have trouble on compile, is it correct?

@wkentaro
Copy link
Member Author

wkentaro commented Aug 1, 2016

is there any test case which did not pass with isolated/linked devel space?

The test fails because of timeout on travis at the catkin build | grep -v Sym....

@k-okada
Copy link
Member

k-okada commented Aug 2, 2016

timeout means, "no output in the last 10 min..."?

◉ Kei Okada

On Mon, Aug 1, 2016 at 11:25 PM, Kentaro Wada notifications@github.com
wrote:

is there any test case which did not pass with isolated/linked devel space?

The test fails because of timeout on travis at the catkin build | grep -v
Sym....


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#288 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeG3LDdAcJHhh7ddVoVWvFwaWvgE2qAks5qbgHXgaJpZM4JZf1q
.

@wkentaro
Copy link
Member Author

wkentaro commented Aug 2, 2016

Yeah.
I'm trying to run testing on Travis for jsk_recognition packages, but it fails because of timeout of 10min for no outputs.
https://travis-ci.org/jsk-ros-pkg/jsk_recognition/builds/148814476

@k-okada
Copy link
Member

k-okada commented Aug 2, 2016

how about

while [ t ]; do echo "alive!"; sleep 120; done & pid=$!; catkin b; kill $pid

@wkentaro
Copy link
Member Author

wkentaro commented Aug 2, 2016

Where should I run it?
in travis.sh?

@k-okada
Copy link
Member

k-okada commented Aug 2, 2016

Yes, but not so much closer confident... Just an idea

2016年8月2日火曜日、Kentaro Wadanotifications@github.comさんは書きました:

Where should I run it?
in travis.sh?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#288 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeG3Iv-KVSzHH5mkYm16b4u8zzcLgmnks5qbzFHgaJpZM4JZf1q
.

◉ Kei Okada

@wkentaro
Copy link
Member Author

wkentaro commented Aug 2, 2016

Hmm. Is it the high priority to use the default configuration of catkin-tools?
I'm not sure why I can't use merge-devel configuration with consideration for people who are not interested in this kind of thing.

@k-okada
Copy link
Member

k-okada commented Aug 3, 2016

i'm afraid we could not say "it should work" with high confidence, when
some one reported compilation error.

◉ Kei Okada

On Tue, Aug 2, 2016 at 10:50 PM, Kentaro Wada notifications@github.com
wrote:

Hmm. Is it the high priority to use the default configuration of
catkin-tools?
I'm not sure why I can't use merge-devel configuration with consideration
for people who are not interested in this kind of thing.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#288 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeG3FqfZilbgATtk7VyBT25vfLYkfC0ks5qb0sOgaJpZM4JZf1q
.

@wkentaro
Copy link
Member Author

wkentaro commented Aug 3, 2016

I found the solution(?) so please merge this > #289.

@wkentaro
Copy link
Member Author

wkentaro commented Aug 3, 2016

Fixed via #289

@wkentaro wkentaro closed this Aug 3, 2016
@wkentaro wkentaro deleted the merge-devel branch August 3, 2016 08:16
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