Bug 37473
Summary: | missing status & stop reports failure always in initscripts | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [Retired] Red Hat Powertools | Reporter: | Zing <shishz> | ||||||||
Component: | radiusd-cistron | Assignee: | Tim Powers <timp> | ||||||||
Status: | CLOSED RAWHIDE | QA Contact: | |||||||||
Severity: | low | Docs Contact: | |||||||||
Priority: | low | ||||||||||
Version: | 7.1 | ||||||||||
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: | 2001-04-27 13:28:05 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: | |||||||||||
Attachments: |
|
Description
Zing
2001-04-24 19:56:31 UTC
Created attachment 16287 [details]
init script modifications - status & stop
I merged your patch in with mine. I have pushed the updated package into http://people.redhat.com/timp/RPMS/radiusd-cistron-1.6.4-4.i386.rpm and http://people.redhat.com/timp/SRPMS/radvd-0.6.2-7.src.rpm Please take a look at these and let me know if they are OK, also let me know if there are further issues. Tim I think radvd was some other bug? :) 1. The status part of the patch didn't seem to fully apply. It'll seg fault. New patch included. 2. New bug, the closing "fi" in the preun script is missing. that'll cause upgrade failure. patch included for spec file... thanks zing Created attachment 16405 [details]
status -> rhstatus
Created attachment 16406 [details]
closing brace missing
Take a look at the poackage again, I uploaded the wrong build of it lakst night, download it again. Oh, and the status vs rhstatus thing can stay the same, I had to merge the patch by hand, and since this is how status is used in mopst of mour other initscripts that's how I put it in. Download it again and tell me what you think. Tim Nope, same problems... 1. we now have two functions called status()! One's pulled in from the top of the init script ("./etc/init.d/functions") and the one you edited in... if you want to, just get rid of the status() function manually edited in and call "status radiusd" from the case status switch at the bottom. 2. the missing "fi" is still missing from the spec file. 3. stopping and starting functions work :) thanks zing OK. radiusd-cistron-1.6.4-5 has been uploaded to people.redhat.com/timp/RPMS. That should do it. Please note that since there is a typo in the %preun script in the package as shipped in Powertools 7.1 you will have to do this to install the fixed package, unfortuantely there is no other way around the problem: # /sbin/service radiusd stop # /sbin/chkconfig --del radiusd # rpm -e --noscripts radiusd-cistron # rpm -ivh radiusd-cistron-1.6.4-5.i386.rpm Tim That did it! thnks zing Resolving this as rawhide since that's where the package is now located. *** Bug 40326 has been marked as a duplicate of this bug. *** |