<div class="gmail_quote">On Wed, Jun 15, 2011 at 12:06 PM, Todd Gamblin <span dir="ltr"><<a href="mailto:tgamblin@llnl.gov">tgamblin@llnl.gov</a>></span> wrote:</div><div class="gmail_quote"><br></div><div class="gmail_quote">
<div>Hi Todd,</div><div>I'm the FindBoost maintainer.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I was somewhat perplexed that FindBoost doesn't properly search the default install layout of the boost-cmake build. i.e. if you set BOOST_ROOT to point to something that looks like this:<br>
</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><snip><br>
things will break. This is, unfortunately, the default install layout for the CMake build of boost. The script only checks path suffixes under the *include* directory, and it expects them to use underscores instead of periods, e.g.:<br>
</blockquote><div><br></div><div>I'm not sure exactly why boost-cmake decided to make their output directories use periods instead of underscores. It seems like a first step towards replacing bjam would at least be to match its install path format, but I digress.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I've pasted below a patch that adds the version numbers with periods to the list of searched path suffixes, and that uses the path suffix list for both the library search and the header search. Any objections to my submitting this as a topic branch and pushing it out to next? I'm not sure I have permission to do that with the FindBoost module, so let me know what the right procedure is.<br>
</blockquote><div><br></div><div>The patch looks good. Feel free to submit it and push it to next.</div><div><br></div><div>Thanks for asking for permission, by the way. In general, feel free to propose changes via email or even topic stages for FindBoost. I do welcome the help. I would like to review changes before they are pushed to next though (mainly to prevent people from breaking things).</div>
<div><br></div></div>-- <br>Philip Lowman<br>