Debian Packaging: Difference between revisions

From Commontk
Jump to navigationJump to search
 
Line 10: Line 10:
== Related Issues and Pull Requests on Github ==
== Related Issues and Pull Requests on Github ==


https://github.com/commontk/CTK/issues/65
[RESOLVED] https://github.com/commontk/CTK/issues/65  


https://github.com/commontk/CTK/pull/158
[RESOLVED] https://github.com/commontk/CTK/pull/158


== Working configuration ==
== Working configuration ==

Latest revision as of 05:51, 23 December 2014

Home < Debian Packaging

Status

There is an experimental debian package here: http://packages.qa.debian.org/c/ctk.html

The current Debian sources for CTK are here: http://anonscm.debian.org/viewvc/debian-med/trunk/packages/ctk/trunk/debian/

Some experimental changes are published here: https://github.com/nolden/ctk-debian and https://github.com/nolden/CTK/commits/debian-patches Update 11/05/2013: everything from here has been merged upstream, apart from the DCMTK workarounds, see below.


Related Issues and Pull Requests on Github

[RESOLVED] https://github.com/commontk/CTK/issues/65

[RESOLVED] https://github.com/commontk/CTK/pull/158

Working configuration

Updated 11/05/2013

This is a working subset (possibly not complete) of CTK that builds with Debian Unstable, use this as an initial cache

CTK_APP_ctkCommandLineModuleExplorer:BOOL=ON
CTK_APP_ctkDICOM:BOOL=ON
CTK_APP_ctkPluginBrowser:BOOL=ON
CTK_APP_ctkPluginGenerator:BOOL=ON
CTK_BUILD_QTDESIGNER_PLUGINS:BOOL=ON
CTK_ENABLE_DICOM:BOOL=ON
CTK_ENABLE_PluginFramework:BOOL=ON
CTK_ENABLE_Python_Wrapping:BOOL=ON
CTK_ENABLE_Widgets:BOOL=ON
CTK_LIB_CommandLineModules/Backend/FunctionPointer:BOOL=ON
CTK_LIB_CommandLineModules/Backend/LocalProcess:BOOL=ON
CTK_LIB_CommandLineModules/Core:BOOL=ON
CTK_LIB_CommandLineModules/Frontend/QtGui:BOOL=ON
CTK_LIB_CommandLineModules/Frontend/QtWebKit:BOOL=ON
CTK_LIB_Core:BOOL=ON
CTK_LIB_DICOM/Core:BOOL=ON
CTK_LIB_DICOM/Widgets:BOOL=ON
CTK_LIB_PluginFramework:BOOL=ON
CTK_LIB_Visualization/VTK/Core:BOOL=ON
CTK_LIB_Visualization/VTK/Widgets:BOOL=ON
CTK_LIB_Widgets:BOOL=ON
CTK_PLUGIN_org.commontk.configadmin:BOOL=ON
CTK_PLUGIN_org.commontk.eventadmin:BOOL=ON
CTK_PLUGIN_org.commontk.log:BOOL=ON
CTK_PLUGIN_org.commontk.metatype:BOOL=ON
CTK_PLUGIN_org.commontk.plugingenerator.core:BOOL=ON
CTK_PLUGIN_org.commontk.plugingenerator.ui:BOOL=ON
CTK_SUPERBUILD:BOOL=OFF
DCMTK_DIR:PATH=/usr

Proposed package structure

/usr/lib/:
  # CTK libraries
  libCTKCore.so
  libCTKDICOM.so
  ...
/usr/lib/ctk-plugins/:
  # CTK plugin framework plugins
  liborg_commontk_eventadmin.so
  liborg_commontk_dah_core.so
  ...
/usr/lib/x86_64-linux-gnu/qt4/plugins/designer/:
  # Qt designer plugins
  libCTKWidgetsPlugins.so
  libCTKDICOMWidgetsPlugins.so
  ...
/usr/include/ctk/:
  # all public library headers
  ctkUtils.h
  ctkComboBox.h
  ctkDICOMDatabase.h
  ...
/usr/include/ctk/org.commontk.dah.core/: 
  # public plugin headers
  ctkDicomAppInterface.h
  ...
/usr/bin:
  # executables (separate package "ctk" or "ctk-utils"?)
  ctkDICOM
  ctkCommandLineModuleExplorer

Open issues

  • CTK side
  • Debian side
    • do we want to create separate packages for the various topics? (libctk-core, libctk-dicom, libctk-cli, ... )
    • currently the installation paths are changed in the debian/rules files, e.g. removing the "ctk-0.1" part of the path. Is that ok or should we try to solve this as well upstream?
    • should we wait for the 3.6.1 experimental package to enter "sid" or try to create some workarounds in CTK and/or the Debian package?
    • if the Debian rules map files from the install tree to a different directory layout in the Debian package, how is this going to play well together with CTKConfig.cmake?

Dependencies with potential problems

ITK

ITK 4 in Debian has limited architecture support:

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=724711

The ITK package also recently added a dependency to an older version of DCMTK. Currently we assume CTK will use the new DCMTK package which is still in experimental.

We need ITK for the CTK_LIB_ImageProcessing/ITK/Core setting. The options are

  • disable the option and move the class to Slicer or other packages needing it
  • enable the option conditionally on supported platform
  • create separate packages for CTK, so most packages can be supported on all platforms

DCMTK

CTK is using a recent snapshot in the superbuild, but Debian includes the latest stable release 3.6.0 . Since the Bologna Hackfest in December 2012 CTK builds (again) with DCMTK 3.6.0, using one backported class that is included conditionally (scu.cc)

Update 11/05/2013:

  • using CTK master and the Debian sid (3.6.0) package you need two patches that would be much more complicated to integrate upstream since the would be specific to Debian
  • using the Debian experimental package (3.6.1) everything works fine, you have to provide -DDCMTK_DIR:PATH=/usr to the initial CMake run

Log4Qt

Log4Qt is in general not used by CTK anymore. There is one plugin left using it but this is more of a technical study and can be turned of:

CTK_PLUGIN_org.commontk.log4qt:BOOL=OFF

Update 11/05/2013: This plugin is outdated will be removed soon, so we should probably remove all related code from the debian package rules

PythonQt

Several patches have been submitted to upstream, some are left, a summary can be found here. If this is resolved more features of CTK can be enabled for the Debian package.

Update 11/05/2013: one header is missing: ctkAbstractPythonManager.cpp:31:33: fatal error: PythonQt_QtBindings.h .

VTK

CTK Visualization Core compiles with VTK 5.8, just missing one small functionality.

There is an issue with VTK linking which is not totally clear, see

http://debian.2.n7.nabble.com/Bug-747436-libvtk6-Libraries-are-possibly-underlinked-td3249140.html

QtSoap

There is a debian package called libqtsolutions-soap-2.7-1, but there is no development package so it's unclear if this can be useful.

Update 11/05/2013: libqtsolutions-soap-2.7-1 is probably not useful. In the CTK context QtSoap is currently only needed for DICOM application hosting which itself is highly experimental and should probably be omitted from an initial debian package anyway