Index | Thread | Search

From:
Edd Barrett <edd@theunixzoo.co.uk>
Subject:
Re: net/syncthing: fix hard-coded paths in patch
To:
Klemens Nanni <kn@openbsd.org>
Cc:
ports@openbsd.org
Date:
Thu, 1 Jan 2026 23:53:09 +0000

Download raw body.

Thread
Hi,

On Thu, Jan 01, 2026 at 11:47:15PM +0000, Klemens Nanni wrote:
> > -+	if err := unix.Unveil("/usr/local/bin/syncthing", "rx"); err != nil {
> > ++	if err := unix.Unveil("${PREFIX}/bin/syncthing", "rx"); err != nil {
> 
> This would be right,
> 
> >  +		panic(err)
> >  +	}
> >  +
> > -+	if err := unix.Unveil("/usr/local/bin/xdg-open", "rx"); err != nil {
> > ++	if err := unix.Unveil("${LOCALBASE}/bin/xdg-open", "rx"); err != nil {
> 
> this, however should be LOCALBASE, see bsd.port.mk(5).

Read again. It is LOCALBASE :)

> That goes to show that you're off the rails and on your own
> as soon as you build ports with anything other than /usr/local/.

I'd be amazed if anyone's ever changed their install location too.

Cheers

-- 
Best Regards
Edd Barrett

https://www.theunixzoo.co.uk