Re: Trying to alter runit a little.

[Gerrit Pape <pape@smarden.org>]

On Fri, Nov 18, 2005 at 03:07:04AM -0800, L. Jason Godsey wrote:
> build# diff -u runit.c.original runit.c
> --- runit.c.original    Fri Nov 18 10:56:29 2005
> +++ runit.c     Fri Nov 18 10:57:41 2005
> @@ -54,7 +54,7 @@
>    int ttyfd;
>    struct stat s;
> 
> -  if (getpid() != 1) strerr_die2x(111, FATAL, "must be run as process
> no 1.");
> +  //if (getpid() != 1) strerr_die2x(111, FATAL, "must be run as
> process no 1.");
>    setsid();
> 
>    sig_block(sig_alarm);
> 
> build# pwd
> /usr/ports/sysutils/runit/work/admin/runit-1.3.1/src
> 
> build# make check
> ./check-local chpst runit runit-init runsv runsvchdir runsvctrl
> runsvdir runsvstat sv  svlogd svwaitdown svwaitup utmpset
> Checking chpst...
> Checking runit...
> Killed
> 
> main# echo $?
> 137

Ah yes.  The runit program test actually is kind of a pseudo test.  It
just tests that the runit program can't be run unless it has pid 1.  So
if you make this change you need to skip or adapt the runit program
test, e.g.:

--- src/Makefile        23 Jul 2005 13:12:39 -0000      1.29
+++ src/Makefile        21 Nov 2005 08:49:52 -0000
@@ -3,8 +3,10 @@
 
 default: sysdeps $(IT)
 
-check: $(IT)
-       ./check-local $(IT)
+check: chpst runit-init runsv runsvchdir runsvctrl runsvdir runsvstat sv \
+         svlogd svwaitdown svwaitup utmpset
+       ./check-local chpst runit-init runsv runsvchdir runsvctrl runsvdir \
+         runsvstat sv svlogd svwaitdown svwaitup utmpset
 
 runit: load runit.o unix.a byte.a
        ./load runit unix.a byte.a -static

Regards, Gerrit.