<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 4, 2013 at 6:16 PM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1j0">Why is there a    #requirespackage 'PETSC_HAVE_PCBDDC'   ?<br>
<br>
Having things like this means that when someone is updating code globally changes to locations like these do not get tested before being pushed and hence is likely broken.<br>
<br>
   That directory appears to depend on PARMETIS so should have that dependency listed in the makefile instead</div></blockquote></div><br>IIRC, the code does not work with complex either. It was put into the repository mostly because we don't have a good system for supporting forks. It was off by default because the code was unstable and untested. There is an example now (src/ksp/ksp/examples/tutorials/ex59.c) so I think we can remove the --with-pcbddc option. Stefano, does this sound right to you?</div>
</div>