tags 620786 + patch

On Mon, Apr 04, 2011 at 04:29:28AM +0200, Marco d'Itri wrote:
> Nowadays alsa-base depends on udev, so the code which tries to create
> the device nodes in postinst should be removed (also because many of the
> tests are wrong and obsolete).
> The /etc/apm/* stuff has to go as well, APM has been obsolete for years.

Patch attached for the /run transition.  This checks for /run/udev
in addition to /dev/.udev and /dev/.udevdb.

Note that this does not address Marco's bug above, that the obsolete
stuff needs to be removed.  It just makes it not-broken on current
systems.  Note that the existing udev checks were /already/ broken
because you never checked for /dev/.udev.

There has been no response to this for over nine months.  It's been
broken for that long since /run was introduced.  Please could you
apply the attached patch to make it work again, and also look at
removing the logic entirely.


Thanks,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux             http://people.debian.org/~rleigh/
 `. `'   Printing on GNU/Linux?       http://gutenprint.sourceforge.net/
   `-    GPG Public Key: 0x25BFB848   Please GPG sign your mail.
diff -urN alsa-driver-1.0.23+dfsg.orig/debian/alsa-base.postinst alsa-driver-1.0.23+dfsg/debian/alsa-base.postinst
--- alsa-driver-1.0.23+dfsg.orig/debian/alsa-base.postinst	2012-01-19 10:20:17.024064610 +0000
+++ alsa-driver-1.0.23+dfsg/debian/alsa-base.postinst	2012-01-19 10:25:56.613352939 +0000
@@ -12,7 +12,7 @@
 
 udev_is_active()
 {
-	test -e /dev/.udev.tdb || test -d /dev/.udevdb || return 1
+	test -d /run/udev || test -d /dev/.udev || test -d /dev/.udevdb || return 1
 	return 0
 }
 

Reply via email to