Thanks! pushed to maint - and merged to master. Satish
On Thu, 18 Feb 2016, Randall Mackie wrote: > Hi Satish, > > This patch works now. > > Thanks very much to all of you for your patience and help. > > Randy > > > > On Feb 17, 2016, at 9:58 PM, Satish Balay <ba...@mcs.anl.gov> wrote: > > > > Look like zdaindexf.c is not listed in makefile - so it never gets compiled. > > > > Also it has code for dmdagetglobalindices_() dmdarestoreglobalindices_(). > > But then > > src/docs/website/documentation/changes/35.html has: > > > >>>>> > > <li>DMDAGetGlobalIndices(DM,PetscInt*,const PetscInt*[]) and > > DMDARestoreGlobalIndices(DM,PetscInt*,const PetscInt*[]) are removed, use > > DMGetLocalToGlobalMapping() to get this information</li> > > <<<< > > > > So perhaps the attached modified patch should be used? [instead of the > > previous fix-fortran-dmdasetaotype.patch] > > > > patch -Np1 -R < fix-fortran-dmdasetaotype.patch > > patch -Np1 < dmdasetaotype-fixed.patch > > > > Might be easier to use 'maint' branch from git.. > > > > Satish > > > > On Wed, 17 Feb 2016, Barry Smith wrote: > > > >> > >> Randy, > >> > >> It is not suppose to be this hard! When you ran the make gnumake after > >> you applied the patch did it recompile the file zdaindexf.c (in fact that > >> file and daindexf.c should be the only files that changed and hence the > >> only files that got recompiled). > >> > >> Barry > >> > >> > >> > >>> On Feb 17, 2016, at 11:41 PM, Randall Mackie <rlmackie...@gmail.com> > >>> wrote: > >>> > >>> Satish, > >>> > >>> It is unclear from this email exchange if I am still suppose to do make > >>> allfortranstubs after updating these two files, or if these are the files > >>> that are suppose to result from that make operation. > >>> > >>> Although these files looked the same as what I tried before with Barry’s > >>> suggestions, I dropped these in, recompiled petsc, but again, when I try > >>> to compile the test code I sent, it still complains about an unresolved > >>> symbol to dmdasetaotype_. > >>> > >>> If I still need to do the make allfortranstubs, I’ll do that via a git > >>> clone tomorrow. > >>> > >>> Thanks, Randy > >>> > >>> > >>>> On Feb 17, 2016, at 8:54 PM, Satish Balay <ba...@mcs.anl.gov> wrote: > >>>> > >>>> Attached are the modified src/dm/impls/da/ftn-auto/daindexf.c and > >>>> src/dm/impls/da/ftn-custom/zdaindexf.c files. > >>>> > >>>> Satish > >>>> > >>>> On Wed, 17 Feb 2016, Jed Brown wrote: > >>>> > >>>>> Randall Mackie <rlmackie...@gmail.com> writes: > >>>>> > >>>>>> this leads to the error ‘bin/maint/generatefortranstubs.py’ …No such > >>>>>> file. > >>>>>> > >>>>>> there is no maint directory under bin. > >>>>> > >>>>> Ah, I missed that you have a tarball, not a clone of the repository. > >>>>> Then you'll have to fix the file manually. I would start from a clean > >>>>> tarball because we have no idea what state your source tree is in after > >>>>> applying the match and manually editing files. Note that this would be > >>>>> easy and completely reliable if you used a clone instead of tarball. > >>>>> > >>>> <daindexf.c><zdaindexf.c> > >>> > >> > > <dmdasetaotype-fixed.patch> > >
