>
> I'm surprised that shellcheck didn't complain about lines 26, 58, 73,
> 96, 98, 141 (of the patch, don't know its correspondences with the code).
Most of those are inside `[[` tests, so they don't require quoting (word
splitting doesn't happen inside `[[`). The other case is a for loop,
Sorry I'm sending yet another email, but be careful with tabs/spaces,
you are mixing them in the patch. Many lines are not actually changed
except for the mix, that is not nice.
I'm surprised that shellcheck didn't complain about lines 26, 58, 73,
96, 98, 141 (of the patch, don't know its
Roger! But don't trust the tests, there're lots missing :P
BTW, the submission format here is using git-email or something like
that, meaning the patch goes inline w/ the email.
Cheers!
On 5/18/19 7:18 PM, ಚಿರಾಗ್ ನಟರಾಜ್ wrote:
> Copy that!
>
> I fixed many (but not all) of the warnings. Two we
Copy that!
I fixed many (but not all) of the warnings. Two we can't _really_ do anything
about (the SC1090s) and two I'm not quite sure how to fix. After my
modifications, it passes all the tests, which is a good sign, but there's
definitely a chance I misinterpreted something in the code.
Well, you might be better of checking out from master and running
shellcheck on that source, then commiting a patch :)
Cheers!
On 5/17/19 7:18 PM, ಚಿರಾಗ್ ನಟರಾಜ್ wrote:
> Hey all,
>
> So I did a cursory search for "shellcheck" on the mailing list archives and
> only found one reference
>