FS#5304 - laptop-mode init script fix & beautification

Attached to Project: Arch Linux
Opened by Manolis Tzanidakis (mtzanidakis) - Monday, 28 August 2006, 14:39 GMT
Last edited by Woody Gilk (Shadowhand) - Tuesday, 10 October 2006, 05:19 GMT
Task Type Bug Report
Category System
Status Closed
Assigned To Aaron Griffin (phrakture)
Architecture not specified
Severity Low
Priority Normal
Reported Version 0.7.2 Gimmick
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

On the attached patch:
{add,rm}_daemon acpid -> laptop-mode
and some beautification (IMO) bits. Please check it out.
(application/octet-stream)    laptop-mode.rc.diff (1.4 KiB)
This task depends upon

Closed by  Aaron Griffin (phrakture)
Wednesday, 22 November 2006, 07:21 GMT
Reason for closing:  Implemented
Additional comments about closing:  Changes applied in CVS
Comment by Aaron Griffin (phrakture) - Tuesday, 10 October 2006, 14:43 GMT
Well, the acpid stuff is an "oops" on my part. I don't know if I like this:
echo -n " `$DAEMON auto init force`"

I will apply most of this when I get a chance.
Comment by Manolis Tzanidakis (mtzanidakis) - Tuesday, 10 October 2006, 19:56 GMT
The 'echo -n " `$DAEMON auto init force`"' just runs the linux_laptop script and
prints running status at the same time.
I use since 08/28 without problems on my laptop.
Comment by Aaron Griffin (phrakture) - Tuesday, 10 October 2006, 20:00 GMT
Right, I understand what it does, but typically archlinux rc.d scripts are silent on start/stop - that's what the "status" arg was put there for.
Comment by Manolis Tzanidakis (mtzanidakis) - Tuesday, 10 October 2006, 20:43 GMT
It doesn't provide the same output as the "status" arg. Just give it a shot to test it for
yourself; it won't eat your cereal :). Anyway, it's your package and thus your call.

Loading...