-
-
Notifications
You must be signed in to change notification settings - Fork 523
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Recommend Fortran compiler as a prerequisite #27054
Comments
This comment has been minimized.
This comment has been minimized.
comment:1
|
comment:3
Replying to @jdemeyer:
there is one, in fact, by the same author. Using a wrong branch, I presume. |
comment:4
NOTE: |
comment:5
Retarging tickets optimistically to the next milestone. If you are responsible for this ticket (either its reporter or owner) and don't believe you are likely to complete this ticket before the next release (8.7) please retarget this ticket's milestone to sage-pending or sage-wishlist. |
comment:7
I don't like this from the point of Fortran point of view - it is possible to build Sage with e.g. Flang. Do not break this. |
Changed branch from u/galois/mrs/4/gfortran to u/slelievre/mrs/4/gfortran |
Changed keywords from none to gfortran |
This comment has been minimized.
This comment has been minimized.
Changed author from Zerline to Odile Bénassy, Samuel Lelièvre |
comment:9
Rephrased as a recommendation rather than a strict prerequisite. New commits:
|
comment:10
On OS X, for what it's worth, I have installed gfortran (6.4.0) and I routinely build Sage with the built-in version of clang plus this gfortran. So maybe in an ideal world, the version of gfortran should match the C compiler, but it doesn't seem to be strictly necessary. Or have I just been getting lucky? |
comment:11
In more detail:
|
comment:12
Replying to @jhpalmieri:
this is what we routinely do on OSX, so it does not look like a problem to me. |
comment:13
My comment was in response to the proposed change, which includes the line "it is necessary to have matching versions of these two compilers". |
comment:14
Thanks for your feedback on my proposed phrasing in commit a35d77e:
How about the following?
|
comment:15
I'm happy with the revised version. |
comment:16
Ticket retargeted after milestone closed (if you don't believe this ticket is appropriate for the Sage 8.8 release please retarget manually) |
comment:17
Tickets still needing working or clarification should be moved to the next release milestone at the soonest (please feel free to revert if you think the ticket is close to being resolved). |
comment:18
Ticket retargeted after milestone closed |
comment:19
See also #26964, which generates the commands to install recommended system packages. |
Dependencies: #26964 |
comment:21
#26964 is ready and takes care of this. |
Reviewer: Michael Orlitzky |
comment:23
ok, thx |
Recommend Fortran compiler as a prerequisite in the "installing from source" guide.
Depends on #26964
CC: @slel
Component: documentation
Keywords: gfortran
Author: Odile Bénassy, Samuel Lelièvre
Branch/Commit: u/slelievre/mrs/4/gfortran @
a35d77e
Reviewer: Michael Orlitzky
Issue created by migration from https://trac.sagemath.org/ticket/27054
The text was updated successfully, but these errors were encountered: