Comment 2 for bug 1036172

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Security review:
- No CVE history
- Simple python library with no binaries, initscripts/upstart jobs, dbus services, use of privileged commands or cron jobs
- packaging is reasonable
- builds fine with packages only in main
- no testsuite. It hints at being able to do: 'python setup.py test', but it doesn't work (needs a patch to setup.py to find py.test and also tries to run it against test_tablib.py, which is not present in the source
- after a high level review, the code seems fine and maintainable

ACK