I'm sorry it took so long -- I finally fixed this on the trunk and have
scheduled this for the v1.8 branch. There were a small number of functions in
the tkr interface that had ierr instead of ierror (some of the Dist_graph
functions), which were probably added after the fixes were applied a year ago.
Did you find any other places where we accidentally had ierr instead of ierror?
On May 15, 2014, at 7:35 PM, Ralph Castain <r...@open-mpi.org> wrote:
> you might try the nightly 1.8.2 build - there were some additional patches to
> fix the darned tkr support. I'm afraid getting all the various compilers to
> work correctly with it has been a major pain.
>
>
> On May 15, 2014, at 5:01 PM, W Spector <w...@earthlink.net> wrote:
>
>> Hi Jeff and the list,
>>
>> A year ago, we had the discussion appended below. I just downloaded v1.8.1
>> and the F90 module is still very broken. And once again I am having to
>> modify my local version. (+1 for open source!) Will it be fixed in v1.8.2?
>>
>> Configure is using the "use-mpi-tkr" version on my system. I can see that
>> the "use-mpi-f08" version is much better.
>>
>> Walter
>>
>> On 04/26/2013 03:14 PM, Jeff Squyres (jsquyres) wrote:
>>> I committed that part; thanks.
>>>
>>> On Apr 26, 2013, at 5:51 PM, W Spector <w...@earthlink.net> wrote:
>>>
>>>> Hi Jeff,
>>>>
>>>> To take care of the ierr->ierror conversion, simply do the following:
>>>>
>>>> cd openmpi-1.7.1/ompi/mpi/fortran/use-mpi-tkr/scripts
>>>> ls -1 *.sh | xargs -i -t ex -c ":1,\$s?ierr?ierror?" -c ":wq" {}
>>>>
>>>> Then go up a level to openmpi-1.7.1/ompi/mpi/fortran/use-mpi-tk and use:
>>>>
>>>> cd ..
>>>> ls -1 fort*.in | xargs -i -t ex -c ":1,\$s?ierr?ierror?" -c ":wq" {}
>>>>
>>>> Last, the use-mpi-ignore-tkr directory:
>>>>
>>>> cd ../use-mpi-ignore-tkr
>>>> ls -1 mpi*.in | xargs -i -t ex -c ":1,\$s?ierr?ierror?" -c ":wq" {}
>>>>
>>>> As you can tell from the below, I needed to use a few MPI_Type calls. So
>>>> fixed the few that I needed in the
>>>> openmpi-1.7.1/ompi/mpi/fortran/use-mpi-tkr/scripts directory. I didn't
>>>> exhaustively go through and verify every interface in the whole MPI
>>>> library.
>>>>
>>>> Walter
>>>>
>>>> On 04/26/2013 11:53 AM, Jeff Squyres (jsquyres) wrote:
>>>>> On Apr 25, 2013, at 10:52 PM, W Spector <w...@earthlink.net> wrote:
>>>>> ...
>>>>>> I went into the openmpi-1.7.1/ompi/mpi/fortran/use-mpi-tkr/scripts
>>>>>> directory and modified the files to use ierror instead of ierr. (One
>>>>>> well-crafted line of shell script.) Did the same with a couple of .h.in
>>>>>> files in the use-mpi-tkr and use-mpi-ignore-tkr directories, and
>>>>>> use-mpi-tkr/attr_fn-f90-interfaces.h.in. (One editor command each.)
>>>>>>
>>>>>> With the above, the mpi module is in much better shape. However there
>>>>>> are still some scattered incorrect non-ierror argument names. A few
>>>>>> examples from the code I am working with:
>>>>>>
>>>>>> MPI_Type_create_struct: The 2nd argument should be
>>>>>> "array_of_blocklengths", instead of "array_of_block_lengths"
>>>>>>
>>>>>> MPI_Type_commit: "datatype" instead of "type"
>>>>>>
>>>>>> MPI_Type_free: Again, "datatype" instead of "type"
>>>>>>
>>>>>> There are more...
>>>>>
>>>>> Cool. Any chance you could send us a patch?
>>>>>
>>>> _______________________________________________
>>>> users mailing list
>>>> us...@open-mpi.org
>>>> http://www.open-mpi.org/mailman/listinfo.cgi/users
>>>
>>>
>> _______________________________________________
>> users mailing list
>> us...@open-mpi.org
>> http://www.open-mpi.org/mailman/listinfo.cgi/users
>
> _______________________________________________
> users mailing list
> us...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/users
--
Jeff Squyres
jsquy...@cisco.com
For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/
