On 2019/02/05 23:44, Alex Holst wrote:
> Hey Edd. Thanks for working on this. Comments below.
> 
> Quoting Edd Barrett ([email protected]):
> > RCS file: /cvs/ports/net/syncthing/pkg/PLIST,v
> > retrieving revision 1.4
> > diff -u -p -r1.4 PLIST
> > --- pkg/PLIST       4 Sep 2018 12:46:19 -0000       1.4
> > +++ pkg/PLIST       5 Feb 2019 22:07:02 -0000
> > @@ -1,6 +1,13 @@
> >  @comment $OpenBSD: PLIST,v 1.4 2018/09/04 12:46:19 espie Exp $
> >  @newgroup _syncthing:768
> >  @newuser _syncthing:768:_syncthing:daemon:Syncthing 
> > user:${VARBASE}/syncthing:/sbin/nologin
> > +@rcscript ${RCDIR}/syncthing
> > +@owner _syncthing
> > +@group _syncthing
> > +@sample ${VARBASE}/syncthing/
> > +@extraunexec rm -rf ${VARBASE}/syncthing/{.,}*
> > +@owner
> > +@group
> 
> I am pretty sure those blank @owner and @group need to go.

No, you don't want the following files (binaries, manuals etc) owned by
_syncthing.

> I have also been running with a diff that includes the stcli command in
> the package, which I need for most of my remote Syncthing systems.
> 
> I'll follow up with that diff.
> 

Reply via email to