Created attachment 637816 [details, diff] icedtea-3.16.0-acinclude-posix-conformance.patch Alexander Tsoy reports in bug 526200 that icedtea-3.16.0 fails to build unless /bin/sh links to bash. Attached is a patch that should resolve the issue. A revised ebuild will be temporarily available at https://bitbucket.org/kerframil/portage-overlay/src/master/dev-java/icedtea/. The originally given error message was as follows, implying that it was tested with dash as /bin/sh. checking if there is a ccache gcc wrapper on the PATH... ./configure: 12319: ./configure: Syntax error: redirection unexpected
(In reply to Kerin Millar from comment #0) > Created attachment 637816 [details, diff] [details, diff] > icedtea-3.16.0-acinclude-posix-conformance.patch > > Alexander Tsoy reports in bug 526200 that icedtea-3.16.0 fails to build > unless /bin/sh links to bash. Attached is a patch that should resolve the > issue. A revised ebuild will be temporarily available at > https://bitbucket.org/kerframil/portage-overlay/src/master/dev-java/icedtea/. > The originally given error message was as follows, implying that it was > tested with dash as /bin/sh. > > checking if there is a ccache gcc wrapper on the PATH... ./configure: 12319: > ./configure: Syntax error: redirection unexpected Could you repost the patch over there? *** This bug has been marked as a duplicate of bug 526200 ***
This is not a duplicate. The other bug concerned a different issue in a version of icedtea that is no longer carried by Gentoo and, as such, should be closed as WONTFIX. The only thing that they had in common was they they both blocked "nonbash". I don't think that is a valid reason to refrain from opening a new bug.
Apologies, Sam. Apparently I'm going blind and got it mixed up with a different bug entirely. I shall do as requested.
Ah, I'm not going blind. You changed the issue title. In that case, I stand by my previous post.
(In reply to Kerin Millar from comment #3) > Apologies, Sam. Apparently I'm going blind and got it mixed up with a > different bug entirely. I shall do as requested. I did just rename the bug, so maybe that was it. I just figured keep the information in the same one. Anyway, sorry for the confusion, and thank you for the patch.
*** Bug 722536 has been marked as a duplicate of this bug. ***
Created attachment 639886 [details, diff] patch to change Bash pattern substition in Makefile to POSIX-sh-compatible way I also need this patch to build with /bin/sh -> dash. I have not submitted it upstream; do so if you like. My modifications are licensed under the same license(s) as the original code, of course.
Created attachment 639888 [details, diff] expands brace expansion (Bashism) in Makefile to allow installation with non-bash POSIX sh And I also need this patch to build with /bin/sh -> dash. I have not submitted it upstream; do so if you like. My modifications are licensed under the same license(s) as the original code, of course.
Good find, Nico. The name of my patch is a misnomer as there are various other POSIX violations, such as the use of readlink. Still, weeding out the bashisms is the more important thing for Gentoo.
I confirm that applying the three attached patches (plus adding "inherit autotools" and "src_prepare() { eautoreconf }" to the ebuild) allow dev-java/icedtea-3.16.0 to configure without error on my ~amd64 system, which has Dash as /bin/sh.
Still happening with latest icedtea in portage. Can we have this finally fixed please?
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0f3df0a7b04dcb6f44632e872fdd252c7fe6bb25 commit 0f3df0a7b04dcb6f44632e872fdd252c7fe6bb25 Author: Georgy Yakovlev <gyakovlev@gentoo.org> AuthorDate: 2020-11-02 19:27:21 +0000 Commit: Georgy Yakovlev <gyakovlev@gentoo.org> CommitDate: 2020-11-02 19:27:21 +0000 dev-java/icedtea: force bash for ./configure Bug: https://bugs.gentoo.org/722292 Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Georgy Yakovlev <gyakovlev@gentoo.org> dev-java/icedtea/icedtea-3.17.0.ebuild | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
I've added a workaround for now to use bash. After patch is upstreamed we can remove it.
Reported upstream: https://icedtea.classpath.org/bugzilla/show_bug.cgi?id=3817.
Package removed.
