<div dir="ltr">On Mon, Jun 24, 2013 at 11:56 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br><div class="im">
> If I am to have Clique build on top of pkg-metis and pkg-parmetis, I will<br>
> need to modify them to install enough header files so that it can make use<br>
> of metislib.h and parmetislib.h.<br>
<br>
</div>So clique requires additional include files? I supporse these are<br>
private include files from metis/parmetis..<br>
<div class="im"><br></div></blockquote><div><br></div><div>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<br>
<a href="https://github.com/poulson/Clique/blob/master/src/metis/Bisect.c">https://github.com/poulson/Clique/blob/master/src/metis/Bisect.c</a><br></div><div>and<br><a href="https://github.com/poulson/Clique/blob/master/src/parmetis/ParallelBisect.c">https://github.com/poulson/Clique/blob/master/src/parmetis/ParallelBisect.c</a><br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="im">
>  I'll try to propose a patch over the next<br>
> few days and include the 'include_directories(headers)' and replace the<br>
> strange 'include(FindMpi); if(NOT MPI_FOUND)' statements with guarded<br>
> 'find_package(MPI); if(NOT MPI_C_FOUND)' statements.<br>
<br>
<br>
</div>Looks like that piece of code is from upstream. It has the comment:<br>
>><br>
# GK commented this out as it seems to be creating problems<br>
<<<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div><br></div><div>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).<br>
<br>Jack<br></div></div></div></div>