----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113365/#review42119 -----------------------------------------------------------
Ship it! Looks sane, and builds fine even when I remove the installed kdesu/kdesu_export.h file. - Alex Merry On Oct. 21, 2013, 12:27 p.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113365/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2013, 12:27 p.m.) > > > Review request for KDE Frameworks and Martin Klapetek. > > > Repository: kdelibs > > > Description > ------- > > kdesu headers are installed to kdesu/ so the #include <kdesu_export.h> cannot > be found if one tries to use kdesu includes in a project. > > > Diffs > ----- > > tier3/kdesu/src/client.h 8f29b93 > tier3/kdesu/src/process.h c527e83 > tier3/kdesu/src/ssh.h 8ab6472 > tier3/kdesu/src/stub.h 68d7440 > tier3/kdesu/src/su.h e83781d > > Diff: http://git.reviewboard.kde.org/r/113365/diff/ > > > Testing > ------- > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel