Comment 60 for bug 857153

Revision history for this message
In , Mgorse (mgorse) wrote :

Comment on attachment 581750
Updated patch.

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
Accessibility will not be enabled on a GNOME 3 system, unless the user manually sets the accessibility gconf key (used by GNOME 2 but not 3, and thus not generally set under GNOME 3).

Testing completed (on m-c, etc.):
Risk to taking this patch (and alternatives if risky):
The old behavior of checking gconf will be used if the dbus call does not succeed (ie, if AT-SPI2 is not installed), so I think that the risk is small.
An alternative approach would be to modify the start-up script to call dbus-send and set GNOME_ACCESSIBILITY=1 if the call indicates that accessibility is enabled, but this would likely affect start-up time more than applying the patch would, so applying the patch seems preferable.