[Debian-med-packaging] Bug#730636: Do not always set CMAKE_BUILD_TYPE to Release

Jörg Riesmeier debian at riesmeier.de
Thu Nov 28 10:34:42 UTC 2013


> dcmtk package is trying to set CMAKE_BUILD_TYPE to Release instead of
> letting the user decide what best to use.
> It would be nice to have an option to override this default, as Debian
> requires specific compilation flags, which conflicts with what
> CMAKE_BUILD_TYPE:Release sets.

> I am attaching current patch.

Your patch removes a patch that was contributed only recently by Jean-
Christophe Fillion-Robin from Kitware (i.e. the publisher of CMake):

  http://git.dcmtk.org/web?p=dcmtk.git;a=commit;h=1beff6732f65c4e364acbd7d4643a02ed02f82ee

Also, as far as I can see, the "IF(NOT CMAKE_BUILD_TYPE AND NOT 
CMAKE_CONFIGURATION_TYPES)" statement should make sure that the build type is 
only set if it was undefined.

Don't you agree?



More information about the Debian-med-packaging mailing list