Page MenuHomePhabricator

Update DCMTK to latest snapshot
Closed, ResolvedPublic

Event Timeline

User kislinsk has pushed new remote branch:

bug-19700-UpdateDCMTK

Having trouble to get things working. For example, MSVC 2013 is not accepted by DCMTK as C++11 conform compiler. Even the latest master branch of DCMTK isn't working. Seems we have to patch. Boils down to calling std::inf instead of ::inf. Also need to be careful with which compilers we activate the C++11 STL features of DCMTK.

I tracked down the std::isinf and related problems. It's probably better to fix the CMake detection code for these functions. Will push a patch soon.

User kislinsk has pushed new remote branch:

bug-19700-UpgradeDCMTK-Fixed

[cf49f3]: Merge branch 'bug-19700-UpgradeDCMTK-Fixed'

Merged commits:

2016-05-10 13:50:34 Stefan Kislinskiy [b36ec7]
COMP: Upgrade DCMTK to latest snapshot

@Marco
Patched already. Should contribute to DCMTK as they have the same error for isinf and isfinite in principle. Additionally they're looking for std::finite instead of std::isfinite.

User kislinsk has pushed new remote branch:

bug-19700-FixDCMTK-CTK

[ceedd6]: Merge branch 'bug-19700-FixDCMTK-CTK'

Merged commits:

2016-05-10 14:59:09 Stefan Kislinskiy [66bfd9]
COMP: Fix DCMTK configuration and patch to work with CTK