Bug 15475 - sysvinitfiles doc to be updated
Summary: sysvinitfiles doc to be updated
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: initscripts
Version: 7.0
Hardware: i386
OS: Linux
low
low
Target Milestone: ---
Assignee: Bill Nottingham
QA Contact:
URL:
Whiteboard:
Keywords:
: 14788 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2000-08-05 14:26 UTC by giulioo
Modified: 2014-03-17 02:15 UTC (History)
1 user (show)

(edit)
Clone Of:
(edit)
Last Closed: 2000-08-22 14:58:36 UTC


Attachments (Terms of Use)

Description giulioo 2000-08-05 14:26:43 UTC
1) It seems you decided to stick with /etc/rc.d/init.d for now and 
make /etc/init.d just a link, so it should say

 /etc/rc.d/init.d/<servicename>
instead of
 /etc/init.d/<servicename>

since all the rpms use directly /etc/rc.d/init.d in their %files and not 
the link.


2) It should document "condrestart", that you added to all of the init 
scripts to conditionally restart services. 


3) The doc makes the example of

case $1 in

	start)
		<all the stuff here>


but all of your latest init scripts use:

start () {
	<all the stuff here>.
}


case $1 in
	start) 
	start
...


Which is the preferred way if any?

Comment 1 Bill Nottingham 2000-08-22 14:58:34 UTC
*** Bug 14788 has been marked as a duplicate of this bug. ***

Comment 2 Bill Nottingham 2000-08-22 15:35:31 UTC
The second way is better, because that way if you implement restart as:

  restart)
	stop
	start

it doesn't run two more shell invocations. More of this stuff will
be documented in initscripts-5.48-1.h


Note You need to log in before you can comment on or make changes to this bug.