Re: [cmake-developers] Review request: Qt4 interface includes and defines (Was Re: Interface includes and defines plumbing)

2013-01-07 Thread Clinton Stimpson
On Monday, January 07, 2013 08:55:40 PM Stephen Kelly wrote:
 Stephen Kelly wrote:
  Brad King wrote:
  On 01/04/2013 12:09 PM, Stephen Kelly wrote:
  Brad King wrote:
  Yes, please.  Thanks for splitting this out.
  
  Done, thanks,
  
  Thanks for cleaning up the dashboard trouble.  I've merged this
  to master!
  
  Great, thanks!
  
  I've pushed two new branches to my gitorious clone.
 
 I also re-pushed the qt4-target-depends branch to add
 INTERFACE_INCLUDE_DIRECTORIES and INTERFACE_COMPILE_DEFINTIONS to the Qt 4
 targets.
 
 Alex, Clinton, could you review before I merge to next?
 

Looks fine to me except that the Qt3Support module needs
-DQT3_SUPPORT

-- 
Clinton Stimpson
Elemental Technologies, Inc
Computational Simulation Software, LLC
www.csimsoft.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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers


Re: [cmake-developers] Review request: Qt4 interface includes and defines

2013-01-07 Thread Stephen Kelly
Clinton Stimpson wrote:

 On Monday, January 07, 2013 08:55:40 PM Stephen Kelly wrote:
 Stephen Kelly wrote:
  Brad King wrote:
  On 01/04/2013 12:09 PM, Stephen Kelly wrote:
  Brad King wrote:
  Yes, please.  Thanks for splitting this out.
  
  Done, thanks,
  
  Thanks for cleaning up the dashboard trouble.  I've merged this
  to master!
  
  Great, thanks!
  
  I've pushed two new branches to my gitorious clone.
 
 I also re-pushed the qt4-target-depends branch to add
 INTERFACE_INCLUDE_DIRECTORIES and INTERFACE_COMPILE_DEFINTIONS to the Qt
 4 targets.
 
 Alex, Clinton, could you review before I merge to next?
 
 
 Looks fine to me except that the Qt3Support module needs
 -DQT3_SUPPORT
 

Good catch. I've added it and merged it to next.

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://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers