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

Support {{ compiler('mpi') }} #110

Open
isuruf opened this issue Jun 4, 2022 · 3 comments
Open

Support {{ compiler('mpi') }} #110

isuruf opened this issue Jun 4, 2022 · 3 comments
Labels

Comments

@isuruf
Copy link
Member

isuruf commented Jun 4, 2022

Comment:

This way, we can setup cross compiling env things like OPAL_PREFIX and build_env compiler.
Also, we can remove hacks like https://github.com/conda-forge/conda-smithy/blob/52db47bc3064f2471c1c8927c02d1149014557fa/conda_smithy/configure_feedstock.py#L280

cc @minrk, @beckermr

@isuruf isuruf added the question label Jun 4, 2022
@leofang
Copy link
Member

leofang commented Jun 5, 2022

Sounds like an alternative solution to #91 to me? (xref: conda-forge/conda-forge.github.io#1752) cc: @carterbox

@jakirkham
Copy link
Member

cc @dalcinl

@dalcinl
Copy link
Contributor

dalcinl commented Jun 6, 2022

Perhaps it should be compiler('mpicc'), compiler('mpicxx'), compiler('mpifort')?
Recipes may need only one, or two, or all of them. Depending on the actual semantics of compiler('mpi'), my comment may be irrelevant. Perhaps the idea is that compiler('mpi') alone is meaningless and should error, unless something like compiler('c') is also present.

Or perhaps the compiler(...) thing could be extended such that it could be used the following way:

{{ compiler('c', 'mpi') }}
{{ compiler('cxx', 'mpi') }}
{{ compiler('fortran', 'mpi') }}

Beside these minor details about defining actual semantics, this is a great idea!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants