Comment 54 for bug 858122

Revision history for this message
Evan Broder (broder) wrote :

Hi Adam -

Thanks for working on getting that patch together. I've gone ahead and sponsored it into precise, and will be working on an SRU for oneiric momentarily. For future reference, I made a few tweaks before uploading:

 1. The insserv package is source format 3.0 (quilt), so it uses quilt to manage patches (http://developer.ubuntu.com/packaging/html/patches-to-packages.html#patches-with-quilt). I created a quilt patch containing your change. We also try to apply the DEP-3 guidelines in distro patches so we can track their provenance down the road (http://dep.debian.net/deps/dep3/)

 2. Your changelog entry targets "unstable" - Ubuntu changes should always refer to the Ubuntu release they're targeting; in this case, that would be "precise"

 3. When Ubuntu changes a Debian package, our policy is to adjust the maintainer field in debian/control - the update-maintainer script in ubuntu-dev-tools will do this for you automatically. It's also our policy to adjust the Vcs-* fields to be XSBC-Debian-Vcs-*. update-maintainer doesn't currently do this, so you have to do it yourself (bug #567629)

I'm also working on a change to initscripts to try and clean up some of the fallout of this bug for people who have hit it; hopefully that should land later today.

Joonas: If you have not yet installed VMware Workstation 8, just removing /usr/sbin/insserv should be sufficient (nothing on an Ubuntu system should actually use it). If you have, run the series of "mv" commands in the bug's description