----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108259/#review25157 -----------------------------------------------------------
Ship it! Ship It! - Stephen Kelly On Jan. 8, 2013, 10:17 a.m., Yury Georgievich Kudryashov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108259/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2013, 10:17 a.m.) > > > 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
