<div dir="ltr"><div>I figured with 9.0 people have to rewrite their CMakeLists files calls to use vtk so it was a good time to rename OpenGL2 back to OpenGL. To wait until the next release means we would break their builds two releases in a row which seems worse to me.</div><div><br></div><div>OpenGL requires 3.2 or ES3 so you could name it vtkRenderingOpenGL3.2orES3 but I would recommend against it. I doubt we will have another C++ OpenGL backend. The first lasted over 20 years.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 8, 2018 at 11:32 AM, Jean-Christophe Fillion-Robin <span dir="ltr"><<a href="mailto:jcfr@kitware.com" target="_blank">jcfr@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 dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi Ben, <br></div><div><br></div><div>Thanks for the update. This look very promising. <br></div><div><br></div><div>Also glad to see the "UseVTK.cmake" disappear, the use of directory properties to handle VTK_DEFINITIONS has been the source of a lot of trouble.</div><div><br></div><div><br></div><div>Will the new system support the following:</div><div>* Creating a VTK SDK installed in a read-only location and support building multiple interdependent VTK external modules against it ?  <br></div><div>* Building any VTK module either externally or along with the VTK project ?  (e.g this will be useful for creating python wheels)</div><div>* Re-using the VTK python wrapping facility without having to buy in the VTK internal build system.</div><div><br></div><div>These are the limitations of the current infrastructure that I would also like to see addressed by such a drastic change.<br></div><div><br></div><div><br></div><div>> build against VTK</div><div><br></div><div>Will the way to use find_package(VTK ...) and to specify required module change ?<br></div><div><br></div><div>I also think updating VTK example to work with the new build system would be a good test case, this will provide an path forward for project that need to support both.<br></div><div><br></div><div><br></div><div>> `vtk.module` / `vtk.kit`  files</div><div><br></div><div>While this is an implementation details, would it be more sensible to standardize on friendly for human, readable by machine and declarative by nature format ? E.g json, toml, ..</div><div>Or may be something like the "Common Package Specification" (see <a href="https://mwoehlke.github.io/cps/" target="_blank">https://mwoehlke.github.io/<wbr>cps/</a>) ?<br></div><div><br></div><div><br></div><div><br></div><div>> rename OpenGL2 back to OpenGL as was the original intent?</div><div><br></div><div>I would prefer we do NOT to that for the next release, it's gonna make work of user of VTK even more difficult. We are still working our way to handle the switch of VTK major version from 9 to 8 ... <br></div><div><br></div><div>Also in the future, I anticipate we will have new version of backend. Should the module be named after the minimum version of OpenGL it support ?  (e.g OpenGL3.2)  <br></div><div>Or since one backend can be built at a time, should alias target be created instead ? <br></div><div><br></div><div><br></div><div><br></div><div>Thanks</div><div>Jc<br></div><div><br></div><div><br></div></div></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 30, 2018 at 2:16 PM David E DeMarle <<a href="mailto:dave.demarle@kitware.com" target="_blank">dave.demarle@kitware.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto">+1 for rename </div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 30, 2018, 2:09 PM Ken Martin <<a href="mailto:ken.martin@kitware.com" target="_blank">ken.martin@kitware.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">So.... if we are significantly changing the modules shoudl we also take the time to rename OpenGL2 back to OpenGL as was the original intent?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 29, 2018 at 5:42 PM, Ben Boeckel <span dir="ltr"><<a href="mailto:ben.boeckel@kitware.com" rel="noreferrer" target="_blank">ben.boeckel@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Oct 29, 2018 at 22:31:38 +0100, Elvis Stansvik wrote:<br>
> Let me just jump in and say that while it sounds neat to embed it<br>
> comments, it might make it slightly harder to make an updated version<br>
> of Utilities/Maintenance/<wbr>WhatModulesVTK.py script (which I love!).<br>
<br>
Thanks for the reminder that this file needs updated.<br>
<br>
CMake supports long-form comments now, so I'd think it'd be in blocks<br>
like this:<br>
<br>
```cmake<br>
#[==[vtk.kit<br>
...<br>
#]==]<br>
#[==[vtk.module<br>
...<br>
#]==]<br>
```<br>
<br>
with a limit of one `vtk.module` per file. Easy to slice out with a<br>
simple `sed` call. Not so easy in CMake, but also not ridiculous.<br>
<br>
This is similar to the way the new module system is documented:<br>
<br>
```cmake<br>
#[==[.md<br>
...<br>
#]==]<br>
```<br>
<br>
which will make it possible to extract out these blocks and feed them to<br>
Doxygen and get the module system API docs on the web with the rest of<br>
VTK.<br>
<br>
There's also `#[==[.md INTERNAL` for internal function documentation as<br>
well.<br>
<br>
--Ben<br>
______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="https://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer noreferrer" target="_blank">https://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="m_-3624356458865376806m_-7064672583049930506m_9116236366839769186m_-3703976905079617927gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Ken Martin PhD<div>Distinguished Engineer<br><span style="font-size:12.8px">Kitware Inc.</span><br></div><div><a href="https://maps.google.com/?q=101+East+Weaver+Street+Carrboro,+North+Carolina+%0D%0A27510+USA&entry=gmail&source=g">101 East Weaver Street</a><br><a href="https://maps.google.com/?q=101+East+Weaver+Street+Carrboro,+North+Carolina+%0D%0A27510+USA&entry=gmail&source=g">Carrboro, North Carolina</a><br>
27510 USA<br><br><div><span style="font-size:10pt;font-family:Tahoma,sans-serif">This communication,
including all attachments, contains confidential and legally privileged
information, and it is intended only for the use of the addressee.  Access to this email by anyone else is
unauthorized. If you are not the intended recipient, any disclosure, copying,
distribution or any action taken in reliance on it is prohibited and may be
unlawful. If you received this communication in error please notify us
immediately and destroy the original message. 
Thank you.</span></div></div></div></div></div></div></div>
</div>
______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="https://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer noreferrer" target="_blank">https://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
<br>
</blockquote></div>
______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="https://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">https://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
<br>
</blockquote></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Ken Martin PhD<div>Distinguished Engineer<br><span style="font-size:12.8px">Kitware Inc.</span><br></div><div>101 East Weaver Street<br>Carrboro, North Carolina<br>
27510 USA<br><br><div><span style="font-size:10pt;font-family:Tahoma,sans-serif">This communication,
including all attachments, contains confidential and legally privileged
information, and it is intended only for the use of the addressee.  Access to this email by anyone else is
unauthorized. If you are not the intended recipient, any disclosure, copying,
distribution or any action taken in reliance on it is prohibited and may be
unlawful. If you received this communication in error please notify us
immediately and destroy the original message. 
Thank you.</span></div></div></div></div></div></div></div>
</div>