Skip to content

Fixes for issues 1212 and 1215#1213

Open
LourensVeen wants to merge 5 commits intoamusecode:mainfrom
LourensVeen:issue-1212-mesa-fixes
Open

Fixes for issues 1212 and 1215#1213
LourensVeen wants to merge 5 commits intoamusecode:mainfrom
LourensVeen:issue-1212-mesa-fixes

Conversation

@LourensVeen
Copy link
Member

MESA r2208 and r15140 download a bunch of files on installation, from Zenodo, GitHub, and a university server. That university server does not like wget, and the official work-around is to set an empty User-Agent. So that's what we did, but recently Zenodo has started rejecting that with a 403, causing the MESA installs to be broken.

Additionally, the MESA people rearranged their GitHub repository recently, and since we were downloading from the master branch that broke our download, because the file is now somewhere else.

Finally, there was an issue with newer macOS (or XCode, not sure) not accepting two -WL,rpath= arguments on the command line, even if they add the same path. This tries to fix that as well. While messing around with this, I noticed some missing dependencies in the mesa-r15140 metadata, so I added those.

Bit of a mixed bag this, but we need to get this fixed also because we have a workshop coming up.

@LourensVeen LourensVeen self-assigned this Mar 4, 2026
@LourensVeen LourensVeen added area: building/installing Issue relates to the build system and installing from source area: community_code Issue concerns a community code and/or its AMUSE bindings area: macOS Issue found on macOS, may require a mac to fix labels Mar 4, 2026
@LourensVeen LourensVeen force-pushed the issue-1212-mesa-fixes branch from fbeaeb5 to 173cb8e Compare March 4, 2026 09:01
@LourensVeen LourensVeen requested a review from a team as a code owner March 4, 2026 09:01
@LourensVeen LourensVeen moved this from Backlog to In progress in Development Board Mar 4, 2026
@LourensVeen LourensVeen force-pushed the issue-1212-mesa-fixes branch from 173cb8e to 5316158 Compare March 4, 2026 20:26
@LourensVeen
Copy link
Member Author

Tweaked this a bit, this should now hopefully solve the download and rpath issues of #1212 and the download problem in #1215.

@rieder
Copy link
Member

rieder commented Mar 5, 2026

This is wider than just fixing MESA ;). (in particular the commit changing the configure files)

Copy link
Member

@rieder rieder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems download speed (of mesa-r15140.zip from Zenodo) has become incredibly slow, could that be related to using curl instead of wget?

Copy link
Member

@rieder rieder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apart from the slow download speed (possible regression?), this works for me - so ok to merge.

@github-project-automation github-project-automation bot moved this from In progress to Done in Development Board Mar 5, 2026
@LourensVeen
Copy link
Member Author

Yeah, I started out with the download problem, then realised that the other download problem was related, and also that I wanted to have a single branch with all fixes for #1212 on it for the reporters to test, so that's why this got out of hand a little.

I guess you tested this on your mac? Have you upgraded to Tahoe yet? It seems like the double rpath became an issue either with that or with a newer XCode.

@LourensVeen LourensVeen changed the title MESA fixes for issue 1212 Fixes for issues 1212 and 1215 Mar 5, 2026
@LourensVeen
Copy link
Member Author

I'm getting about 20MB/s from Zenodo using curl, so it looks like the speed problem is transient or location-dependent. At least curl isn't the limiting factor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: building/installing Issue relates to the build system and installing from source area: community_code Issue concerns a community code and/or its AMUSE bindings area: macOS Issue found on macOS, may require a mac to fix

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants