<div dir="auto">We can drop VTK5 support.<div dir="auto"><br></div><div dir="auto">Thanks</div><div dir="auto"><br></div><div dir="auto">Bill</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 10, 2019, 6:05 AM Ben Boeckel <<a href="mailto:ben.boeckel@kitware.com">ben.boeckel@kitware.com</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jan 10, 2019 at 05:24:17 +0000, Andras Lasso wrote:<br>
> Yes, we need this. Please give us examples that show how to write our<br>
> CMake and C++ files to work with both the old and new VTK module<br>
> system.<br>
<br>
I have some patches for WikiExamples locally, but have been focusing on<br>
the dashboards first.<br>
<br>
For supporting both, something like this pattern is what I was doing:<br>
<br>
    find_package(VTK)<br>
    if (VTK_VERSION VERSION_LESS "8.90")<br>
      # old system<br>
      include(${VTK_USE_FILE})<br>
      # modules are linked via `vtkCommonCore`<br>
      # VTK_DEFINITIONS has autoinit information<br>
    else ()<br>
      # modules are linked via `VTK::CommonCore`<br>
      # vtk_module_autoinit is needed<br>
    endif ()<br>
<br>
WikiExamples also tries to support VTK5, so it has a bit more logic to<br>
go in there as well.<br>
<br>
--Ben<br>
</blockquote></div>