search

dgobbi / vtk-dicom

A set of classes for using DICOM in VTK.
BSD 3-Clause "New" or "Revised" License
258 stars 94 forks source link

BUILD date not included #203

Closed JensMunkHansen closed 4 months ago

JensMunkHansen commented 3 years ago

Hi David

The DICOM_BUILD_DATE and DICOM_BUILD_TIME is not included for external builds or if the vtk_9 macros handles the inclusion of the Source directory. Is this on purpose?

Just a thing that I noted

dgobbi commented 3 years ago

Thanks for your note. I hadn't noticed because at the time that I rearranged the CMakeLists.txt I was still doing my deployment builds with VTK 7. The DICOM_BUILD_DATE is only used by some of the executables and nowhere else. I'll consider what to do with it the next time I clean up the cmake files, it never worked properly on Windows, just Linux/UNIX.

dgobbi commented 4 months ago

I've removed the DICOM_BUILD_DATE and DICOM_BUILD_TIME variables as part of a simplification of the cmake scripts.