On Thu, Aug 15, 2024 at 10:57:09AM GMT, Stuart Henderson wrote: > On 2024/08/15 09:52, Omar Polo wrote: > > On 2024/08/15 00:57:31 +0100, Stuart Henderson <s...@spacehopper.org> wrote: > > > I'd use pretty much any other old uid than _nginx as that's one of the > > > more > > > common ports to install. Just looking at spare numbers showing in the > > > diff, > > > 594/596 are way less common so fewer people will need to mess about with > > > rmuser. > > > > _nginx is actually the oldest one, Feb 2012, followed by apache-http in > > Jul 2015. > > yes, exactly - avoided reusing earlier because those were common and > the user interface for a uid conflict is a little clunky (you need to > install to see the error, uninstall, remove user/group, reinstall). > > > But I get your point, I was a bit worried too. I'll get 665 > > then, used by audio/hgd and removed in august 2016.
Updated to use 665.
>
> fine with me.
>
> comments inline:
>
> : # GPLv2
>
> "either version 2 of the License, or (at your option) any later
> version." -> GPLv2+
Done.
>
> : - pidfile = "var/run/hopm.pid";
> : + pidfile = "/var/run/hopm/hopm.pid";
> ...
> : -# scanlog = "var/log/scan.log";
> : +# scanlog = "/var/log/hopm/scan.log";
> ...
> : @sample ${LOCALSTATEDIR}/log/hopm/
> : @sample ${LOCALSTATEDIR}/run/hopm/
>
> if you use /var in the patch, use /var in the PLIST (and I think that
> is better than LOCALSTATEDIR here)
Changed to use /var. Removed sample for run.
>
> /var/run is cleared at boot, this must be created in the rc script
Created in rc_pre()
>
> otherwise ok
>
--
Chaz
hopm.tgz
Description: application/gtar-compressed
