Created attachment 703482 [details, diff]
isl's configure.ac contains Bashisms (use of == operator with 'test' built-in). Thus, configure emits errors when /bin/sh is not Bash. This doesn't cause the build to fail outright, but it may be causing misdetection of features.
AM_CONDITIONAL(SMALL_INT_OPT, test "x$with_int" == "ximath-32")
AS_IF([test "x$with_int" == "ximath-32"], [
/var/tmp/portage/dev-libs/isl-0.23-r1/work/isl-0.23/configure: 20657: test: xgmp: unexpected operator
/var/tmp/portage/dev-libs/isl-0.23-r1/work/isl-0.23/configure: 20665: test: xgmp: unexpected operator
The attached patch removes the Bashisms and allows dev-libs/isl-0.23-r1 to configure without error when /bin/sh is Dash.
Note that such a fix would therefore ideally be accompanied with a revbump.
Can you send the patch upstream as well?
Should be a matter of sending patch to https://groups.google.com/g/isl-development
The bug has been closed via the following commit(s):
Author: Sergei Trofimovich <email@example.com>
AuthorDate: 2021-05-02 21:31:09 +0000
Commit: Sergei Trofimovich <firstname.lastname@example.org>
CommitDate: 2021-05-02 21:31:19 +0000
dev-libs/isl: bump up to 0.24
While at it applied bashism fix by Matt Whitlock.
Fix-by: Matt Whitlock
Package-Manager: Portage-3.0.18, Repoman-3.0.3
Signed-off-by: Sergei Trofimovich <email@example.com>
dev-libs/isl/Manifest | 1 +
dev-libs/isl/files/isl-0.24-nobash.patch | 35 ++++++++++++++++
dev-libs/isl/isl-0.24.ebuild | 71 ++++++++++++++++++++++++++++++++
3 files changed, 107 insertions(+)
(In reply to Sergei Trofimovich from comment #2)
> Can you send the patch upstream as well?
> Should be a matter of sending patch to
I sent on your behalf as https://groups.google.com/g/isl-development/c/GOmQqKVDpRk. I hope it's ok.