[Bug 683463] Review Request: trafficserver - Apache Traffic Server

[bugzilla at redhat.com]

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 at tanso.net> 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.