Summary: | sci-libs/scipy-1.4.1 fails with gcc-10 - Error: Rank mismatch | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Michelangelo Scopelliti <kernelpanic> |
Component: | Current packages | Assignee: | Gentoo Science Related Packages <sci> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | edes, gabemarcano, gentoo, harrisl, holgersson, mgorny, ondra-a, python, silvio.gerli, strogdon |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://bugs.gentoo.org/show_bug.cgi?id=727590 https://bugs.gentoo.org/show_bug.cgi?id=706426 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | build log (MAKEOPTS="-j1" |
Description
Michelangelo Scopelliti
2020-05-09 13:47:58 UTC
Created attachment 637042 [details]
build log (MAKEOPTS="-j1"
Hi, I can confirm both bug and workaround on default/linux/amd64/17.1/no-multilib/hardened. The workaround works for me (although it's only FFLAGS that should be set as there is no equivalent CFLAGS switch). However, there's a worse problem here. scipy lists sci-libs/arpack as a dependency, but the ebuild uses an internal copy of arpack. sci-libs/arpack has the same bug and workaround for all released versions, including 3.7.0 which is not in the tree). The live version works, though. I have filed a bug with upstream arpack asking for a new release. PR to add the flag to ebuild: https://github.com/gentoo/gentoo/pull/16093 @Joel Berendzen: fwiw, here's a arpack-3.7.0.ebuild that I might PR: https://github.com/acolinisi/casper-ebuilds/blob/master/sci-libs/arpack/arpack-3.7.0.ebuild The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d80e7eafbdb6184a289564ca15d5976d67bf8d17 commit d80e7eafbdb6184a289564ca15d5976d67bf8d17 Author: Alexei Colin <acolin@isi.edu> AuthorDate: 2020-06-06 20:04:51 +0000 Commit: Alexey Shvetsov <alexxy@gentoo.org> CommitDate: 2020-06-07 21:03:31 +0000 sci-libs/scipy: -fallow-argument-mismatch for gfortran v10 Already fixed in upstream master: https://github.com/scipy/scipy/pull/11842 Closes: https://bugs.gentoo.org/721860 Signed-off-by: Alexei Colin <acolin@isi.edu> Closes: https://github.com/gentoo/gentoo/pull/16093 Signed-off-by: Alexey Shvetsov <alexxy@gentoo.org> sci-libs/scipy/scipy-1.4.1.ebuild | 3 +++ 1 file changed, 3 insertions(+) Reverting, this breaks gcc-9. (In reply to Michał Górny from comment #6) > Reverting, this breaks gcc-9. And what do you think about something conditional, like [ $(gcc-major-version) -ge 10 ] && append-fflags -fallow-argument-mismatch using toolchain-funcs? Sure but please figure out how to check gfortran version. (e.g. people may have CC/CXX set to clang) (In reply to Michał Górny from comment #8) > Sure but please figure out how to check gfortran version. In bug #721860 (which you marked FIXED?!?) I already mentioned a solution by replacing: append-fflags -fallow-argument-mismatch with: append-fflags $(test-flag-FC -fallow-argument-mismatch) I was only able to test this on amd64 with gcc-9.3/gcc-10. (In reply to Stephan Hartmann from comment #10) > (In reply to Michał Górny from comment #8) > > Sure but please figure out how to check gfortran version. > > In bug #721860 (which you marked FIXED?!?) I already mentioned a solution by > replacing: > append-fflags -fallow-argument-mismatch > with: > append-fflags $(test-flag-FC -fallow-argument-mismatch) > > I was only able to test this on amd64 with gcc-9.3/gcc-10. Oops, it is actually bug #727590 (In reply to Michał Górny from comment #9) > (e.g. people may have CC/CXX set to clang) Also, other fortran compilers are available. So, maybe, a combination of tc-getFC() and gcc-major-version(): [[ $(tc-getFC) == *gfortran* ]] && [ $(gcc-major-version) -ge 10 ] && append-fflags -fallow-argument-mismatch Is this viable? test-flag-FC sounds like a better solution. According to Github; this may be fixed upstream in the next release as well: https://github.com/scipy/scipy/issues/11611 The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c695c8ece7b823d102964684c5642f17389cba72 commit c695c8ece7b823d102964684c5642f17389cba72 Author: David Seifert <soap@gentoo.org> AuthorDate: 2020-06-16 22:01:58 +0000 Commit: David Seifert <soap@gentoo.org> CommitDate: 2020-06-16 22:01:58 +0000 sci-libs/scipy: Conditionally add -fallow-argument-mismatch Closes: https://bugs.gentoo.org/721860 Package-Manager: Portage-2.3.101, Repoman-2.3.22 Signed-off-by: David Seifert <soap@gentoo.org> sci-libs/scipy/scipy-1.1.0.ebuild | 6 ++++++ sci-libs/scipy/scipy-1.4.1.ebuild | 6 ++++++ 2 files changed, 12 insertions(+) |