[Dune] [#599] dune-common does not recognize MVAPICH2

[Christian Engwer]

> Unfortunately, there is still a minor problem. I tried reopening the  
> issue in FlySpray,

Sorry, I reopend te tesk now.

> but only got an SQL command thrown back at me. So I'll just post on  
> the list:
> 
> You probably tested this with the current version of MVAPICH2, version  
> 1.4.
> That version defines the macro MVAPICH2_VERSION, but MVAPICH2 1.2,
> which is installed here in Stuttgart, defines MVAPICH_VERSION (not  
> very intuitive,
> but that's the way it is...). I checked the sources of MVAPICH1 1.1,  
> and there is no
> MVAPICH_VERSION macro there either (they still use the MPICH_VERSION  
> macro),
> so the presence of MVAPICH_VERSION is a clear indicator for MVAPICH2  
> 1.2.
> 
> The following patch made the detection work with MVAPICH2 1.2:
> 
> diff --git a/m4/mpi-config.m4 b/m4/mpi-config.m4
> index c436bbc..bad1c23 100644
> --- a/m4/mpi-config.m4
> +++ b/m4/mpi-config.m4
> @@ -215,6 +215,16 @@ test_mvapich2() {
>   #define _OSU_MVAPICH_
>   #include <mpi.h>
>   #include <stdio.h>
> +
> +/* MVAPICH2_VERSION is only defined for MVAPICH2 1.4+
> + * MVAPICH_VERSION is only defined for MVAPICH2 1.2.*
> + * We can thus fall back to MVAPICH_VERSION if MVAPICH2_VERSION
> + * is not defined.
> + */
> +#ifndef MVAPICH2_VERSION
> +#define MVAPICH2_VERSION MVAPICH_VERSION
> +#endif
> +
>   int main() { printf("%s\n",MVAPICH2_VERSION); return 0; }
>   _EOF
> 
> Again, thank you for fixing this so quickly!

I can apply the patch. Does the parameter extraction work?

Actually the latest mvapich2 was recognized as mpich2. This wouldn't
be a big poblem either.

If the new patch works I'll close the task.

Cheers Christian