----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115744/#review50283 -----------------------------------------------------------
> Fix the real issue: Since I have https://git.reviewboard.kde.org/r/115210/ > applied > locally the install path contains C:/kf5/install/../../ProgramData/share. > This resulted in a wrong relative path in kdex.dtd at line 100 (8 times ../ > instead of 4). Just to be sure: the fix solves the issue even if the patch above (dropped) is not applied? - Luigi Toscano On Feb. 19, 2014, 7:45 p.m., Alexander Richardson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115744/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2014, 7:45 p.m.) > > > Review request for Documentation, KDE Frameworks and Luigi Toscano. > > > Repository: kdoctools > > > Description > ------- > > Fix projects using KDocTools on windows. > > If ${CMAKE_INSTALL_PREFIX}/${DATA_INSTALL_DIR} contained /../ elements > file(RELATIVE_PATH) would interpret these as valid directory names and > therefore return a relative path which has too many /../ > > REVIEW: 115744 > > > Diffs > ----- > > src/CMakeLists.txt 241d56ffdb9f5cf4859eaf06aad491b9e1ca94fa > > Diff: https://git.reviewboard.kde.org/r/115744/diff/ > > > Testing > ------- > > kservice didn't compile before this change, now it does (win32), doesn't > affect other platforms > > > Thanks, > > Alexander Richardson > > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.kde.org/pipermail/kde-doc-english/attachments/20140219/028afcbb/attachment.html>