#30067 closed defect (fixed)
gfortran spkgconfigure: Reject versions >= 10
Reported by:  mkoeppe  Owned by:  

Priority:  blocker  Milestone:  sage9.2 
Component:  porting  Keywords:  
Cc:  dimpase, bump, mjo, jhpalmieri, fbissey  Merged in:  
Authors:  Matthias Koeppe  Reviewers:  Dima Pasechnik 
Report Upstream:  N/A  Work issues:  
Branch:  f6b123f (Commits, GitHub, GitLab)  Commit:  
Dependencies:  Stopgaps: 
Description (last modified by )
(from https://groups.google.com/d/msg/sagedevel/OXOmOjlYQLo/7gOLtzsJCQAJ)
We already reject gcc 10, but if gfortran is installed separately, we also have to check that version.
We also check for gfortran9
first, and on homebrew
, we recommend installing it via brew install gcc@9
.
Change History (24)
comment:1 Changed 23 months ago by
 Description modified (diff)
comment:2 Changed 23 months ago by
 Cc mjo added
comment:3 Changed 23 months ago by
comment:4 Changed 23 months ago by
It can be overridden by ./configure withoutsystemgfortran
.
comment:5 Changed 23 months ago by
If Homebrew is detected, should we check for gfortran9
and recommend installing it via brew install gcc@9
?
comment:6 Changed 22 months ago by
I hope that we can get #29456 working...
comment:7 Changed 22 months ago by
 Branch set to u/mkoeppe/gfortran_spkg_configure__reject_versions____10
comment:8 Changed 22 months ago by
 Cc jhpalmieri added
 Commit set to 2ee7c310b68e7f314e8583e8aee6ee6d989f9ee6
 Description modified (diff)
 Priority changed from critical to blocker
 Status changed from new to needs_review
comment:9 Changed 22 months ago by
where is the list
AC_PROG_FC([gfortran9 gfortran g95 xlf95 f95 fort ifort ifc efc pgfortran pgf95 lf95 ftn nagfor xlf90 f90 pgf90 pghpf epcf90 g77 xlf f77 frt pgf77 cf77 fort77 fl32 af77])
coming from? I know that Sage sort of works with flang
, which is not on the list not sure if g95
and other old stuff listed still works...
comment:10 followup: ↓ 14 Changed 22 months ago by
I put "gfortran9" in front of the autoconf 2.69 default
comment:11 followup: ↓ 12 Changed 22 months ago by
by the way, a new Autotools beta version has been released, they promise 2.70 in 3 months. Time to submit bugs we know?
comment:12 in reply to: ↑ 11 Changed 22 months ago by
Replying to dimpase:
by the way, a new Autotools beta version has been released, they promise 2.70 in 3 months. Time to submit bugs we know?
Once the beta is available in Gentoo (https://bugs.gentoo.org/732648) I'll start building sage with it to make sure there are no new problems.
comment:13 Changed 22 months ago by
 Commit changed from 2ee7c310b68e7f314e8583e8aee6ee6d989f9ee6 to f6b123f2bb70f27529526eaed2637eeab42ea7f0
Branch pushed to git repo; I updated commit sha1. New commits:
f6b123f  Add gcc@9/9.3.0 stuff to LIBARY_PATH

comment:14 in reply to: ↑ 10 Changed 22 months ago by
Replying to mkoeppe:
I put "gfortran9" in front of the autoconf 2.69 default
could you add flang
and lfortran
there too, just in case?
comment:15 Changed 22 months ago by
That's not within the scope of this ticket.
comment:17 Changed 22 months ago by
comment:18 Changed 22 months ago by
this can be seen in action for localhomebrewmacosstandard
at https://github.com/mkoeppe/sage/runs/878896230
 Checking whether SageMath should install SPKG gfortran... checking the version of gfortran9... 9.3.0 configure: will use system package and not install SPKG gfortran 
comment:19 Changed 22 months ago by
... and for fedora32standard
at https://github.com/mkoeppe/sage/runs/878897518
Checking whether SageMath should install SPKG gfortran... checking the version of gfortran... 10 configure: Installing gfortran because gfortran is version 10, which is too recent for this version of Sage configure: no suitable system package found for SPKG gfortran
(though it does not really make a difference because it had already discovered that gcc is too new)
comment:20 Changed 22 months ago by
 Reviewers set to Dima Pasechnik
 Status changed from needs_review to positive_review
lgtm
comment:21 Changed 22 months ago by
Thanks!
comment:22 Changed 22 months ago by
 Branch changed from u/mkoeppe/gfortran_spkg_configure__reject_versions____10 to f6b123f2bb70f27529526eaed2637eeab42ea7f0
 Resolution set to fixed
 Status changed from positive_review to closed
comment:23 Changed 22 months ago by
 Commit f6b123f2bb70f27529526eaed2637eeab42ea7f0 deleted
just make sure it is possible to override with
hmm, should it be with
I am confused here