----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110083/#review32015 -----------------------------------------------------------
libkfbapi/userinfo.h <http://git.reviewboard.kde.org/r/110083/#comment23854> No, this would be global symbol. I meant as a constant of UserInfo, i.e. UserInfo::InvalidTimeZone. It is a special value returned by UserInfo::timezone(), right? - Kevin Krammer On May 4, 2013, 9:34 a.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110083/ > ----------------------------------------------------------- > > (Updated May 4, 2013, 9:34 a.m.) > > > Review request for kdelibs and KDEPIM-Libraries. > > > Description > ------- > > kdepim-libs are needed in the facebook library only to return user info as > KABC::Addressee and note as KMime::Message, plus events use KCalCore classes. > This patch makes dependency on kdepim-libs optional and disables building the > event classes completely in case kdepim-libs was not found. > > > Diffs > ----- > > LibKFbAPI-KDEPIMConfig.cmake.in PRE-CREATION > CMakeLists.txt 5aefdcf > LibKFbAPI-KDEPIM.pc.in PRE-CREATION > LibKFbAPI.pc.in af537d1 > libkfbapi/CMakeLists.txt dac62bc > libkfbapi/commentinfo.h e5578c7 > libkfbapi/kdepim-utils.h PRE-CREATION > libkfbapi/kdepim-utils.cpp PRE-CREATION > libkfbapi/likeinfo.h e06052e > libkfbapi/noteinfo.h 2492db1 > libkfbapi/noteinfo.cpp 154593d > libkfbapi/notificationinfo.h a882694 > libkfbapi/userinfo.h ac22a7e > libkfbapi/userinfo.cpp 26c64da > libkfbapi/userinfoparser_p.h 0189a17 > > Diff: http://git.reviewboard.kde.org/r/110083/diff/ > > > Testing > ------- > > Builds correctly here in both cases > > > Thanks, > > Martin Klapetek > >