Skip to content

Updating VTK9 fix to remove specific VTK9 option

Jeremy Cohen requested to merge jhc02/nektar:fix/vtk9 into fix/vtk9

This MR builds on the changes by @kayarre (thanks for the initial fix @kayarre) to remove the specific NEKTAR_USE_VTK9 option and instead to find the most recently available version of VTK on the target system.

See the discussion in MR1300 in the main Nektar++ repository for further information.

A couple of additional points on this update:

  • From testing builds with different versions of VTK, given the updates in the original fix submission, the NEKTAR_USE_VTK option successfully finds VTK9 even without specifying NEKTAR_USE_VTK9 (although the original module names have been updated, the old names are still available but generate deprecation warnings).
  • Some users working with systems with packaged versions of VTK may not actually know what version of VTK is installed on their system without further investigation.

As a result, it was felt that a simpler, non-version-specific approach might be easier in this case.

This MR also adds a couple of fixes to add required headers to the VtkToPng and VtkToFld tools so that they build with VTK9.

Hope this all sounds OK with you @kayarre. Happy to discuss, otherwise if this is OK, please merge these changes into yours and I guess we should then be able to merge everything into the main repo under the original MR - wanted to maintain the history with your contribution.

Merge request reports