Ubuntu
ubiquity package

Drop oem-config hal dependency

Bug #493139 reported by Martin Pitt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
Fix Released
High
Martin Pitt
Lucid
Fix Released
High
Martin Pitt

Bug Description

Binary package hint: ubiquity

When X.org stops depending on hal (which should happen really soon, probably next week), oem-config is the only remaining package on a standard Ubuntu CD which pulls in hal. We want to get rid of hal, since it's not being maintained any more, obsolete, and also increases boot speed. (See https://wiki.ubuntu.com/Halsectomy)

I had a quick look through the source, and the only place that uses hal seems to be scripts/ubi-restart-hal which currently does "/etc/init.d/hal restart" to re-read keyboard layouts.

For achieving the same effect in a hal-free world (after X.org moved from hal to udev), the same can be achieved using

  udevadm trigger --action=change --property-match=ID_INPUT_KEY=1

The package should drop the hal dependency, and the script should do above, and "restart hal || true" for now.

Related branches

lp:~pitti/ubiquity/halsectomy
Ubuntu Installer Team: Pending requested
Diff: 61 lines (+20/-5)
4 files modified
debian/changelog (+12/-0)
debian/control (+1/-1)
debian/oem-config.install (+1/-1)
scripts/ubi-reload-keyboard (+6/-3)
lp:ubuntu/lucid/ubiquity
Martin Pitt (pitti)
Changed in ubiquity (Ubuntu Lucid):
importance: Undecided → High
Revision history for this message
Martin Pitt (pitti) wrote :

Fixed in my halsectomy branch (can't push to ~ubuntu-installer directly):

  https://code.launchpad.net/~pitti/ubiquity/halsectomy

Changed in ubiquity (Ubuntu Lucid):
assignee: nobody → Martin Pitt (pitti)
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubiquity - 2.1.4

---------------
ubiquity (2.1.4) lucid; urgency=low

  [ Mario Limonciello ]
  * Correct a variable that didn't get changed from pychecker
    cleanup in the intro plugin (LP: #492873).

  [ Martin Pitt ]
  * scripts/ubi-restart-hal:
    - Restart hal the upstart way.
    - Do not fail if hal does not exist, and drop hal dependency.
      (LP: #493139)
    - Also trigger an udev change event for key devices, so that the changed
      keyboard settings also apply to an udevified X.org server.
  * scripts/ubi-restart-hal: Rename to scripts/ubi-reload-keyboard to state
    the intent, not the means (since it now primarily applies to udev).
    Adjust debian/oem-config.install accordingly.

  [ Colin Watson ]
  * Automatic update of included source packages: base-installer
    1.103ubuntu2, preseed 1.43.
 -- Colin Watson <email address hidden> Tue, 08 Dec 2009 11:58:27 +0000

Changed in ubiquity (Ubuntu Lucid):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.