tags 622208 + patch
thanks

On Mon, Apr 11, 2011 at 03:28:19AM +0200, Marco d'Itri wrote:
> Package: udftools
> Version: 1.0.0b3-14
> Severity: important
> 
> Checking for /dev/.udev/ is not reliable due to /run.
> 
> I think this package should just depend on udev, it will probably allow
> you to remove a lot of other hacks.

I've attached a patch which checks for /run/udev in addition to
/dev/.udev, hence is backward-compatible.  Please remove the
/dev/.udev part after wheezy is released.

> devfs support needs to be removed as well, since Linux has not had devfs
> in a very long time.

Agreed.  It's been dead for a long time now.  This patch does not
address this additional problem, but it's only a two minute job
to remove it from deb/postinst, deb/config and debian/udftools.init.


Regards,
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 udftools-1.0.0b3.orig/debian/udftools.init udftools-1.0.0b3/debian/udftools.init
--- udftools-1.0.0b3.orig/debian/udftools.init	2012-01-18 10:32:29.409627118 +0000
+++ udftools-1.0.0b3/debian/udftools.init	2012-01-18 10:33:14.346212247 +0000
@@ -60,7 +60,7 @@
         fi
     fi
     if test -z "$UDEV"; then # User did not set UDEV, try auto-detection
-        if test -e /dev/.udev; then
+        if test -e /run/udev || test -e /dev/.udev; then
             UDEV=true
         else
             UDEV=false

Reply via email to