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

Make install script executable #965

Merged

Conversation

everythingfunctional
Copy link
Member

This has been broken for a while. Finally getting around to fixing it.

@everythingfunctional
Copy link
Member Author

Should I bother worrying about that one CI failure? It looks like maybe an MSYS package changed at some point?

@perazz
Copy link
Contributor

perazz commented Oct 12, 2023

It says that MSYS2 cannot install gcc-fortran, maybe something's changed in MSYS?

@vmagnin
Copy link
Member

vmagnin commented Oct 12, 2023

It says that MSYS2 cannot install gcc-fortran, maybe something's changed in MSYS?

See " gcc: conflict with gcc-fortran - So pacman suggests removing the later":
msys2/MSYS2-packages@4cda67a
I don't know if that means that installing the gcc package will also install gfortran.

@everythingfunctional
Copy link
Member Author

See " gcc: conflict with gcc-fortran - So pacman suggests removing the later":
msys2/MSYS2-packages@4cda67a
I don't know if that means that installing the gcc package will also install gfortran.

Good catch. Just removing the gcc-fortran package let it get farther. Now it's saying it can't find MS-MPI in the very last step:

<ERROR> *cmd_build* Model error: MS-MPI error: MS-MPI Runtime directory is missing. check environment variable %MSMPI_BIN% or that the folder is in %PATH%.

@perazz
Copy link
Contributor

perazz commented Oct 12, 2023

Nice job @everythingfunctional. The latter problem has been dangling for some time and I think we can live with it as my previous attempts to fix it (#924 #922 (comment)) have failed. Most windows-latest runners work, but some don't. They do not pass %MSMPI_BIN% to the image, so when fpm looks for all possible MPI locations, it doesn't find any. Typically, running the job again works, but that's clearly not a solution. I don't have an answer on how this could be solved though.

@everythingfunctional
Copy link
Member Author

Thanks for the background @perazz . Seeing that, I'm going to plan on merging. I'll wait till later today to give anybody an opportunity to object.

@perazz
Copy link
Contributor

perazz commented Oct 12, 2023

I agree @everythingfunctional. Looks good, as far as I'm concerned.

@everythingfunctional everythingfunctional merged commit 4c1a93f into fortran-lang:main Oct 12, 2023
14 of 15 checks passed
@everythingfunctional everythingfunctional deleted the fix-install-script branch October 12, 2023 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants