Control: merge -1 920461 On Tuesday, November 06 2018, Sam Morris wrote:
> /etc/init.d/openipmi uses /bin/sh but uses some features that require
> bash:
>
> $ checkbashisms debian/openipmi.init
> possible bashism in debian/openipmi.init line 55 (should be 'b = a'):
> if [ "${kernel}" == "2.4" ]; then
> possible bashism in debian/openipmi.init line 202 ($"foo" should be
> eval_gettext "foo"):
> log_begin_msg $"Starting ipmi_watchdog driver: "
> possible bashism in debian/openipmi.init line 213 ($"foo" should be
> eval_gettext "foo"):
> log_begin_msg $"Stopping ipmi_watchdog driver: "
> possible bashism in debian/openipmi.init line 245 (should be 'b = a'):
> if [ "${IPMI_POWERCYCLE}" == "yes" ]; then
>
> Per policy sec. 10.4:
>
> If a shell script requires non-POSIX.1-2017 features from the shell
> interpreter other than those listed above, the appropriate shell
> must be specified in the first line of the script (e.g.,
> #!/bin/bash) and the package must depend on the package providing
> the shell (unless the shell package is marked “Essential”, as in the
> case of bash).
Thanks for the report. This bug can be merged with 920461. I'm
preparing a release to address them.
Thanks,
--
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF 31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
https://sergiodj.net/
signature.asc
Description: PGP signature
