Skip to content

Add LFortran support #78

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

Closed
wants to merge 11 commits into from
Closed

Conversation

wpbonelli
Copy link
Contributor

Builds on #57, opening new PR as I don't have permission to push directly to your branch @gha3mi. Thanks for putting in the main effort here.

I think we can just use the builtin conda on linux/windows and install/use micromamba on mac, and move ahead with this, rather than wait for it on other package managers as I was thinking before. Some more things:

  • this now installs lfortran to the base environment, not a custom one
  • lfortran seems to pair better with MSVC on Windows — install MSVC toolchain with it and set cl as the C/C++ env vars and outputs, still use gcc/g++ on linux and mac though
  • tweaked the testing conditions, there is some issue with lfortran executable discovery on powershell and command prompt, so windows support is limited to bash for now unless it's invoked by abs path
  • updated the test matrix and compatibility files

gha3mi and others added 9 commits March 26, 2024 11:20
* use preinstalled conda on linux and windows
* install/use micromamba on mac
* install lfortran to base env
* update fortran test action
* restore c/c++ test actions
* simplify env var setting
* remove dupe function
@gha3mi
Copy link
Contributor

gha3mi commented Mar 26, 2024

Thanks, @wpbonelli, and sorry about that. I forgot to give permission, but it is possible now.

@wpbonelli
Copy link
Contributor Author

thanks @gha3mi, I will close this

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.

2 participants