Jack Poulson writes: > On Mon, Jun 24, 2013 at 11:56 AM, Satish Balay <[email protected]> wrote: > >> >> > If I am to have Clique build on top of pkg-metis and pkg-parmetis, I will >> > need to modify them to install enough header files so that it can make >> use >> > of metislib.h and parmetislib.h. >> >> So clique requires additional include files? I supporse these are >> private include files from metis/parmetis.. >> >> > Yes, I wanted nodal bisection routines, and (par)metis only provides a > restrictive nested dissection routine. I therefore implemented sequential > and parallel versions using metis and parmetis's internal APIs. See > https://github.com/poulson/Clique/blob/master/src/metis/Bisect.c > and > https://github.com/poulson/Clique/blob/master/src/parmetis/ParallelBisect.c > > >> > I'll try to propose a patch over the next >> > few days and include the 'include_directories(headers)' and replace the >> > strange 'include(FindMpi); if(NOT MPI_FOUND)' statements with guarded >> > 'find_package(MPI); if(NOT MPI_C_FOUND)' statements. >> >> >> Looks like that piece of code is from upstream. It has the comment: >> >> >> # GK commented this out as it seems to be creating problems >> << >> >> > Agreed. But the reason it is commented out might be because of incorrect > usage of the CMake script. From the point of view of someone just trying to > build parmetis, it helps to have it there (whether optional or not).
Jack, I can't guarantee I'll work on this soon but since I need to update metis / parmetis in macports, I will eventually be dealing with the cmake issues you've brought up. For the next week or two, I'm pretty busy but I'll try to find some time soon.
