On Sun, Jan 1, 2012 at 5:37 PM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov">jedbrown@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>This code is so fragile. Isn't there a way to say "if the user requested ParMetis in *any* way" instead of enumerating the list of ways that it could be asked for. Should we also check whether the user asked for ParMetis using --with-parmetis-{include,lib}? I hate the idea of checking argDB directly like this.</div>

<div><br></div><div>      if self.framework.argDB.get('download-parmetis') or self.framework.argDB.get('with-parmetis'):</div><div>        self.parmetis   = framework.require('PETSc.packages.parmetis',self)</div>

<div>        self.deps.append(self.parmetis)</div>
</blockquote></div><br>This makes zero sense on its face. Why is the dependency conditional? It should be unconditional<div>and then we check whether it was found. Change it!</div><div><br></div><div>   Matt</div><div><div>
<br></div>-- <br>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener<br>
</div>