ament/ament_tools — pull request #9 by dirk-thomas (2014-09-10 23:09:21+00:00)
Title: Random stuff

@tfoote @wjwwood Please review.

I am not yet happy with the last commit (at least it got me to run all tests for now). The duplication between the `build*` and `test*` verbs is everything but good. I am not convinced that it is a good idea that `test` just invokes `build`.

I think I will refactor the `build` verbs to not know anything about testing stuff. Based on that the `test` verbs should either only call sub-functions of the `build` verbs or the `build` API must provide hooks / callbacks which enable the `test` verbs to extend the behavior.

What do you think about it?