On Tue, 29.01.13 00:36, Sérgio Basto (sergio(a)serjux.com) wrote:
On Ter, 2013-01-29 at 00:50 +0100, Kay Sievers wrote:
> On Sat, Jan 26, 2013 at 6:14 AM, Sérgio Basto <sergio(a)serjux.com> wrote:
> > how I should do the old command udevadm: control --reload-rules ?
> > is correct : systemctl restart udev.service ?
> > I don't find any in documentation that guarantee this .
> >
> > BTW in F18: udev.service change the name to systemd-udevd.service.
> > So put in package spec, is this correct way ?
> >
> > %if 0%{?fedora} < 18
> > systemctl restart udev.service
> > systemctl restart udev-trigger.service
> > systemctl restart udev-settle.service
> > %else
> > systemctl restart systemd-udevd.service
> > systemctl restart systemd-udev-trigger.service
> > systemctl restart systemd-udev-settle.service
> > %endif
>
> Please get rid of all of that, none of it is necessary, udev will
> notice that on its own.
>
> It is completely wrong to ever do that and to restart udev or other
> essential services from packages. No package besides udev itself is
> allowed to do restart these services.
Hi,
Even when package adds some rules to /etc/udev/rules.d ?
Yes, even then. udev will notice rules dropped there.
like :
cat /etc/udev/rules.d/90-vboxdrv.rules
SUBSYSTEM=="usb_device", ACTION=="add",
RUN+="VBoxCreateUSBNode.sh
$major $minor $attr{bDeviceClass} vboxusers"
SUBSYSTEM=="usb", ACTION=="add",
ENV{DEVTYPE}=="usb_device", RUN
+="VBoxCreateUSBNode.sh $major $minor $attr{bDeviceClass} vboxusers"
SUBSYSTEM=="usb_device", ACTION=="remove",
RUN+="VBoxCreateUSBNode.sh
--remove $major $minor"
SUBSYSTEM=="usb", ACTION=="remove",
ENV{DEVTYPE}=="usb_device", RUN
+="VBoxCreateUSBNode.sh --remove $major $minor"
Ahum, you are aware that this will run your script for any USB device
plugged in? Which Fedora package includes this? This code really looks
really not right.
Lennart
--
Lennart Poettering - Red Hat, Inc.