Skip to content
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

Building gfortran fails on GNU/Linux with Apple silicon #38424

Closed
2 tasks done
aikrahguzar opened this issue Jul 25, 2024 · 7 comments · Fixed by #38442
Closed
2 tasks done

Building gfortran fails on GNU/Linux with Apple silicon #38424

aikrahguzar opened this issue Jul 25, 2024 · 7 comments · Fixed by #38442

Comments

@aikrahguzar
Copy link
Contributor

Environment

- **OS**: Fedora Asahi Remix 40
- **Sage Version**: SageMath version 10.5.beta0, Release Date: 2024-07-24

Steps To Reproduce

I ran on a Macbook air M2 running Fedora Asahi Linux,

./configure
make build

Config log

config.log.txt

Package logs

gfortran-13.2.0.log.txt

Additional Information

The error seems to be due to the fact that the build process tries to install darwin version of the gfrotran and not the linux version.

I can workaround this by installing gfrotran from fedora repos. However that runs into issue with installing tachyon #23712 (comment)

[Aside] I have a working installation of sage from nixpkgs. I wonder how they are getting around the tachyon issue.

Checklist

  • I have searched the existing issues for a bug report that matches the one I want to file, without success.
  • I have read the documentation and troubleshoot guide
@dimpase
Copy link
Member

dimpase commented Jul 25, 2024

Sage should leave building Fortran to experts.
Can't you use gfortran from the OS distribution?

@dimpase
Copy link
Member

dimpase commented Jul 25, 2024

tachyon issue is a different one. for one, tachyon doesn't use Fortran.

That is, just use gfortran from OS.

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 25, 2024

Sage should leave building Fortran to experts.

Thanks very much, this package is in the hand of an expert.

@aikrahguzar
Copy link
Contributor Author

Sage should leave building Fortran to experts. Can't you use gfortran from the OS distribution?

Installing the fedora package works.

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 28, 2024

@aikrahguzar
Copy link
Contributor Author

aikrahguzar commented Jul 28, 2024

@aikrahguzar Please try with

* [`build/pkgs/gcc`: Update to 13.3 #38442](https://github.com/sagemath/sage/pull/38442)

Thanks a lot. I checked out that commit, uninstalled the fedora gfortran and then ran make build again. gfortran was built successfully and now the build has moved on to openblas. So the original issue seems fixed.

Edit: The build itself fails while building tachyon but that is an unrelated issue and has been fixed separately.

@mkoeppe
Copy link
Contributor

mkoeppe commented Aug 3, 2024

Thanks for testing!

vbraun pushed a commit to vbraun/sage that referenced this issue Aug 9, 2024
    
<!-- ^ Please provide a concise and informative title. -->
<!-- ^ Don't put issue numbers in the title, do this in the PR
description below. -->
<!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method
to calculate 1 + 2". -->
<!-- v Describe your changes below in detail. -->
<!-- v Why is this change required? What problem does it solve? -->
<!-- v If this PR resolves an open issue, please link to it here. For
example, "Fixes sagemath#12345". -->

- Fixes sagemath#38424

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [ ] The description explains in detail what this PR is about.
- [ ] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation and checked the documentation
preview.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#38442
Reported by: Matthias Köppe
Reviewer(s):
@mkoeppe mkoeppe added this to the sage-10.5 milestone Aug 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants