<div class="gmail_quote">On Thu, Jun 2, 2011 at 3:18 PM, Rafael Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div id=":1yy">+  // FIXME: Using CXX_INCLUDE_ROOT is here is a bit of a hack, but<br>
+  // avoids adding yet another option to configure/cmake.<br>
+  // It would probably be cleaner to break it in two variables<br>
+  // CXX_GCC_ROOT with just /foo/bar<br>
+  // CXX_GCC_VER with 4.5.2<br>
+  // Then we would have<br>
+  // CXX_INCLUDE_ROOT = CXX_GCC_ROOT/include/c++/CXX_GCC_VER<br></div></blockquote><div><br></div><div>While I agree deriving the GCC root from the include root is a hack, keep in mind that we actually use Clang in an environment where the include root is completely synthetic and doesn't correspond to any GCC root or similar structure. It really is just an arbitrary root for the C++ standard headers. I'd like to preserve that functionality with what ever final solution we end up with.</div>
<div><br></div><div>Having variables dedicated to easily configuring Clang to use existing GCC installations and versions certainly doesn't seem like a bad thing to me.</div></div>