#942367 openipmi: Bashisms in init script

Package:
src:openipmi
Source:
openipmi
Submitter:
Sam Morris
Date:
2021-05-24 21:15:45 UTC
Severity:
normal
#942367#5
Date:
2018-11-06 15:05:34 UTC
From:
To:
/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).
#942367#10
Date:
2020-07-08 02:00:37 UTC
From:
To:
Control: merge -1 920461

Thanks for the report.  This bug can be merged with 920461.  I'm
preparing a release to address them.

Thanks,
#942367#15
Date:
2021-05-24 21:11:50 UTC
From:
To:
Greeting to you once again



I am writing for the second time as I did not receive any reply from you to
my previous message.



Kindly reply so we can discuss the subject as soon as possible.



Sincerely,



Andy Landman