tags 689769 + pending patch
thanks
On Sun, 25 Nov 2012 21:26:26 +0100
Julien Cristau <jcris...@debian.org> wrote:
> > + if [ -d /var/run/ample ] ; then
> > + chown
> > ${DEFUSER:-nobody}:${DEFGROUP:-nogroup} /var/run/ample
> > + fi
>
> The whole point of nobody/nogroup is they don't own anything. This is
> broken.
I have uploaded ample 0.5.7-7 (QA upload because it’s effectively
orphaned) to DELAYED/5.It contains a patch to make ample drop privileges instead of using start-stop-daemon’s --chuid, so that ample can create the pidfile as root. Furthermore, I have changed the pidfile location to /var/run/ample.pid since ample does not require any other files in /var/run/. The debdiff is attached. Julien, I’d be thankful if you could review these changes and speak up in case I should correct anything. Thanks. -- Best regards, Michael
ample.debdiff
Description: Binary data
