<div dir="ltr">On Mon, Jun 24, 2013 at 11:31 AM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
</div></div>Looks like parmetis.py is using the cmake option<br>
'-DGKLIB_PATH=../headers' to set the path to GKlib.h<br>
<br>
[I'm not sure if this is prefereable to changing<br>
pkg-parmetis/CMakeLists.txt as you've indicated]<br>
<div class="im"><br></div></blockquote><div><br></div><div>Thanks, Satish. I would strongly lean towards having pkg-metis/pkg-parmetis be a bit more intuitive to build as standalone packages (which is to say, I think it makes sense to have the include_directories(headers) statement). It is not obvious that one is expected to:<br>
</div><div>1) Install metis<br></div><div>2) Set METIS_PATH to the installation directory<br></div><div>3) Set GKLIB_PATH to ./headers<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
> Is PETSc using these CMakeLists.txt files? I can now build pkg-parmetis on<br>
> top of a pkg-metis install if I make the above change (and uncomment out<br>
> lines 9-13 of pkg-parmetis/CMakeLists.txt and actually search for MPI).<br>
<br>
</div>Hm - PETSc configure sets up MPI correctly before building<br>
parmetis. So it should't automatically be searching for MPI.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>A compromise might be to only perform the search if the requisite MPI variables aren't already defined.<br><br></div><div>
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. 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.<br>
<br>Jack<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
Satish<br>
<br>
</font></span></blockquote></div><br></div></div>