On Sun, Nov 16, 2014 at 11:46:06AM +0100, intrigeri wrote:
> Hi,
> 
> Cameron Norman wrote (16 Nov 2014 03:01:32 GMT) :
> > diff --git a/debian/ekeyd.postrm b/debian/ekeyd.postrm
> > index 484db5c..4efc368 100644
> > --- a/debian/ekeyd.postrm
> > +++ b/debian/ekeyd.postrm
> > @@ -1,9 +1,5 @@
> >  #!/bin/sh -e
> >  
> > -if test -x /sbin/udevcontrol; then
> > -    udevcontrol --reload_rules 2>/dev/null || udevcontrol reload_rules 
> > 2>/dev/null
> > -elif test -x /sbin/udevadm; then
> > -    udevadm control --reload-rules 2>/dev/null || udevadm control 
> > --reload_rules 2>/dev/null
> > -fi
> > +invoke-rc.d --quiet udev reload || true
> >  
> >  #DEBHELPER#
> 
> IIRC, nowadays udev rules changes are picked up on new udev events
> anyway. So, it may be that simply removing ekeyd.postrm would do
> the job.
> 
> But anyway, a smaller fix may be better suited for a freeze exception:
> simply adding "|| true" at the end of each udevadm or udevcontrol
> pipeline should be enough. What do you think?

I'd rather have Cameron's patch. The whole udevadm thing is a bug in
itself.

-- 
Jonathan Wiltshire                                      j...@debian.org
Debian Developer                         http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

Attachment: signature.asc
Description: Digital signature

Reply via email to