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

Remove no need grepping with the default CATKIN_TOOLS_BUILD_OPTIONS #289

Merged
merged 1 commit into from
Aug 3, 2016

Conversation

wkentaro
Copy link
Member

@wkentaro wkentaro commented Aug 2, 2016

The default option is --summarize --no-status so there is no
Symlinking.. output, so we can remove this line.

The default option is `--summarize --no-status` so there is no
`Symlinking..` output, so we can remove this line.
@k-okada
Copy link
Member

k-okada commented Aug 3, 2016

Great!

memo:
https://github.com/jsk-ros-pkg/jsk_travis/blob/master/travis.sh#L61

if [ ! "$CATKIN_TOOLS_BUILD_OPTIONS" ]; then export CATKIN_TOOLS_BUILD_OPTIONS="--summarize --no-status"; fi

@k-okada k-okada merged commit 8776f2d into jsk-ros-pkg:master Aug 3, 2016
@wkentaro wkentaro deleted the remove-build-log-grep branch August 6, 2017 09:07
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