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

supress the output of run_tests #386

Merged
merged 4 commits into from
Oct 23, 2019
Merged

Conversation

k-okada
Copy link
Member

@k-okada k-okada commented Oct 22, 2019

@pazeshun
Copy link
Collaborator

@k-okada Thank you. I'll try to use this.

pazeshun added a commit to pazeshun/jsk_apc that referenced this pull request Oct 22, 2019
@pazeshun
Copy link
Collaborator

@k-okada I think this PR includes typos.
Please check k-okada#1

@pazeshun
Copy link
Collaborator

With this PR and k-okada#1, travis test of start-jsk/jsk_apc#2697 passed thanks to short log.

For comparison, I extracted logs about testing jsk_apc2016_common.
Logs before this PR (extracted from https://api.travis-ci.org/v3/job/601132003/log.txt):
https://gist.github.com/pazeshun/3385e4dfd6ead8d1b4372c1472572fce
Logs after this PR (extracted from https://api.travis-ci.org/v3/job/601276340/log.txt):
https://gist.github.com/pazeshun/57c7446dee307077842ce98b2f0fe66b

I found that some logs are missing in new log.
For example,

[jsk_apc2016_common:make] /usr/bin/cmake -H/home/travis/ros/ws_jsk_apc/src/jsk_apc/jsk_apc2016_common -B/home/travis/ros/ws_jsk_apc/build/jsk_apc2016_common --check-build-system CMakeFiles/Makefile.cmake 0
[jsk_apc2016_common:make] make[1]: Entering directory `/home/travis/ros/ws_jsk_apc/build/jsk_apc2016_common'
[jsk_apc2016_common:make] Dependee "/home/travis/ros/ws_jsk_apc/build/jsk_apc2016_common/CMakeFiles/clean_test_results_jsk_apc2016_common.dir/DependInfo.cmake" is newer than depender "/home/travis/ros/ws_jsk_apc/build/jsk_apc2016_common/CMakeFiles/clean_test_results_jsk_apc2016_common.dir/depend.internal".

catkin Make Args should only be run_tests, as --catkin-make-args stops with -- just after run_tests in moveit/moveit_ci#18.
@pazeshun
Copy link
Collaborator

@k-okada Could you merge and release this?

@k-okada k-okada merged commit 7786bf4 into jsk-ros-pkg:master Oct 23, 2019
@k-okada k-okada deleted the remove_message branch October 23, 2019 10:47
@k-okada
Copy link
Member Author

k-okada commented Oct 23, 2019

released 0.5.4

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