search

conda-forge / msmpi-feedstock

A conda-smithy repository for msmpi.
BSD 3-Clause "New" or "Revised" License
1 stars 7 forks source link

Need more tests in the recipe #3

Closed leofang closed 3 years ago

leofang commented 3 years ago

Blocked by #2.

I'm leaving a note for what I noticed. Currently, the recipe here only tests if the built files are in the right place or not. But the two other MPI packages in Conda-Forge have additional tests, such as actually building and running a simple MPI hello world program:

so to get the same level of quality guarantee we owe to do the same thing.

However, running MS-MPI programs is currently not possible due to #2. So perhaps testing compiling an MPI program is enough for the time being?

leofang commented 3 years ago

In #6 I added simple tests for compiling and running MPI programs. If someone is interested the C++ and Fortran helloworld tests can be migrated from those repos to here, but I don't think it's a must, so closing this issue.