#998784 comedilib: bashism in configure script

Package:
src:comedilib
Source:
comedilib
Submitter:
Andrej Shadura
Date:
2021-11-07 19:42:03 UTC
Severity:
normal
Tags:
#998784#5
Date:
2021-11-07 19:38:40 UTC
From:
To:
Hi,

Your package uses configure script with bash features not present in
POSIX without explicitly declaring the need to bash shell; this
currently works as configure scripts select bash, but when dash enables
LINENO support, your configure script will start failing:

     ./configure: 14378: test: /usr/bin/swig: unexpected operator
     ./configure: 14392: test: yes: unexpected operator
     ./configure: 14910: test: no: unexpected operator
     checking to see if compiler understands -fno-strict-aliasing... yes
     checking to see if compiler understands -Wno-unused-function... yes
     ./configure: 15000: test: no: unexpected operator
     ./configure: 15068: test: no: unexpected operator
     ./configure: 15096: test: no: unexpected operator
     ./configure: 15115: test: yes: unexpected operator
     configure: WARNING: No hotplug mechanism will be installed. Consult ./configure --help if you want hotplug.
     checking that generated files are newer than configure... done
     configure: creating ./config.status
     config.status: creating Makefile
     config.status: creating comedi_config/Makefile
     config.status: creating comedi_board_info/Makefile
     ...
     checking if libtool supports shared libraries... yes
     checking whether to build shared libraries... yes
     checking whether to build static libraries... yes
     checking for egrep... (cached) /bin/grep -E
     checking for thread local storage (TLS) class... __thread
     checking for swig... /usr/bin/swig
     ./configure: 14378: test: /usr/bin/swig: unexpected operator
     ./configure: 14392: test: no: unexpected operator
     ./configure: 14910: test: no: unexpected operator
     checking to see if compiler understands -fno-strict-aliasing... yes
     checking to see if compiler understands -Wno-unused-function... yes
     ./configure: 15000: test: no: unexpected operator
     ./configure: 15068: test: no: unexpected operator
     ./configure: 15096: test: no: unexpected operator
     ./configure: 15115: test: yes: unexpected operator
     configure: WARNING: No hotplug mechanism will be installed. Consult ./configure --help if you want hotplug.
     checking that generated files are newer than configure... done
     configure: creating ./config.status
     config.status: creating Makefile
     config.status: creating comedi_config/Makefile
     config.status: creating comedi_board_info/Makefile
     config.status: creating c++/Makefile
     ...

To test this, you can install dash from experimental and re-run the
configure script.

Please replace non-POSIX features with their equivalents to make sure
the script runs with dash. Most common ones are usage of == instead of =
and for with arrays (not lists).