I would just leave it out until we have an actual need for it.  No reason to add complexity (however small) that nobody needs.<br><br><div class="gmail_quote">On Mon Nov 17 2014 at 9:57:53 AM Ed Maste <<a href="mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: scripts/CMakeLists.txt:12<br>
@@ -12,1 +11,3 @@<br>
+         DEPENDS ${SWIG_INPUTS}<br>
+         COMMAND ${PYTHON_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/<u></u>buildSwigWrapperClasses.py "--srcRoot=${LLDB_SOURCE_DIR}" "--targetDir=${CMAKE_CURRENT_<u></u>BINARY_DIR}" "--cfgBldDir=${CMAKE_CURRENT_<u></u>BINARY_DIR}" "--prefix=${CMAKE_BINARY_DIR}" "--swigDir=${SWIG_DIR}" "--swigExecutable=${SWIG_<u></u>EXECUTABLE}" -m<br>
          COMMENT "Python script building LLDB Python wrapper")<br>
----------------<br>
ismailp wrote:<br>
> emaste wrote:<br>
> > I must be missing something -- I don't see how swigDir is used.<br>
> CMake is setting that variable. That directory seems like to include skeleton/template/boilerplate files. buildSwigWrapperClasses.py, however, doesn't use it. I thought it might be useful in the future, and kept it. Perhaps it's best to remove it to prevent confusion.<br>
Yeah, if there are no consumers yet I would leave it out for now. Or, add a comment that it's not yet used but will be soon if that's the case.<br>
<br>
<a href="http://reviews.llvm.org/D6290" target="_blank">http://reviews.llvm.org/D6290</a><br>
<br>
<br>
</blockquote></div>