Bug#833053: Syntax error in the script that setupcon outputs

2016-08-03 Thread Anton Zinoviev
On Tue, Aug 02, 2016 at 10:43:45PM +0200, Philip Hands wrote: > It looks like there are a couple of other instances of the potentially > empty braces problem in this script, so we could address them at the > same time just in case. I agree. Even if the current script didn't have more empty

Bug#833053: Syntax error in the script that setupcon outputs

2016-08-02 Thread Philip Hands
It looks like there are a couple of other instances of the potentially empty braces problem in this script, so we could address them at the same time just in case. Also, if the variable somehow ended up containing just whitespace then the proposed fix would fail. How about just adding : to each

Bug#833053: Syntax error in the script that setupcon outputs

2016-07-31 Thread Geert Stappers
Control: tag -1 patch stop On Sun, Jul 31, 2016 at 07:02:03PM +0900, Jonny wrote: > --- /a/setupcon > +++ /b/setupcon > @@ -911,7 +911,7 @@ > if ls /dev/fb* >/dev/null 2>/dev/null; then > for i in /dev/vcs[0-9]*; do > { > -$commands_f > +${commands_f:-:} >

Processed: Re: Bug#833053: Syntax error in the script that setupcon outputs

2016-07-31 Thread Debian Bug Tracking System
Processing control commands: > tag -1 patch Bug #833053 [console-setup] Syntax error in the script that setupcon outputs Added tag(s) patch. -- 833053: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=833053 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems

Bug#833053: Syntax error in the script that setupcon outputs

2016-07-31 Thread Jonny
Package: console-setup Version: 1.148 Hi, if you choose "Do not change the boot/kernel font" in debconf questions '"VGA" has a traditional appearance and has medium coverage of ...', and outputs the "/etc/console-setup/cached_setup_font.sh" such as the following. #!/bin/sh if ls /dev/fb*