opencollab / arpack-ng

Collection of Fortran77 subroutines designed to solve large scale eigenvalue problems.
Other
290 stars 124 forks source link

configure.ac: fix bashisms #441

Closed thesamesam closed 11 months ago

thesamesam commented 1 year ago

Pull request purpose

configure scripts need to be runnable with a POSIX-compliant /bin/sh.

On many (but not all!) systems, /bin/sh is provided by Bash, so errors like this aren't spotted. Notably Debian defaults to /bin/sh provided by dash which doesn't tolerate such bashisms as '=='.

This retains compatibility with bash.

Fixes configure warnings/errors like:

checking for cheev_ in -llapack... yes
./configure: 8590: test: x: unexpected operator
checking how to print strings... printf

Detailed changes proposed in this pull request

fghoussen commented 11 months ago

Thanks!

thesamesam commented 11 months ago

Thank you!