Code review comment for lp:~canonical-platform-qa/url-dispatcher/fake_dispatcher

Revision history for this message
Ted Gould (ted) wrote :

Some discussion on IRC putting here for tracking:

* The tests need to be run by CTest not by debian/rules
* We shouldn't override autotest here to run it under dbus, that should be in the individual test
* The package shouldn't have a build-dep on a binary it produces

review: Needs Fixing

« Back to merge proposal