[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#705599: finish-install: 07speakup needs to be updated for Wheezy and GNOME 3



On Thu, 18 Apr 2013, Samuel Thibault wrote:
> So in the end, it would rather be the attached patch, which also enables
> the screen reader.

Honestly, I prefer my patch over yours:

1/ you're setting up dbus within d-i instead of letting it happen during
   the first boot like on other installation scenarios

2/ the multiple dbus-launch means that you start multiple dbus that are
   left around during the installation (this goes against policy-rc.d
   which disables daemons on purpose)

3/ you change directly the user configuration instead of adjusting
   the defaults (this one is debatable since supplementary users might
   not want a11y enabled by default but this is a minor inconvenience)

4/ if the system is reconfigured so that gsettings uses something else
   than dconf as backend, your configuration work is useless whereas
   with my approach it's not since it's handled in a higer layer (at the
   gsettings level)

Cheers,
-- 
Raphaël Hertzog ◈ Debian Developer

Get the Debian Administrator's Handbook:
→ http://debian-handbook.info/get/


Reply to: