search

conda-forge / mpich-feedstock

A conda-smithy repository for mpich.
BSD 3-Clause "New" or "Revised" License
2 stars 26 forks source link

add mpich-mpicc,cxx,fort outputs #32

Closed minrk closed 5 years ago

minrk commented 5 years ago

these depend on conda compilers as an easy way to install the mpi compiler and its requirements.

mpich package is unchanged, and actually ships with the mpi compiler wrappers. They just may not work without the compiler packages. They do actually work fine on mac without the conda compilers, but not linux.

closes #31

conda-forge-linter commented 5 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

leofang commented 5 years ago

Hi @minrk thanks for this PR! It'd be very useful as I also encountered situations like #31 recently. I have two questions:

  1. am I right that this recipe generates 4 packages mpich, mpich-mpicc, mpich-mpicxx and mpich-mpifort? If so, can downstream packages (say those that need mpicc) depend only on the corresponding package (mpich-mpicc) and not on the base compiler ({{ compiler('c') }})?
  2. this should probably be an issue in conda-forge/openmpi-feedstock: would there be a PR mirroring this one for openmpi?

Thanks!

minrk commented 5 years ago

Hi @minrk thanks for this PR! It'd be very useful as I also encountered situations like #31 recently. I have two questions:

  1. am I right that this recipe generates 4 packages mpich, mpich-mpicc, mpich-mpicxx and mpich-mpifort? If so, can downstream packages (say those that need mpicc) depend only on the corresponding package (mpich-mpicc) and not on the base compiler ({{ compiler('c') }})?

Yes, the mpich-mpicc package is an empty package that depends on mpich and compiler('c') so that the compiler has its dependencies.

  1. this should probably be an issue in conda-forge/openmpi-feedstock: would there be a PR mirroring this one for openmpi?

Yeah, assuming I get this one to work, it will make sense to do the same in openmpi.