#1032181 checkbashisms: replace with shellchek -s dash?

Package:
devscripts
Source:
devscripts
Description:
scripts to make the life of a Debian Package maintainer easier
Submitter:
Gioele Barabucci
Date:
2023-04-05 09:57:03 UTC
Severity:
normal
#1032181#5
Date:
2023-03-01 09:23:48 UTC
From:
To:
Dear devscripts maintainers,

what do you think of replacing checkbashisms with a wrapper over
`shellcheck -s dash`?

shellcheck has evolved to perform all the checks that checkbashisms does
and many others. A cursory review of the open bugs against checkbashisms
that report false positives or false negatives shows that pretty much
all of them are non present in shellcheck (e.g. #531326, #609765,
#807278, #994718, ...).

Shellcheck also provides more comprehensive explanations and its use of
codified error messages (e.g. SC3052) allows users to quickly find
relevant online discussions. (See #556113, #856051.)

Relying on checkbashims also gives developers a false sense of security
because it lacks many of the more minute checks performed by shellcheck.

Regards,

#1032181#12
Date:
2023-04-05 09:54:04 UTC
From:
To:
Hi Gioele,

replacing checkbashisms with `shellchek -s dash` is a great idea. I plan
to do that after the Debian bookworm release. Patches or merge request
for this change are welcome.