Comment 2 for bug 912861

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

txaws review:
 * no CVE history
 * no sudo fragments, privileged operations, dbus services, setuid binaries, initscripts or daemons
 * has a test suite and it is run in the build and will fail the build on error
 * build logs are clean
 * lots of binaries without man pages
 * build depends are all in main
 * while the package is only in Ubuntu, it is supported by the server team and with upstream employed by Canonical
 * aws-status: uses gnome-keyring, which is fine, but this doesn't work with Unity (gtk.StatusIcon needs to move to app indicators). There is no documentation. Should be fixed and documented or dropped. Could alternatively leave binary in universe, but it is unusable atm so I'm not sure of the benefit there.

Conditional ACK provided the following is fixed:
 * binaries have worthwhile man pages (--help is useful, but aws-status doesn't
   have it and at least txaws-discover is out of date)
 * aws-status be fixed and documented or dropped. Optionally put this binary in universe.