FS#23205 - [initscripts] set_consolefont in /etc/rc.d/functions fails to call stat_done on a branch

Attached to Project: Arch Linux
Opened by Cheer Xiao (xiaq) - Wednesday, 09 March 2011, 11:48 GMT
Last edited by Tom Gundersen (tomegun) - Wednesday, 13 April 2011, 09:22 GMT
Task Type Bug Report
Category Arch Projects
Status Closed
Assigned To Tom Gundersen (tomegun)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

In /etc/rc.d/functions it goes:

if (($? != 0)); then
stat_fail
elif [[ $CONSOLEMAP ]]; then
cat <<"EOF" >>/etc/profile.d/locale.sh
if [ "$CONSOLE" = "" -a "$TERM" = "linux" -a -t 1 ]; then printf "\033(K"; fi
EOF
stat_done
fi

On the implicit "else" branch, stat_done is not called, leaving the text "[BUSY]". Also no newline, which in turn leads to the next message overwriting the one about console font.

Additional info:
* initscripts 2011.02.1-1


Steps to reproduce:

1. Set CONSOLEFONT in rc.conf.
2. Reboot and watch the startup message.
   bad.png (41.1 KiB)
This task depends upon

Closed by  Tom Gundersen (tomegun)
Wednesday, 13 April 2011, 09:22 GMT
Reason for closing:  Fixed
Additional comments about closing:  I hope it is ok to close now, Im going through the reports in preparation for a release.
Comment by Greg (dolby) - Wednesday, 09 March 2011, 11:59 GMT
This has been fixed in git: http://projects.archlinux.org/initscripts.git/commit/?id=15ff1596246ba37521ff8807722a7679390fdf01
Will be in next release.

Maybe this might be worth staying open for some time, as its the third or fourth bug report about this already.
Comment by Cheer Xiao (xiaq) - Wednesday, 09 March 2011, 12:07 GMT
So... the fix is due in next release, instead of being directly pushed to the repository. That doesn't agree with my understanding of rolling release. Point me out where I'm wrong.
Comment by Greg (dolby) - Wednesday, 09 March 2011, 13:12 GMT
Software has to be released in order to be packaged. Just like every other application. Its nothing critical or even important anyway. Just cosmetic.
Comment by Cheer Xiao (xiaq) - Wednesday, 09 March 2011, 13:30 GMT
By release are your referring to what I see in https://www.archlinux.org/download/? But I get updates every few days. (And that's rolling release, right?) I must have misunderstood something.
Comment by Greg (dolby) - Wednesday, 09 March 2011, 13:33 GMT
No, i mean release of the initscripts package.
Comment by Cheer Xiao (xiaq) - Wednesday, 09 March 2011, 13:39 GMT
I figured that out just now :) Thanks for your patience and explanation.

Loading...