[bug] dash eats one line

[bug] dash eats one line

[Pedro]

[-- Attachment #1.1.1: Type: text/plain, Size: 944 bytes --]

Hi,

I am using dash on debian 10 buster (stable release) which according to 
`dpkg -S dash` is `Version: 0.5.10.2-5`

evilham encountered a bug where dash eats one line but not in freebsd's 
shell nor bash

attached you can find an example to reproduce it

execution of the example in my computer (enableUFWRules is not in new line):

  ./dash-bug.sh
# Enable UFW rules only if requested (it was)enableUFWRules

  $ bash dash-bug.sh
# Enable UFW rules only if requested (it was)
enableUFWRules

and as extra reference, here you have some workarounds to avoid the bug 
in your dash script [1]

Salut,
Pedro

[1]

failed

https://git.sr.ht/~evilham/cdist-evilham/commit/bf65172c8291ec4be712576050bf7a4bcf462017

failed

https://git.sr.ht/~evilham/cdist-evilham/commit/1aa8b012200b0160e78a65520175bb224ffc9899

worked

https://git.sr.ht/~evilham/cdist-evilham/commit/8996e2b66531e6b54b52207ea5bdf3b71c795656

[-- Attachment #1.1.2: dash-bug.sh --]
[-- Type: application/x-shellscript, Size: 201 bytes --]

[-- Attachment #1.1.3: OpenPGP_0x9D64597C3A982DCA.asc --]
[-- Type: application/pgp-keys, Size: 959 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

Re: [bug] dash eats one line

[Harald van Dijk]

Hi,

On 27/11/2020 12:16, Pedro wrote:
> Hi,
> 
> I am using dash on debian 10 buster (stable release) which according to 
> `dpkg -S dash` is `Version: 0.5.10.2-5`
> 
> evilham encountered a bug where dash eats one line but not in freebsd's 
> shell nor bash
> 
> attached you can find an example to reproduce it
> 
> execution of the example in my computer (enableUFWRules is not in new 
> line):
> 
>   ./dash-bug.sh
> # Enable UFW rules only if requested (it was)enableUFWRules
> 
>   $ bash dash-bug.sh
> # Enable UFW rules only if requested (it was)
> enableUFWRules

This looks like 
<https://www.mail-archive.com/dash@vger.kernel.org/msg01883.html>, which 
has been fixed in dash 0.5.11. I can confirm that your script produces 
the expected output with dash 0.5.11 as well as dash built from current Git.

Cheers,
Harald van Dijk