Re: [CMake] How to handle RelWithDebInfo and MinSizeRel when not built?

2013-01-21 Thread Stephen Kelly
John Drescher wrote:

>> The suggestion in the bug report looks somewhat reasonable to me, but
>> then again, we don't actually create separate debug info, and the
>> 'Release' configuration would be no different to the 'RelWithDebInfo'
>> configuration, so it's sort of a 'lie'.
>>
>> The real need for it though is probably that it allows the downstream of
>> Qt to use RelWithDebInfo themselves?
>>
>> The alternative way to do that would be to set the MAP_IMPORTED_CONFIG*
>> properties. I'm not sure whether I should advise that and close the bug
>> report, or should I implement the suggested change.
>>
>> Any ideas?
>>
> As a windows programmer who regularly uses CMake generated
> RelWithDebInfo targets with Qt4 since 2008 I believe the behavior of
> using the Release versions for RelWithDebugInfo is fine (and actually
> preferred for me).

Hi John,

Thanks for your input. Do you have to do anything in particular to enable 
this when working with FindQt4.cmake?

> If I need to debug deeply into a Qt problem I will
> switch to use the Debug build however most of the time for itk or vtk
> I want to use RelWithDebInfo to debug since these tend to be slow to
> Debug in Debug mode that is when I use dicom images that are similar
> resolution to what my programs typically use.

Do you have to do anything in particular to enable this when working with 
FindQt4.cmake? Or do you mean that you use vtk in RelWithDebInfo config, and 
Qt in Release config in that case?

Thanks,

Steve.


--

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://www.cmake.org/mailman/listinfo/cmake


Re: [CMake] How to handle RelWithDebInfo and MinSizeRel when not built?

2013-01-18 Thread John Drescher
> The suggestion in the bug report looks somewhat reasonable to me, but then
> again, we don't actually create separate debug info, and the 'Release'
> configuration would be no different to the 'RelWithDebInfo' configuration,
> so it's sort of a 'lie'.
>
> The real need for it though is probably that it allows the downstream of Qt
> to use RelWithDebInfo themselves?
>
> The alternative way to do that would be to set the MAP_IMPORTED_CONFIG*
> properties. I'm not sure whether I should advise that and close the bug
> report, or should I implement the suggested change.
>
> Any ideas?
>
As a windows programmer who regularly uses CMake generated
RelWithDebInfo targets with Qt4 since 2008 I believe the behavior of
using the Release versions for RelWithDebugInfo is fine (and actually
preferred for me). If I need to debug deeply into a Qt problem I will
switch to use the Debug build however most of the time for itk or vtk
I want to use RelWithDebInfo to debug since these tend to be slow to
Debug in Debug mode that is when I use dicom images that are similar
resolution to what my programs typically use.

John
--

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://www.cmake.org/mailman/listinfo/cmake


[CMake] How to handle RelWithDebInfo and MinSizeRel when not built?

2013-01-18 Thread Stephen Kelly

Hi,

In Qt 5, we create CMake Config files and ship them with the Qt installation 
to make it possible to use Qt 5 with CMake.

Qt on Windows and Mac is packaged with both a debug configuration and a 
release configuration, so an IMPORTED library is created with both a DEBUG 
location and a RELEASE location. 

We don't currently create a location for RelWithDebInfo or MinSizeRel, 
leading to this bug report:

 https://bugreports.qt-project.org/browse/QTBUG-29186

I checked the FindQt4 module, and it also does not create IMPORTED locations 
for RelWithDebInfo or MinSizeRel either, so I wonder if this issue has come 
up before?

As I don't use Windows or Mac, I'm not really familiar with those systems or 
configurations, so I'm looking for some guidance on what should be done 
here. 

The suggestion in the bug report looks somewhat reasonable to me, but then 
again, we don't actually create separate debug info, and the 'Release' 
configuration would be no different to the 'RelWithDebInfo' configuration, 
so it's sort of a 'lie'. 

The real need for it though is probably that it allows the downstream of Qt 
to use RelWithDebInfo themselves?

The alternative way to do that would be to set the MAP_IMPORTED_CONFIG* 
properties. I'm not sure whether I should advise that and close the bug 
report, or should I implement the suggested change. 

Any ideas?

Thanks,

Steve.


--

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://www.cmake.org/mailman/listinfo/cmake