On Dec 10, 2013, at 10:42 AM, Dave Love <d.l...@liverpool.ac.uk> wrote:
> This doesn't seem to have been fixed, and I think it's going to bite > here. Is this the right change? Thanks for reminding us. > --- openmpi-1.6.5/ompi/config/ompi_setup_mpi_fortran.m4~ 2012-04-03 > 15:30:24.000000000 +0100 > +++ openmpi-1.6.5/ompi/config/ompi_setup_mpi_fortran.m4 2013-12-10 > 12:23:54.232854527 +0000 > @@ -127,8 +127,8 @@ > AC_MSG_RESULT([skipped (no Fortran bindings)]) > else > bytes=`expr 4 \* $ac_cv_sizeof_int + $ac_cv_sizeof_size_t` > - num_integers=`expr $bytes / $OMPI_SIZEOF_FORTRAN_INTEGER` > - sanity=`expr $num_integers \* $OMPI_SIZEOF_FORTRAN_INTEGER` > + num_integers=`expr $bytes / $ac_cv_sizeof_int` > + sanity=`expr $num_integers \* $ac_cv_sizeof_int` I think this is right, but it is has different implications for different series: 1. No more releases are planned for the v1.6 series. We can commit this fix over there, and it will be available via nightly tarballs. There are also ABI implications -- see #2, below. 2. This fix changes the ABI for the 1.5/1.6 and 1.7/1.8 series (separately, of course). As such, we will need to make this a non-default configure option. E.g., only do this new behavior if --enable-abi-breaking-fortran-status-i8-fix is specified (or some name like that). By default, we have to keep the ABI for the entire 1.5/1.6 and 1.7/1.8 series -- so if you specify this switch, you acknowledge that you're breaking ABI for the -i8 case. 3. For the v1.9 series (i.e., currently the SVN trunk), we can make this be the default, and the --enable-abi-breaking... switch will not exist. Sound ok? -- Jeff Squyres jsquy...@cisco.com For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/