Your message dated Tue, 26 Nov 2019 21:43:15 +0000
with message-id 
<CABwkT9rThPS2snS7HFQKQQkYnUwdDG=9ztcydbtcyqs50l+...@mail.gmail.com>
and subject line 
has caused the Debian Bug report #753959,
regarding shellcheck: false negative with another bad use of ls
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
753959: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=753959
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: shellcheck
Version: 0.3.3-1
Severity: normal

shellcheck doesn't detect this bad use of ls (SC2045) from [1]:

# BAD! Don't do this!
arr=($(ls)) # Word-splitting and globbing here, same mistake as above
for f in "${arr[@]}"; do
    echo "$f"
done

     1. http://mywiki.wooledge.org/ParsingLs

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (900, 'testing'), (800, 'unstable'), (700, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.14-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_AU.UTF-8, LC_CTYPE=en_AU.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages shellcheck depends on:
ii  libc6     2.19-4
ii  libffi6   3.1-2
ii  libgmp10  2:6.0.0+dfsg-4

shellcheck recommends no packages.

shellcheck suggests no packages.

-- no debconf information

-- 
bye,
pabs

http://wiki.debian.org/PaulWise

Attachment: signature.asc
Description: This is a digitally signed message part


--- End Message ---
--- Begin Message ---
Just tested the example code with latest version 0.7.0 and got:
In test.sh line 2:
arr=($(ls)) # Word-splitting and globbing here, same mistake as above
     ^---^ SC2207: Prefer mapfile or read -a to split command output (or
quote to avoid splitting).

Solved by double quoting $(ls), both the warning and the issue itself are
fixed.

Please feel free to reopen if you think there has been a mistake.

-- 
Samuel Henrique <samueloph>

--- End Message ---
_______________________________________________
Pkg-haskell-maintainers mailing list
[email protected]
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-haskell-maintainers

Reply via email to