Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=683463 --- Comment #13 from Jan-Frode Myklebust <janfrode@xxxxxxxxx> 2011-06-17 06:37:38 EDT --- Only thing I find strange currently is that you install the daemon binaries to /usr/bin instead of /usr/sbin/. Is that intentional ? Hmm.. I see the Gentoo layout used has defined "sbindir: ${exec_prefix}/sbin" -- so it's strange it's not putting anything there. Also, it would probably be nice to restart ts on upgrades in %postun: if [ $1 -eq 1 ] ; then /sbin/service trafficserver condrestart &>/dev/null || : fi I think it's starting to look quite good, and I think it would be great to get it into the package db as soon as possible to ease builds and also get more testers. Unfortunately I'm just an EPEL packager, don't know if my reviews counts for anything. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- 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