Index | Thread | Search

From:
Stuart Henderson <stu@spacehopper.org>
Subject:
Re: misc/llama.cpp: add rc script for llama-server
To:
ports@openbsd.org
Date:
Mon, 10 Nov 2025 19:18:37 +0000

Download raw body.

Thread
On 2025/11/10 19:27, Kirill A. Korinsky wrote:
> On Mon, 10 Nov 2025 18:56:23 +0100,
> Stuart Henderson <stu@spacehopper.org> wrote:
> > 
> > i'm not super happy to keep burning the limited resource of static uids
> > for things which like maybe 1 or 2 people will use. (finding an
> > additional range in the not too high digits above 1000 has been tricky,
> > and I think there was some objection about large sparse files if using
> > much higher numbers)..
> > 
> > not sure of a way forward on that though.
> >
> 
> I need a user for login class :(

rc.d uses su -c to set the login class based on the daemon name so you
don't actually need a user for that. but you do need a user for
ownership of the dir and there isn't currently a good choice other
than adding as a fixed uid. so I suppose that's ok, but we really
need a better way to handle these (i.e. dynamically adding).

> > other than that, there's a discrepancy between these two:
> > 
> > > +@sample ${LOCALSTATEDIR}/db/llama-server/
> > 
> > > +     :setenv=LLAMA_CACHE=/var/db/llama-server:\
> > 
> 
> Probably don't understand. LOCALSTATEDIR is /var, isn't it?

the whole point of having a variable is for the extremely unlikely
case somebody might rebuild packages with it set to something
different.

so, in that case, all references within the package need to be
consistent with each other, either /var, or ${LOCALSTATEDIR}.