[CMake] Why drop -D from wxWidgets_DEFINITIONS ?

2010-10-07 Thread Steve M. Robbins
Hi,

The FindwxWidget.cmake module in 2.8 is breaking the build of
MRIConvert [1] because it strips -D from the output of wx-config
--cxxflags.

Is this not a bug?  

This question was asked here last October [2] but not answered.

Thanks,
-Steve

[1] http://lcni.uoregon.edu/~jolinda/MRIConvert/
[2] http://www.cmake.org/pipermail/cmake/2009-October/032831.html


signature.asc
Description: Digital signature
___
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] Why drop -D from wxWidgets_DEFINITIONS ?

2009-12-03 Thread Dmitry Marakasov
* Orion Poplawski (or...@cora.nwra.com) wrote:

 Note that this is breaking previously working builds of plplot because 
 of the missing -D.  I don't see this being done in any other module.

This also breaks Rigs of Rods with the same symptoms.

-- 
Dmitry Marakasov   .   55B5 0596 FF1E 8D84 5F56  9510 D35A 80DD F9D2 F77D
amd...@amdmi3.ru  ..:  jabber: amd...@jabber.ruhttp://www.amdmi3.ru
___
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] Why drop -D from wxWidgets_DEFINITIONS ?

2009-10-28 Thread Orion Poplawski

On Tue, October 27, 2009 2:15 pm, Werner Smekal wrote:
 Hi Orion,

 On Oct 27, 2009, at 6:53 PM, Orion Poplawski wrote:

 On 10/26/2009 12:44 PM, Orion Poplawski wrote:
 in 2.8.0, FindwxWidgets.cmake strips -D from
 wxWidgets_DEFINITIONS.  Why?


 Note that this is breaking previously working builds of plplot
 because of the missing -D.  I don't see this being done in any
 other module.

 I can't talk for Miguel, but these changes should not break your code,
 if you use the UsewxWidgets.cmake file which adds all the definitions,
 flags, etc. accordingly. We (for plplot) couldn't use this macro since
 we needed the definitions only for some source files, and since they
 were not placed in a separate directory, we just set the properties
 for the source files (without using usewxwidgets.cmake). This was
 maybe not the best thing to do, but for cmake 2.8.0 the plplot cmake
 build system is now broken (for the wxWidgets driver).

 Anyway, we need to change the plplot cmake build system for the
 wxWidgets part, since the changes Miguel applied fixed some problems
 for the MinGW and Visual C++ compiler toolset and I also plan to
 include FindwxWidgets.cmake to the plplot source, so that cmake 2.6.x
 users can also benefit from these changes as well.

Okay.  Let me know when a fix has been checked in to plplot and I'll test.

-- 
Orion Poplawski
Technical Manager 303-415-9701 x222
NWRA/CoRA DivisionFAX: 303-415-9702
3380 Mitchell Lane  or...@cora.nwra.com
Boulder, CO 80301  http://www.cora.nwra.com
___
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] Why drop -D from wxWidgets_DEFINITIONS ?

2009-10-27 Thread Orion Poplawski

On 10/26/2009 12:44 PM, Orion Poplawski wrote:

in 2.8.0, FindwxWidgets.cmake strips -D from wxWidgets_DEFINITIONS.  Why?



Note that this is breaking previously working builds of plplot because 
of the missing -D.  I don't see this being done in any other module.


--
Orion Poplawski
Technical Manager 303-415-9701 x222
NWRA/CoRA DivisionFAX: 303-415-9702
3380 Mitchell Lane  or...@cora.nwra.com
Boulder, CO 80301  http://www.cora.nwra.com
___
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] Why drop -D from wxWidgets_DEFINITIONS ?

2009-10-27 Thread Werner Smekal

Hi Orion,

On Oct 27, 2009, at 6:53 PM, Orion Poplawski wrote:


On 10/26/2009 12:44 PM, Orion Poplawski wrote:
in 2.8.0, FindwxWidgets.cmake strips -D from  
wxWidgets_DEFINITIONS.  Why?




Note that this is breaking previously working builds of plplot  
because of the missing -D.  I don't see this being done in any  
other module.


I can't talk for Miguel, but these changes should not break your code,  
if you use the UsewxWidgets.cmake file which adds all the definitions,  
flags, etc. accordingly. We (for plplot) couldn't use this macro since  
we needed the definitions only for some source files, and since they  
were not placed in a separate directory, we just set the properties  
for the source files (without using usewxwidgets.cmake). This was  
maybe not the best thing to do, but for cmake 2.8.0 the plplot cmake  
build system is now broken (for the wxWidgets driver).


Anyway, we need to change the plplot cmake build system for the  
wxWidgets part, since the changes Miguel applied fixed some problems  
for the MinGW and Visual C++ compiler toolset and I also plan to  
include FindwxWidgets.cmake to the plplot source, so that cmake 2.6.x  
users can also benefit from these changes as well.


Regards,
Werner



--
Orion Poplawski
Technical Manager 303-415-9701 x222
NWRA/CoRA DivisionFAX: 303-415-9702
3380 Mitchell Lane  or...@cora.nwra.com
Boulder, CO 80301  http://www.cora.nwra.com
___
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



___
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] Why drop -D from wxWidgets_DEFINITIONS ?

2009-10-26 Thread Orion Poplawski

in 2.8.0, FindwxWidgets.cmake strips -D from wxWidgets_DEFINITIONS.  Why?

--
Orion Poplawski
Technical Manager 303-415-9701 x222
NWRA/CoRA DivisionFAX: 303-415-9702
3380 Mitchell Lane  or...@cora.nwra.com
Boulder, CO 80301  http://www.cora.nwra.com
___
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