Bug 1290564 - Consider disabling ITKv3 compatibility
Summary: Consider disabling ITKv3 compatibility
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: InsightToolkit
Version: rawhide
Hardware: All
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Mario Ceresa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-12-10 19:52 UTC by Dmitry Mikhirev
Modified: 2016-01-04 18:34 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-01-04 18:34:16 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Dmitry Mikhirev 2015-12-10 19:52:19 UTC
Description of problem:

Hello!

I'm currently working on building SimpleITK package for Fedora, that is, in turn, required to build 3D-Slicer. Unfortunately building against InsightToolkit Fedora package is cumbered because it is configured with ITKv3 compatibility enabled. I asked ITK developers for help, and got an answer that building SimleITK against ITK with ITKv3 compatibility is not tested. Some workarounds were also suggested, you can read the whole thread here:

http://public.kitware.com/pipermail/insight-developers/2015-December/024687.html

There are also some more issues introduced by ITKv3 compatibility (that is incomplete anyway), in particular ITK can not handle big images (pixel addresses are 32-bit).

I searched through the Fedora repo and did not find any package depending on InsightToolkit, so disabling ITKv3 compatibility should not break anything. So please consider disabling ITKv3 compatibility in ImageToolkit package or, if it is unacceptable for some reason, removing problematic lines from itkHistogram.h file.

Comment 1 Sebastian Pölsterl 2015-12-23 14:11:33 UTC
If you provide a patch, I'd be happy to apply it.

Comment 2 Dmitry Mikhirev 2015-12-23 19:20:48 UTC
Patch to disable this feature? It's trivial:

diff --git a/InsightToolkit.spec b/InsightToolkit.spec
index c48c118..f3887b1 100644
--- a/InsightToolkit.spec
+++ b/InsightToolkit.spec
@@ -155,7 +155,7 @@ pushd %{_target_platform}
        -DCMAKE_VERBOSE_MAKEFILE=ON\
        -DBUILD_TESTING=ON\
        -DITK_FORBID_DOWNLOADS=ON \
-       -DITKV3_COMPATIBILITY:BOOL=ON \
+       -DITKV3_COMPATIBILITY:BOOL=OFF \
        -DITK_BUILD_DEFAULT_MODULES:BOOL=ON \
        -DModule_ITKVtkGlue:BOOL=ON \
        -DITK_WRAP_PYTHON:BOOL=OFF \

Comment 3 Sebastian Pölsterl 2016-01-04 18:34:16 UTC
Thanks! An updated package is available in rawhide.


Note You need to log in before you can comment on or make changes to this bug.