Bug#1063518: console-setup: setupcon: 1386: Syntax error: Missing '))'

2024-02-09 Thread Philip Hands
Michael Tokarev writes: > 09.02.2024 16:58, ca...@allfreemail.net >> Package: console-setup >> Followup-For: Bug #1063518 >> >> Consider making all scripts provided by console-setup >> shellcheck-clean, there are lots of tiny issues that can turn into >> big issues under the right conditions. >

Bug#1063518: console-setup: setupcon: 1386: Syntax error: Missing '))'

2024-02-09 Thread Michael Tokarev
09.02.2024 17:57, Thorsten Bonow : dash << 'EOF'    [15:28:53] if $( (true) 2>/dev/null); then  echo "42" fi EOF 42 which only works in dash because of the added space between the command substitution $(...) and the subshell (...). Does dash think it

Bug#1063518: console-setup: setupcon: 1386: Syntax error: Missing '))'

2024-02-09 Thread Thorsten Bonow
Thorsten Bonow writes: [...] But what's POSIX take on this? I couldn't find anything. Is this a bug in dash or in setupcon? I'm stupid[1]. It's a bug in setupcon, POSIX requires the space: "The syntax of the shell command language has an ambiguity for expansions beginning with "$((",

Bug#1063518: console-setup: setupcon: 1386: Syntax error: Missing '))'

2024-02-09 Thread Thorsten Bonow
Package: console-setup Version: 1.225 Severity: grave After the upgrade from 1.223, console-setup.service failed to start due to a syntax error in the setupcon script: , | $ setupcon | /usr/bin/setupcon: 1386: Syntax error: Missing '))' ` It looks like dash does not like the

Bug#1063518: console-setup: setupcon: 1386: Syntax error: Missing '))'

2024-02-09 Thread Michael Tokarev
09.02.2024 16:58, ca...@allfreemail.net Package: console-setup Followup-For: Bug #1063518 Consider making all scripts provided by console-setup shellcheck-clean, there are lots of tiny issues that can turn into big issues under the right conditions. Please do not do this. Shellcheck is a

Bug#1063518: console-setup: setupcon: 1386: Syntax error: Missing '))'

2024-02-09 Thread cacin
Package: console-setup Followup-For: Bug #1063518 Consider making all scripts provided by console-setup shellcheck-clean, there are lots of tiny issues that can turn into big issues under the right conditions. https://tracker.debian.org/pkg/shellcheck

Bug#1063518: console-setup: setupcon: 1386: Syntax error: Missing '))'

2024-02-09 Thread Sven Joachim
Package: console-setup Version: 1.225 Severity: grave After the upgrade from 1.223, console-setup.service failed to start due to a syntax error in the setupcon script: , | $ setupcon | /usr/bin/setupcon: 1386: Syntax error: Missing '))' ` It looks like dash does not like the construct