-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108259/
-----------------------------------------------------------

Review request for Build System and Stephen Kelly.


Description
-------

It makes no sense to get_target_property() before add_executable/library, hence 
use CMAKE_AUTOMOC instead.

If/when cmake will allow setting properties on target before creating them, 
we'll need something like
get_target_property(_tgt_AUTOMOC ${_target_NAME} AUTOMOC)
if(CMAKE_AUTOMOC)
  set(_tgt_AUTOMOC TRUE)
endif()


Diffs
-----

  cmake/modules/KDE4Macros.cmake 6bb1240 

Diff: http://git.reviewboard.kde.org/r/108259/diff/


Testing
-------

Built zanshin with CMAKE_AUTOMOC=TRUE (+some changes to #include ...moc... 
lines).


Thanks,

Yury Georgievich Kudryashov

_______________________________________________
Kde-buildsystem mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-buildsystem

Reply via email to