Skip to content

ENH: update DCMTK

Sam Horvath requested to merge github/fork/fedorov/update-dcmtk-DCMTK-3.6.1_20160216 into master

Created by: fedorov

This updates DCMTK to the latest snapshot DCMTK-3.6.1_20160216

Note that the two patches that were applied to the previous update of DCMTK hash here https://github.com/commontk/DCMTK/commits/patched-DCMTK-3.6.1_20150924 are no longer needed, as they were integrated into the DCMTK master. Therefore, the repo is checked out from the official DCMTK source.

I tested that Slicer compiles with this change on Mac OS X (Yosemite 10.10.5, LLVM version 7.0.0), and that no test failures related to DCMTK are introduced (I looked at the individual failures, and there is nothing to indicate these are due to DCMTK update):

Total Test time (real) = 7071.30 sec

The following tests FAILED:
     51 - vtkMRMLNonlinearTransformNodeTest1 (Failed)
    101 - vtkObserverManagerTest1 (Failed)
    475 - ModelToLabelMapTest (Failed)
    477 - N4ITKBiasFieldCorrectionTest (Failed)
    572 - py_RSNAVisTutorial (Timeout)
    601 - vtkMRMLFiberBundleNodeTest1 (Failed)
    609 - DiffusionWeightedVolumeMaskingTest (Failed)
Errors while running CTest

cc: @michaelonken @jriesmeier

Merge request reports