On Sat, Dec 24, 2011 at 1:34 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.com</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">

High level questions:<div><br></div><div>Why generate tarballs as well as installing the actual docs?</div></blockquote><div><br>Dunno, i've been following Makefile build.<br><br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div>Why set some of the binaries to 'true' and run those command unconditionally, but in other cases just omit steps when the tools necessary are unavailable? The latter sounds cleaner.</div></blockquote><div><br>

Yes, this seems OK. Will fix it.<br> <br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div>Does any of this happen for a normal build, or is it all only triggered by an explicit target or 'make install'?</div></blockquote><div><br>All targets created by add_custom_target(tgt ALL ...) are run during normal build, but also can be invoked explicitly with `make tgt`. Everything that mentioned in install() is installed by `make install`.<br>

<br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Please go ahead and commit the cpack version number fix, it's independent of the rest.<br>

</div></blockquote><div><br>I do not have commit rights.<br> <br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>
I'll try to look through these more carefully, but they're a bit confusing. Comments would likely help as the CMake languages doesn't make this kind of stuff obvious.<br></div></blockquote><div><br>Thanks for quick review!<br>

</div></div>