Re: add pledge to devel/cvsweb

2019-10-01 Thread Ingo Schwarze
Hi Solene, Solene Rapenne wrote on Tue, Oct 01, 2019 at 02:15:02PM +0200: > I don't really want to work more on cvsweb, Fair enough, you get to choose what you want to work on. :-) > I did this because it's a cgi scripts run on official > cvsweb.openbsd.org website and I wanted to help there.

Re: add pledge to devel/cvsweb

2019-10-01 Thread Solene Rapenne
On Sun, Sep 29, 2019 at 07:37:26PM +0200, Ingo Schwarze wrote: > Hi Solene, > > Solene Rapenne wrote on Thu, Sep 26, 2019 at 05:27:08PM +0200: > > > Hi, now that we have OpenBSD::pledge I thought it would be nice to use > > it in devel/cvsweb > > I think this is a thoroughly bad idea. > >

Re: add pledge to devel/cvsweb

2019-09-29 Thread Ingo Schwarze
Hi Solene, Solene Rapenne wrote on Thu, Sep 26, 2019 at 05:27:08PM +0200: > Hi, now that we have OpenBSD::pledge I thought it would be nice to use > it in devel/cvsweb I think this is a thoroughly bad idea. Pledge is useful for well-understood high-quality code. But CVSweb, at this point, is

Re: add pledge to devel/cvsweb

2019-09-28 Thread Solene Rapenne
On Fri, Sep 27, 2019 at 06:53:56PM -0700, Andrew Hewus Fresh wrote: > On Fri, Sep 27, 2019 at 09:28:39AM +0200, Solene Rapenne wrote: > > On Thu, Sep 26, 2019 at 05:40:38PM +0200, Otto Moerbeek wrote: > > > On Thu, Sep 26, 2019 at 05:27:08PM +0200, Solene Rapenne wrote: > > > > > > > Hi, now that

Re: add pledge to devel/cvsweb

2019-09-27 Thread Andrew Hewus Fresh
On Fri, Sep 27, 2019 at 09:28:39AM +0200, Solene Rapenne wrote: > On Thu, Sep 26, 2019 at 05:40:38PM +0200, Otto Moerbeek wrote: > > On Thu, Sep 26, 2019 at 05:27:08PM +0200, Solene Rapenne wrote: > > > > > Hi, now that we have OpenBSD::pledge I thought it would be nice to use > > > it in

Re: add pledge to devel/cvsweb

2019-09-27 Thread Andrew Hewus Fresh
On Fri, Sep 27, 2019 at 09:28:39AM +0200, Solene Rapenne wrote: > On Thu, Sep 26, 2019 at 05:40:38PM +0200, Otto Moerbeek wrote: > > On Thu, Sep 26, 2019 at 05:27:08PM +0200, Solene Rapenne wrote: > > > > > Hi, now that we have OpenBSD::pledge I thought it would be nice to use > > > it in

Re: add pledge to devel/cvsweb

2019-09-27 Thread Florian Obser
On Fri, Sep 27, 2019 at 09:28:39AM +0200, Solene Rapenne wrote: > On Thu, Sep 26, 2019 at 05:40:38PM +0200, Otto Moerbeek wrote: > > On Thu, Sep 26, 2019 at 05:27:08PM +0200, Solene Rapenne wrote: > > > > > Hi, now that we have OpenBSD::pledge I thought it would be nice to use > > > it in

Re: add pledge to devel/cvsweb

2019-09-27 Thread Sebastien Marie
On Fri, Sep 27, 2019 at 10:10:29AM +0200, Solene Rapenne wrote: > On Fri, Sep 27, 2019 at 10:02:35AM +0200, Sebastien Marie wrote: > > > > > Index: patches/patch-cvsweb_cgi > > > === > > > RCS file:

Re: add pledge to devel/cvsweb

2019-09-27 Thread Solene Rapenne
On Fri, Sep 27, 2019 at 10:02:35AM +0200, Sebastien Marie wrote: > Just two comments. > > > Index: patches/patch-cvsweb_cgi > > === > > RCS file: /data/cvs/ports/devel/cvsweb/patches/patch-cvsweb_cgi,v > > retrieving revision 1.13 >

Re: add pledge to devel/cvsweb

2019-09-27 Thread Sebastien Marie
Just two comments. > Index: patches/patch-cvsweb_cgi > === > RCS file: /data/cvs/ports/devel/cvsweb/patches/patch-cvsweb_cgi,v > retrieving revision 1.13 > diff -u -p -r1.13 patch-cvsweb_cgi > --- patches/patch-cvsweb_cgi 7 Apr 2013

Re: add pledge to devel/cvsweb

2019-09-27 Thread Solene Rapenne
On Thu, Sep 26, 2019 at 05:40:38PM +0200, Otto Moerbeek wrote: > On Thu, Sep 26, 2019 at 05:27:08PM +0200, Solene Rapenne wrote: > > > Hi, now that we have OpenBSD::pledge I thought it would be nice to use > > it in devel/cvsweb > > > > I've been able to tight it to "rpath proc exec prot_exec",

Re: add pledge to devel/cvsweb

2019-09-26 Thread Otto Moerbeek
On Thu, Sep 26, 2019 at 05:27:08PM +0200, Solene Rapenne wrote: > Hi, now that we have OpenBSD::pledge I thought it would be nice to use > it in devel/cvsweb > > I've been able to tight it to "rpath proc exec prot_exec", removing > wpath and cpath was possible by commenting lines piping STDERROR

add pledge to devel/cvsweb

2019-09-26 Thread Solene Rapenne
Hi, now that we have OpenBSD::pledge I thought it would be nice to use it in devel/cvsweb I've been able to tight it to "rpath proc exec prot_exec", removing wpath and cpath was possible by commenting lines piping STDERROR to /dev/null, that doesn't mean creating dev/null is not required anymore,