From jake@h24-64-221-247.gv.wave.shaw.ca  Sun Jan 17 13:44:16 1999
Received: from h24-64-221-247.gv.wave.shaw.ca (h24-64-221-247.gv.wave.shaw.ca [24.64.221.247])
          by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id NAA03659
          for <FreeBSD-gnats-submit@freebsd.org>; Sun, 17 Jan 1999 13:44:15 -0800 (PST)
          (envelope-from jake@h24-64-221-247.gv.wave.shaw.ca)
Received: (from jake@localhost)
	by h24-64-221-247.gv.wave.shaw.ca (8.9.2/8.9.2) id NAA00583;
	Sun, 17 Jan 1999 13:44:11 -0800 (PST)
	(envelope-from jake)
Message-Id: <199901172144.NAA00583@h24-64-221-247.gv.wave.shaw.ca>
Date: Sun, 17 Jan 1999 13:44:11 -0800 (PST)
From: Jake <jake@checker.org>
Reply-To: jake@checker.org
To: FreeBSD-gnats-submit@freebsd.org
Subject: syntax error concerning loading vinum from /etc/rc
X-Send-Pr-Version: 3.2

>Number:         9544
>Category:       bin
>Synopsis:       syntax error concerning loading vinum from /etc/rc
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sun Jan 17 13:50:00 PST 1999
>Closed-Date:    Thu Jun 1 19:43:10 PDT 2000
>Last-Modified:  Thu Jun 01 19:46:24 PDT 2000
>Originator:     Jake
>Release:        FreeBSD 3.0-CURRENT i386
>Organization:
no organization
>Environment:

	
3.0-CURRENT as of 01/17 12:41:24 

>Description:

	
A stub to have vinum loaded and configured on startup was
added to /etc/rc ( Thanks! ), unfortunately, the variable
$vinum_slices is not properly initialized when vinum is started.
/etc/rc.conf has to be sucked in before the command
vinum read $vinum_slices is executed.

>How-To-Repeat:

	
Try to have vinum loaded on startup using the stubs in /etc/rc and 
/etc/rc.conf.  

>Fix:
	
	
The code fragment

# If there is a global system configuration file, suck it in.
if [ -f /etc/rc.conf ]; then
        . /etc/rc.conf
fi

needs to be moved from where it is now to somewhere before
vinum gets configured.

Here is an attempt at a patch:
h24-64-221-247# diff -u /usr/src/etc/rc /etc/rc | less
--- /usr/src/etc/rc     Sun Jan 17 09:43:03 1999
+++ /etc/rc     Sun Jan 17 13:33:43 1999
@@ -22,6 +22,11 @@
 PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin
 export PATH
 
+# If there is a global system configuration file, suck it in.
+if [ -f /etc/rc.conf ]; then
+        . /etc/rc.conf
+fi
+
 # Configure ccd devices.
 if [ -f /etc/ccd.conf ]; then
        ccdconfig -C
@@ -88,11 +93,6 @@
 if [ $? != 0 ]; then
        echo "Filesystem mount failed, startup aborted"
        exit 1
-fi
-
-# If there is a global system configuration file, suck it in.
-if [ -f /etc/rc.conf ]; then
-       . /etc/rc.conf
 fi
 
 # If old file exists, whine until they fix it.

Thank you
>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: gnats-admin->freebsd-bugs 
Responsible-Changed-By: fenner 
Responsible-Changed-When: Fri Jan 22 13:01:39 PST 1999 
Responsible-Changed-Why:  
Fixing misfiled PR. 
State-Changed-From-To: open->closed 
State-Changed-By: nrahlstr 
State-Changed-When: Thu Jun 1 19:43:10 PDT 2000 
State-Changed-Why:  
The option vinum_slices no long appears in the rc.conf files. 
It appears to have been ultimately put to death in revision 1.186 
of /etc/rc by grog. 


http://www.freebsd.org/cgi/query-pr.cgi?pr=9544 
>Unformatted:
