Hey guys,<div><br></div><div>Unless this is fixing a very specific problem, let's not do it in this release. As it is, we have taken on a lot of changes and we are really stretched to make the release soon. Even though it sounds like a trivial change, nothing ever is and this will probably end up requiring days if not weeks of effort when extended to all codes.</div>
<div><br></div><div>-berk<br><br><div class="gmail_quote">On Sun, Sep 2, 2012 at 1:21 PM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Sat, Sep 1, 2012 at 4:35 PM, David Thompson<br>
<<a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a>> wrote:<br>
> Hi Marcus (et al.),<br>
><br>
> We had talked about simplifying the list of include directives VTK 6 passes to the compiler. Attached is a ReadMe and a couple of Python scripts to walk through the VTK source tree and change all the include directives in the source so they contain the kit name. For example:<br>

><br>
>     #include "vtkDataArray.h"<br>
><br>
> would get modified to<br>
><br>
>     #include "Common/Core/vtkDataArray.h"<br>
><br>
> The ReadMe goes into a little more detail. I haven't tried altering CMakeLists.txt to change the list of include directives passed to the compiler yet (much less tried compiling), but figured people might have comments on things so far.<br>

><br>
> The map that the first script builds should also be usable for external projects that include VTK.<br>
><br>
</div></div>Sounds good to me, but I was always in support of moving to something<br>
like this. I think with a Python script like this to help others<br>
change their code it should be pretty easy to port. I was also<br>
looking, and we can do what other projects such as Qt do where the<br>
found modules are added to the compiler's include path if the use file<br>
is included, if not they would need to use the full path.<br>
<br>
I will try running the script and see what it looks like, it would be<br>
great to only have to add one or two lines to the compiler's include<br>
path by default. I can probably expand out the CMake support to<br>
preserve the directories on install, this would also have the major<br>
advantage of removing many differences present when building against a<br>
build or install tree.<br>
<br>
Marcus<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</blockquote></div><br></div>