-
Notifications
You must be signed in to change notification settings - Fork 26
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
setup.py missing test_suite support #11
Comments
How do most other packages handle this? This warning is rather annoying:
|
I haven't found a very clear standard in the way that people are incorporating tests. There is a pure distutils approach that I've seen, but it looked a lot more complicated. So I've used this approach. Having setuptools installed makes the warning go away. |
Fyi, @eighthave is creating a python-pgpdump package in Debian, hence this ;) |
its done and already in Debian: http://packages.qa.debian.org/p/python-pgpdump.html :-D |
I'd like this to be merged as well, or at least something comparable to make it straightforward to run the tests in a standard way. |
The setup.py file does not include the test script. That is the standard interface for running the tests, and means that things that wrap setup.py will automatically run the tests. Such things include Debian packaging, pip, etc.
This patch adds test support while maintaining distutils compatibility:
The text was updated successfully, but these errors were encountered: