[FFe] Upgrade Computer Janitor to 2.0 (dbus edition)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
computer-janitor (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Binary package hint: computer-janitor
Computer Janitor 2.0 is a major refactoring that splits the core system changing functionality into a separate dbus process. This allows us to greatly simplify the gtk and cli clients, and even allow other clients to be easily written against the dbus API. The biggest user visible benefit is that the gtk client is now single threaded, eliminating all the hard-to-diagnose and fix hangs and thread related problems that have been reported. Also, the clients do not need to be run as root any more; the dbus daemon uses PolicyKit authentication whenever a system-changing action is needed. This means that users can run the clients in "query" mode to get useful information out of the system, without root access, if they aren't making any changes to the system.
The code has more tests now, and is generally much cleaner (which is good for a janitor! :). The application Python package is structured in such a way that e.g. a Qt client could be added fairly easily.
Related branches
- Martin Pitt: Approve
- Michael Vogt: Pending requested
- computer-janitor-hackers: Pending requested
-
Diff: 1273 lines (+514/-383)16 files modified.bzrignore (+1/-0)
computer-janitor (+15/-1)
computer-janitor-gtk (+4/-0)
computerjanitorapp/gtk/areyousure.py (+19/-5)
computerjanitorapp/gtk/ui.py (+12/-2)
computerjanitord/application.py (+7/-26)
computerjanitord/errors.py (+35/-7)
data/ComputerJanitor.ui (+1/-9)
debian/changelog (+33/-0)
po/POTFILES.in (+11/-11)
po/computerjanitor.pot (+61/-52)
po/es.po (+63/-54)
po/fi.po (+63/-54)
po/fr.po (+63/-54)
po/ja.po (+63/-54)
po/pl.po (+63/-54)
Oh, CJ 2.0 requires python-argparse 1.1 which has already gone through the FFE and MIR process, and is just waiting for a sponsor to upload. mvo has offered to do this.
Also, the gtk ui has been cleaned up so that the unused "Recommended" column is gone. That was a hard-coded no-op previously.