----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123171/#review78163 -----------------------------------------------------------
ksysguardd/CMakeLists.txt (line 6) <https://git.reviewboard.kde.org/r/123171/#comment53543> from what I understand of the ECM docs, this is a deprecated variable and we should use KDE_INSTALL_FULL_SYSCONFDIR instead which will also fix the problem. - David Edmundson On March 29, 2015, 11:47 a.m., David Rosca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123171/ > ----------------------------------------------------------- > > (Updated March 29, 2015, 11:47 a.m.) > > > Review request for Plasma and John Tapsell. > > > Repository: ksysguard > > > Description > ------- > > This fixes -DKSYSGUARDDRCFILE definition so that there are no extra quotes. > It also fixes issue with using non-default install prefix (eg. when > CMAKE_INSTALL_PREFIX is not /usr and thus SYSCONF_INSTALL_DIR is relative and > not /etc) > > > Diffs > ----- > > ksysguardd/CMakeLists.txt 6bd8679 > > Diff: https://git.reviewboard.kde.org/r/123171/diff/ > > > Testing > ------- > > No error in syslog after applying this patch: cannot open config file > '"etc/ksysguarddrc"' > > > Thanks, > > David Rosca > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
