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

[Mathieu Malaterre]

On Thu, Dec 5, 2013 at 1:41 PM, Dmitrijs Ledkovs <xnox at debian.org> wrote:
> On 5 December 2013 11:50, Mathieu Malaterre <malat at debian.org> wrote:
>> Control: tags -1 = wontfix
>>
>>
>> ---------- Forwarded message ----------
>>
>>
>> Hi Mathieu,
>>
>> regarding this bug:
>>
>>   http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=730636
>>
>> I talked to JC from Kitware (who contributed the patch), and his advice
>> is the following:
>>
>>> After setting the env:
>>>
>>> export CXXFLAGS=<whatever-value>
>>>
>>> and configuring passing
>>>
>>> -DCMAKE_BUILD_TYPE:STRING=None
>>>
>>>
>>> None of the default flags related to either Debug, Release,
>>> RelWithDebInfo, .. are used. Only the one specified using CXXFLAGS.
>>> This was confirmed after doing a small experiment.
>
> BUILD_TYPE:STRING=None is the cmake default. One shouldn't
> override/reselect anything else. Please see CMake documentation.
> Default BUILD_TYPE shouldn't change across different packages using
> CMake, as that breaks transparent inclusion / building CMake packages.
> If such change is desired, it should be done by CMake, in CMake
> upstream, not on per package change.
> This will bite us later on...
>
> =(

[CCing cmake maintainer]

My undertanding was that BUILD_TYPE:STRING="" was the actual
default...while BUILD_TYPE:STRING=None explicitely state the user
intention.