Package:  console-tools
Version:  1:0.2.3dbs-70
Severity: normal
Tags:     patch
File:     /etc/init.d/console-screen.sh

    $ grep -A1 'Setting console screen modes and fonts' /var/log/boot
    Sat May 12 23:14:32 2012: Setting console screen modes and fonts.
    Sat May 12 23:14:32 2012: ^[[9;30]^[[14;30]Setting kernel variables ...done.
    $

  I think it is inevitable to have something like ^[[9;30]^[[14;30]
because setterm is writing to stdout.  I also think that [[9;30]^[[14;30]
is because of the default settings in /etc/console-tools/config.  
Therefore, I propose the following patch.  I deliberately didn't put 
log_action_*_msg after
[ "$VERBOSE" != "no" ].

--- a/etc/init.d/console-screen.sh      2010-03-02 22:29:07.000000000 +0200
+++ b/etc/init.d/console-screen.sh      2012-05-13 03:15:20.000000000 +0300
@@ -209,7 +209,15 @@ setup_screensaver() {
         setterm_args="$setterm_args -powerdown $POWERDOWN_TIME"
     fi
     if [ "$setterm_args" ]; then
-        setterm $setterm_args
+        if [ "$BLANK_TIME"  -o  "$POWERDOWN_TIME" ]; then
+            # No point to guard log_action_... with
+            # [ "$VERBOSE" != "no" ]
+            log_action_begin_msg "    console screensaver "
+            setterm $setterm_args
+            log_action_end_msg 0
+        else
+            setterm $setterm_args
+        fi
     fi

 }




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to