Bug 430816 - Xinetd'd initscript contains syntax errors (uninstall fails)
Summary: Xinetd'd initscript contains syntax errors (uninstall fails)
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: xinetd
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Jan Safranek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 430909 431104 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-29 22:45 UTC by Stewart Adam
Modified: 2008-02-01 08:10 UTC (History)
2 users (show)

Fixed In Version: xinetd-2.3.14-17.fc9
Clone Of:
Environment:
Last Closed: 2008-01-30 08:31:07 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Patch to fix initscript syntax error, link condrestart --> restart (342 bytes, patch)
2008-01-29 22:45 UTC, Stewart Adam
no flags Details | Diff
another possible fix (372 bytes, patch)
2008-01-30 01:27 UTC, Michal Jaegermann
no flags Details | Diff

Description Stewart Adam 2008-01-29 22:45:55 UTC
Description of problem:
Uninstall of xinetd-2.3.14-14.fc8.x86_64 fails because of an error present in
the initscript (also present in xinetd-2.3.14-16.fc9.x86_64).

The patch attached fixes the syntax error, as well as links condrestart to the
restart function so that the rpm scriptlets restarts xinetd.

Version-Release number of selected component (if applicable):
xinetd-2.3.14-14.fc8.x86_64
xinetd-2.3.14-16.fc9.x86_64

How reproducible:
Always

Steps to Reproduce:
1. Update/remove xinetd when xinetd-2.3.14-16.fc9.x86_64 was installed
  
Actual results:
The scriptlet fails, package remains installed

Expected results:
The package uninstalls and where applicable condrestart restarts the Xinetd service

Additional info:

Comment 1 Stewart Adam 2008-01-29 22:45:55 UTC
Created attachment 293353 [details]
Patch to fix initscript syntax error, link condrestart --> restart

Comment 2 Michal Jaegermann 2008-01-30 01:27:20 UTC
Created attachment 293359 [details]
another possible fix

It appears that actually something simpler, like that, was intended
in ther first place.

Comment 3 Stewart Adam 2008-01-30 02:02:39 UTC
<snip>
+    condrestart|try-restart)
 	condrestart <-- shouldn't this be "restart"?
<snip>


Comment 4 Jan Safranek 2008-01-30 08:31:07 UTC
Thanks for the bug report! It should be fixed in xinetd 2.3.14-17.

Comment 5 Jan Safranek 2008-01-30 15:17:29 UTC
*** Bug 430909 has been marked as a duplicate of this bug. ***

Comment 6 Jan Safranek 2008-02-01 08:10:02 UTC
*** Bug 431104 has been marked as a duplicate of this bug. ***


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