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

add doc for spack install with external modules #33

Closed
wants to merge 101 commits into from

Conversation

gknapp1
Copy link

@gknapp1 gknapp1 commented Jul 1, 2024

I've added documentation for installing the additivefoam spack module when you are using modules external to spack. This is useful when you need to install OpenFOAM on the system in addition to the AdditiveFOAM solver and you have system modules available to you for common libraries like C++ and MPI.

Copy link
Collaborator

@streeve streeve left a comment

Choose a reason for hiding this comment

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

I could be wrong, but I think this is more general: "spack install with existing compiler/dependencies"

docs/installation.md Show resolved Hide resolved
docs/installation.md Outdated Show resolved Hide resolved
docs/installation.md Outdated Show resolved Hide resolved
@gknapp1
Copy link
Author

gknapp1 commented Jul 2, 2024

I addressed the above comments. It's a good point that this could be more general. One of the other use cases that I didn't think of originally is that if there is a system module for OpenFOAM-10, then you can create a spack package specification to wrap it and still use the spack installation of additivefoam.

I'd guess it will be less straightforward to use a standalone install of OpenFOAM-10 with spack, becuase of the environmental setup that would need to be run when loading the spack module. But I don't know enough about spack to write an example of that right now.

@streeve
Copy link
Collaborator

streeve commented Aug 2, 2024

@colemanjs good to merge?

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.

None yet

3 participants