Bug#644319: mdadm: /run transition: Please update use of /dev/.udev

Roger Leigh rleigh at codelibre.net
Wed Jan 18 10:52:01 UTC 2012


tags 644319 + patch
thanks

On Tue, Oct 04, 2011 at 11:49:17PM +0200, biebl at debian.org wrote:
> During an automated test your package mdadm was flagged to reference
> files/directories matching /dev/.udev A log of this test can be found
> at [2].
> 
> In most cases checking for /dev/.udev is used to determine if udev is
> active. This check no longer works with udev using /run/udev now.
> 
> In most cases, checking for udev being installed is not appropriate
> and should be avoided.

I've attached a patch for /run/udev checks.

While this patch does not address the concern that mdadm should not
be checking for udev, it does correct the checks for /dev/.udev in
two source files.  It checks for both /run/udev and /dev/.udev so
it's backward compatible.  The /dev/.udev check can be dropped
post-wheezy.  Please forward upstream if such a change has not
already been merged.


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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mdadm-run-udev.patch
Type: text/x-diff
Size: 1050 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-mdadm-devel/attachments/20120118/0ab12621/attachment.patch>


More information about the pkg-mdadm-devel mailing list