----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116980/#review53796 -----------------------------------------------------------
docs/kded5/man-kded5.8.docbook <https://git.reviewboard.kde.org/r/116980/#comment37729> All other docbook man pages in frameworks have "&kde; User's Manual" here, why this change? docs/kded5/man-kded5.8.docbook <https://git.reviewboard.kde.org/r/116980/#comment37728> Are you sure <filename>~/.config/kdedrc</filename> is correct? Shouldn't kdedrc be in "kf5-config --path config"? But here it is in: /home/kdeframeworks/.kde/share/config/kdedrc docs/kded5/man-kded5.8.docbook <https://git.reviewboard.kde.org/r/116980/#comment37724> grep CheckStamps frameworks/kded -> no hit $ wcgrep CheckFileStamps frameworks/kded/ frameworks/kded/README.md:62: CheckFileStamps=true frameworks/kded/src/kded.cpp:749: checkStamps = cg.readEntry("CheckFileStamps", true); frameworks/kded/src/kded.cpp:765: checkStamps = cg.readEntry("CheckFileStamps", true); Should be changed to "CheckFileStamps" - Burkhard Lück On March 22, 2014, 5:53 p.m., Alex Merry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116980/ > ----------------------------------------------------------- > > (Updated March 22, 2014, 5:53 p.m.) > > > Review request for Documentation and KDE Frameworks. > > > Repository: kded > > > Description > ------- > > Update the kded5 man page > > > Diffs > ----- > > docs/kded5/man-kded5.8.docbook 4fca21f00c43be7842f07470b3d52d6607a40c32 > > Diff: https://git.reviewboard.kde.org/r/116980/diff/ > > > Testing > ------- > > Built, installed, visually inspected man page. > > > Thanks, > > Alex Merry > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
