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
