Bug 15475
| Summary: | sysvinitfiles doc to be updated | ||
|---|---|---|---|
| Product: | [Retired] Red Hat Linux | Reporter: | giulioo |
| Component: | initscripts | Assignee: | Bill Nottingham <notting> |
| Status: | CLOSED RAWHIDE | QA Contact: | |
| Severity: | low | Docs Contact: | |
| Priority: | low | ||
| Version: | 7.0 | CC: | rvokal |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | i386 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2000-08-22 14:58:36 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
*** Bug 14788 has been marked as a duplicate of this bug. *** 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 |
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?