https://bugzilla.redhat.com/show_bug.cgi?id=839527

--- Comment #5 from Peter Lemenkov <lemen...@gmail.com> ---
(In reply to comment #3)
> (In reply to comment #2)
> > * I really don't like the idea to introduce another SysVinit-script. It
> > won't be easy to switch to systemd later. You may use the following approach
> > which is used in libvirt-clients
> 
> I thought long and hard about this whilst I was preparing this for review
> but am hesitant to do this - I'm sure Rui upstream will get there eventually
> and I'd prefer to leave it up to him (he's started packaging for Suse so I'm
> sure he'll get there).  As this is moving from CCRMA I'm also very cognizant
> of shifting the ground under the feet of current users.

Ok, understood. I won't insist here.

> > * Installation path for udev rules is wrong. In fact you should install them
> > into /usr/lib/udev/rules.d unless you're installing them on EL5. Since EL6
> > and Fedora 12 (afaik) /etc/udev/rules.d is intended for udev-rules override
> > by the sysadmin.

How about this one? ^^^

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to