[Merge] lp:~canonical-platform-qa/url-dispatcher/fake_dispatcher into lp:url-dispatcher
Brendan Donegan
brendan.donegan at canonical.com
Thu Jul 17 13:39:34 UTC 2014
> 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
All good points - CMake is proving very difficult to work with at the moment though. I updated the branch with some CMake changes, but they don't result in my python code being copied to build/tests/url_dispatcher_testability as I'd expect.
>From reading about FindPythonLibs, I think it's intended as being useful in the case where one wants to create C/C++ extensions for python. I don't think it helps in this situation - all I have is some python modules that I want to test and install.
--
https://code.launchpad.net/~canonical-platform-qa/url-dispatcher/fake_dispatcher/+merge/226829
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~canonical-platform-qa/url-dispatcher/fake_dispatcher into lp:url-dispatcher.
More information about the Ubuntu-reviews
mailing list