Re: Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113887/ --- (Updated Nov. 18, 2013, 2:22 p.m.) Status -- This change has been

Re: Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-18 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113887/#review43891 --- This review has been submitted with commit

Re: Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-18 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113887/#review43910 --- ${KAUTH_POLICY_FILES_INSTALL_DIR} This is bogus (but e.g.

Re: Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-17 Thread Stephen Kelly
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113887/#review43856 --- Ship it! Ship It! - Stephen Kelly On Nov. 15, 2013, 5:34

Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-15 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113887/ --- Review request for Build System and KDE Frameworks. Repository: kdelibs

Re: Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-15 Thread Kevin Ottens
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113887/#review43762 --- Looks fine to me, but I'd rather have a second pair of eyes