Bugzilla (bugzilla.redhat.com) will be under maintenance for infrastructure upgrades and will not be available on July 31st between 12:30 AM - 05:30 AM UTC. We appreciate your understanding and patience. You can follow status.redhat.com for details.
Bug 522708 - Wrong test in amtu initscript
Summary: Wrong test in amtu initscript
Alias: None
Product: Fedora
Classification: Fedora
Component: amtu
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Steve Grubb
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: InitScriptsProject 536887
TreeView+ depends on / blocked
Reported: 2009-09-11 06:50 UTC by Miroslav Vadkerti
Modified: 2010-03-11 17:35 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 536887 (view as bug list)
Last Closed: 2009-09-14 13:49:48 UTC
Type: ---

Attachments (Terms of Use)

Description Miroslav Vadkerti 2009-09-11 06:50:48 UTC
Description of problem:
amtu initscript doesn't work with correct configuration file /etc/sysconfig/amtu. The bug is in these two tests (they always fail):

	test x"$AMTU_HALT_ON_FAILURE" = "x" || exit 6
	test x"$HALT_COMMAND" = "x" || exit 6

The correct code should be:

	test 'x"$AMTU_HALT_ON_FAILURE" = "x"' || exit 6
	test 'x"$HALT_COMMAND" = "x"' || exit 6

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

How reproducible:

Steps to Reproduce:
1. service amtu start
Actual results:
Doesn't start and returns 6 (not configured correctly)

Expected results:
Correctly start

Additional info:

Comment 1 Steve Grubb 2009-09-11 15:19:59 UTC
Hmm...wonder how that got in there. The correct code should be:

 test x"$AMTU_HALT_ON_FAILURE" != "x" || exit 6
 test x"$HALT_COMMAND" != "x" || exit 6

Comment 2 Steve Grubb 2009-09-11 18:49:14 UTC
I built amtu-1.0.8-5 in rawhide to solve this problem. Please give it a test. Thanks.

Comment 3 Miroslav Vadkerti 2009-09-14 10:55:42 UTC
I tested the newest package and problem seems to be fixed. Although the there is no result after the run (I mean [  OK  ] or [  FAILED  ]). It would be nice if this would be displayed.

Comment 4 Steve Grubb 2009-09-14 13:49:48 UTC
Yes, it would be nice to have pass or fail on the screen. But, there is no way to do this without patching amtu itself as best as I can tell. The basic problem is that amtu has no quiet mode and will scribble all over the screen. I will close this bug since the basic requirements have been met. Thanks for reporting the bug.

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