Re: Patch to fix /etc/rc.d/identd...

2013-12-11 Thread Jérémie Courrèges-Anglas
Adam Jeanguenat a...@voyager.6v6.org writes:

Below is a patch to fix the identd rc.d script, which currently
 doesn't allow you to stop the daemon because ${pexp} is passed
 incorrectly.

Fixed, thanks.

Note the string identd: resolver is 16 chars long and at the
 limit of what OpenBSD cares about (according to the pgrep/pkill(1) man
 page), but it works fine. I suppose both strings could be shortened if
 desired, but I figured the least amount of ambiguity was the best.

This 16-bytes limitation is not a problem when using -f.

If identd is already running, /var/run/rc.d/identd will need to be
 manually deleted first as it contains the wrong ${pexp}.

--avj


 Index: identd
 ===
 RCS file: /home/cvsync/src/etc/rc.d/identd,v
 retrieving revision 1.6
 diff -u -p -r1.6 identd
 --- identd8 Aug 2013 15:41:28 -   1.6
 +++ identd4 Dec 2013 19:17:26 -
 @@ -7,6 +7,7 @@ daemon_flags=-e
  
  . /etc/rc.d/rc.subr
  
 +pexp=identd: (listen|resolver)
  rc_reload=NO
  
  rc_cmd $1


-- 
jca | PGP: 0x06A11494 / 61DB D9A0 00A4 67CF 2A90  8961 6191 8FBF 06A1 1494



Patch to fix /etc/rc.d/identd...

2013-12-04 Thread Adam Jeanguenat
   Below is a patch to fix the identd rc.d script, which currently
doesn't allow you to stop the daemon because ${pexp} is passed
incorrectly.

   Note the string identd: resolver is 16 chars long and at the
limit of what OpenBSD cares about (according to the pgrep/pkill(1) man
page), but it works fine. I suppose both strings could be shortened if
desired, but I figured the least amount of ambiguity was the best.

   If identd is already running, /var/run/rc.d/identd will need to be
manually deleted first as it contains the wrong ${pexp}.

   --avj


Index: identd
===
RCS file: /home/cvsync/src/etc/rc.d/identd,v
retrieving revision 1.6
diff -u -p -r1.6 identd
--- identd  8 Aug 2013 15:41:28 -   1.6
+++ identd  4 Dec 2013 19:17:26 -
@@ -7,6 +7,7 @@ daemon_flags=-e
 
 . /etc/rc.d/rc.subr
 
+pexp=identd: (listen|resolver)
 rc_reload=NO
 
 rc_cmd $1