Skip to content

skip test directory when building tests is not requested#111

Merged
scpeters merged 2 commits intoros:masterfrom
ros2:fix_linker_warning_upstream
Apr 19, 2018
Merged

skip test directory when building tests is not requested#111
scpeters merged 2 commits intoros:masterfrom
ros2:fix_linker_warning_upstream

Conversation

@dirk-thomas
Copy link
Copy Markdown
Member

Same as ros2#10.

@scpeters
Copy link
Copy Markdown
Contributor

Currently, BUILD_TESTING is not defined. Do you mind if we add a line to include(CTest) before checking the variable?

@dirk-thomas
Copy link
Copy Markdown
Member Author

Sounds fine with me: c092053.

@scpeters scpeters merged commit 52c200d into ros:master Apr 19, 2018
@dirk-thomas dirk-thomas deleted the fix_linker_warning_upstream branch April 19, 2018 18:50
dirk-thomas added a commit to ros2/urdfdom that referenced this pull request Apr 19, 2018
* skip test directory when building tests is not requested

* include CTest
dirk-thomas added a commit to ros2/urdfdom that referenced this pull request May 1, 2018
* skip test directory when building tests is not requested

* include CTest
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.

3 participants