On Wed, Jun 08, 2022 at 11:45:52AM BST, Aisha Tammy wrote:
> 
> 
> On 6/8/22 05:14, Raf Czlonka wrote:
> > Hello,
> > 
> > Given the recent changes[0][1], the template should probably be updated.
> > 
> > [0] https://marc.info/?l=openbsd-cvs&m=165313015803498&w=2
> > [1] https://marc.info/?l=openbsd-cvs&m=165356436530341&w=2
> > 
> > I'm not subscribed to this mailing list so please CC me if need be.
> > 
> > Regards,
> > 
> > Raf
> > 
> > Index: infrastructure/templates/rc.template
> > ===================================================================
> > RCS file: /cvs/ports/infrastructure/templates/rc.template,v
> > retrieving revision 1.14
> > diff -u -p -r1.14 rc.template
> > --- infrastructure/templates/rc.template    6 Mar 2022 20:10:00 -0000       
> > 1.14
> > +++ infrastructure/templates/rc.template    8 Jun 2022 09:07:38 -0000
> > @@ -6,6 +6,7 @@ See rc.subr(8) for more information.
> >   #!/bin/ksh
> >   daemon="${TRUEPREFIX}/bin/foobar"
> > +#daemon_execdir=
> >   #daemon_flags=
> >   #daemon_logger=
> >   #daemon_rtable=0
> > @@ -23,7 +24,7 @@ daemon="${TRUEPREFIX}/bin/foobar"
> >   #}
> >   #rc_start() {
> > -#  ${rcexec} "${daemon_logger:+set -o pipefail; }${daemon} 
> > ${daemon_flags}${daemon_logger:+ 2>&1 |
> > +#  rc_exec "${daemon_logger:+set -o pipefail; }${daemon} 
> > ${daemon_flags}${daemon_logger:+ 2>&1 |
> >   #         logger -ip ${daemon_logger} -t ${_name}}"
> >   #}
> Don't need daemon_logger part in rc_start now, should be only

Hi Aisha,

Oh sure, I'm sure there are other changes needed there - the above
is simply a minimal diff which incorporates only the recent changes.

BTW, the below is obviously not a diff but care should be taken if
copy-pasted as it includes three UTF-8-encoded NBSP characters.

> rc_start() {
>     rc_exec "${daemon} ${daemon_flags}"
> }

Regards,

Raf

Reply via email to