On Thu, Feb 23, 2012 at 10:26:39AM +0100, frantisek holop wrote: > hmm, on Thu, Feb 23, 2012 at 07:54:41AM +0100, Antoine Jacoutot said that > > On Wed, Feb 22, 2012 at 11:26:27PM +0100, frantisek holop wrote: > > > hi there, > > > > > > please find attached my update for tinyproxy. > > > > > > -the license is now GPLv2 (was v3) > > > > > > -configuration moved upstream from /etc/tinyproxy/ > > > to /etc/tinyproxy.conf > > > > > > i hope it's not a big problem that it's not in patch > > > format... > > > > > > thanks to Federico G. Schwindt for prodding and something > > > to compare to. i think that creating /var/run/tinyproxy > > > is better at install time, and not in the rc script > > > because otherwise running the daemon manually fails. > > > > No that is wrong. /var/run is removed at every reboot, so please keep the > > rc script the way it was. > > ok. can it stay in the package as well for the reason above? > i see no harm in it..
No. If you need to start something manually then you are supposed to do all required steps manually. And what happens when you pkg_add your stuff, reboot then try to start it manually? You will _have_ to recreate manually /var/run/... anyway. So please remove this from the pkg. -- Antoine
