Bug 237802 - Wrong init script
Summary: Wrong init script
Keywords:
Status: CLOSED DUPLICATE of bug 246875
Alias: None
Product: Fedora
Classification: Fedora
Component: avahi
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Martin Bacovsky
QA Contact:
URL:
Whiteboard:
Depends On: 237800
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-04-25 14:53 UTC by Michal Marciniszyn
Modified: 2014-02-10 23:02 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-08-28 15:49:38 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
A proposed patch for the avahi-daemon init script (1.37 KB, patch)
2007-05-18 15:38 UTC, Jakub Hrozek
no flags Details | Diff
A proposed patch for the avahi-dnsconf init script (1.19 KB, patch)
2007-05-18 15:39 UTC, Jakub Hrozek
no flags Details | Diff

Description Michal Marciniszyn 2007-04-25 14:53:57 UTC
+++ This bug was initially created as a clone of Bug #237800 +++

Description of problem:
Following code ends up with error code 0 when networking is not presented even
though the service is not started. It also forbids the correct run of status
command of the script.
[ ${NETWORKING} = "no" ] && exit 0

AVAHI_BIN=/usr/sbin/avahi-daemon
test -x $AVAHI_BIN || exit 5
It appears in both avahi-daemon and avahi-dnsconfd. Also incorrect return codes
through the code are present.

Version-Release number of selected component (if applicable):
all

How reproducible:
always

Steps to Reproduce:
1. service netowrk stop
2. service avahi-daemon start/status
3. echo $?
  
Actual results:
0

Expected results:
non-zero result with respect to our guidelines.

Additional info:
When fixing this bug, please obbey our init script guidelines and be sure that
status command is run correctly.
Our guidelines are on following two pages:
http://intranet.corp.redhat.com/ic/intranet/InitscriptsSpec.html
http://intranet.corp.redhat.com/ic/intranet/InitscriptGuidelines.html

For an example of the script that returns the error codes correctly and always
runs status see:
http://devserv.devel.redhat.com/~mmarcini/amd

This bug is tracked by 237789.

Comment 1 Jakub Hrozek 2007-05-18 15:38:42 UTC
Created attachment 155008 [details]
A proposed patch for the avahi-daemon init script

Comment 2 Jakub Hrozek 2007-05-18 15:39:51 UTC
Created attachment 155009 [details]
A proposed patch for the avahi-dnsconf init script

Comment 3 Martin Bacovsky 2007-08-28 15:49:38 UTC

*** This bug has been marked as a duplicate of 246875 ***


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