CTK icon indicating copy to clipboard operation
CTK copied to clipboard

Published 20 hours ago • verified publisher icon commontk

COMP: Add vtk < 9.1 guard for vtkConfigure.h

Open jadh4v opened this issue 3 years ago • 1 comments

vtkConfigure.h has been deprecated and should only be included if VTK version is < 9.1.

jadh4v avatar Dec 17 '21 23:12 jadh4v

@jcfr take a look. This is needed for VTK-install to work. CTK was picking up the vtkConfigure.h from build directory even though VTK_LEGACY_REMOVE flag is set. Nevertheless, this header is not needed for vtk >= 9.1 as also confirmed by @mathstuf

jadh4v avatar Dec 17 '21 23:12 jadh4v

Closing. This was superseded by https://github.com/commontk/CTK/commit/2e1c014b826ffa2a4d22935818f29451c69df418 introduced through https://github.com/commontk/CTK/pull/1041

@jadh4v Thanks for contributing this change in the first place and my apology for not integrating it earlier and not giving proper credits in the latest pull request integrating a similar change.

jcfr avatar Oct 24 '22 17:10 jcfr

No worries

jadh4v avatar Oct 27 '22 22:10 jadh4v