Summary: | dev-ml/parmap-1.0_rc6 build failure during install phase | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Rémi Cardona (RETIRED) <remi> |
Component: | [OLD] Unspecified | Assignee: | Gentoo Team for the ML programming language family <ml> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 526268 | ||
Attachments: |
build.log
files/parmap-1.0_rc6-fix-bashisms.patch parmap-1.0_rc6.ebuild.patch |
Description
Rémi Cardona (RETIRED)
2015-06-29 20:59:07 UTC
please attach the full build log; nothing in the makefile shows that 'install' should call 'uninstall' first :/ (which is what is happening here) Created attachment 406102 [details]
build.log
I have also tried rebuilding the few ocaml libs and programs I have with USE=-ocamlopt, but it still fails the same way. hmmm, can you try with /bin/sh as bash ? Good catch, I completely missed configure almost-failing-but-not-quite. I tried fixing the obvious == into =, but it still fails elsewhere. Rerunning eautoreconf fails due to missing macros (notably AC_PROG_OCAML). I guess the best fix is to call econf with CONFIG_SHELL=/bin/bash (In reply to Rémi Cardona from comment #5) > I tried fixing the obvious == into =, but it still fails elsewhere. > Rerunning eautoreconf fails due to missing macros (notably AC_PROG_OCAML). this is dev-ml/ocaml-autoconf > I guess the best fix is to call econf with CONFIG_SHELL=/bin/bash yep could be the easiest thing to do, but if you/we have an upstreamable fix it's much better :) Ah! Thanks for the pointers, I'll take another swing at it then! Created attachment 406624 [details, diff]
files/parmap-1.0_rc6-fix-bashisms.patch
Here's the upstream patch. The "==" was fairly obvious, but it took me a bit longer to figure out that "&>" isn't kosher in POSIX. This seems to work ok with both bash and dash.
Created attachment 406626 [details, diff]
parmap-1.0_rc6.ebuild.patch
And here's the ebuild patch. I'm not 100% confident about the autotools eclass part, I'm not quite sure about the WANT_* variables. Maybe you'll have a better clue.
great, thanks; could you please send a pull request at https://github.com/rdicosmo/parmap ? for WANT_* variables, they default to 'latest' which is the best choice; those variables are there when packages that require older versions and for which it is not easy to port them Damn! I thought I could weasel out of going to upstream with the patch! :p Anyway: https://github.com/rdicosmo/parmap/pull/36 Cheers + 14 Jul 2015; Alexis Ballier <aballier@gentoo.org> parmap-1.0_rc6.ebuild, + +files/parmap-1.0_rc6-fix-bashisms.patch: + Fix bashisms in configure, bug #553596 by Rémi Cardona. + thanks a lot! |