https://bugzilla.redhat.com/show_bug.cgi?id=839527 Peter Lemenkov <lemenkov@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(brendan.jones.it@ | |gmail.com) --- Comment #7 from Peter Lemenkov <lemenkov@xxxxxxxxx> --- (In reply to comment #5) > (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? ^^^ Brendan, ping! -- You are receiving this mail because: You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review