Bug#998779: bs1770gain: bashism in configure script

2022-12-05 Thread Petter Reinholdtsen
[Petter Reinholdtsen] > Very good. While these changes in the latest version of bs1770gain kept > the script working with bash, it was not enough to get it working with > dash. Sorry, my mistake. > This is the output I get when trying 'dash ./configure': I have to use 'dash ./configure

Bug#998779: bs1770gain: bashism in configure script

2022-12-05 Thread Petter Reinholdtsen
[Peter Belkner] > Hi Petter, > > I've tested it with all '=' replaced by '==' and it seemed to be ok. Very good. While these changes in the latest version of bs1770gain kept the script working with bash, it was not enough to get it working with dash. This is the output I get when trying 'dash

Bug#998779: bs1770gain: bashism in configure script

2022-09-20 Thread Peter Belkner
Hi Petter, I've tested it with all '=' replaced by '==' and it seemed to be ok. Many thanks Peter On 20.09.2022 07:11, Petter Reinholdtsen wrote: [Peter Belkner] is the intention of your patch to substitute all single comparison singes ('=') by double comparison singes ('==')? My

Bug#998779: bs1770gain: bashism in configure script

2022-09-19 Thread Peter Belkner
Hi Petter, thank you for the clarification! Best regards Peter On 20.09.2022 07:11, Petter Reinholdtsen wrote: [Peter Belkner] is the intention of your patch to substitute all single comparison singes ('=') by double comparison singes ('==')? My intention with the patch was the other

Bug#998779: bs1770gain: bashism in configure script

2022-09-19 Thread Petter Reinholdtsen
[Peter Belkner] > is the intention of your patch to substitute all single comparison > singes ('=') by double comparison singes ('==')? My intention with the patch was the other way, to replace 'test x == y' with 'test x = y', as he latter is POSIX notation, while the former is bash notation.

Bug#998779: bs1770gain: bashism in configure script

2022-09-19 Thread Andrej Shadura
Hi, On Mon, 19 Sep 2022, at 19:11, Petter Reinholdtsen wrote: > [Andrej Shadura] >> Your package uses configure script with bash features not present in >> POSIX without explicitly declaring the need to bash shell; this >> currently works as configure scripts select bash, but when dash enables >>

Bug#998779: bs1770gain: bashism in configure script

2022-09-19 Thread Petter Reinholdtsen
[Andrej Shadura] > Your package uses configure script with bash features not present in > POSIX without explicitly declaring the need to bash shell; this > currently works as configure scripts select bash, but when dash enables > LINENO support, your configure script will start failing: Can you