On Tue, 29.01.13 00:36, Sérgio Basto (sergio@xxxxxxxxxx) 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@xxxxxxxxxx> 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.
--
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/devel
