Review Request 123588: Port kdelibs4configmigratortest to QStandardPaths, so it works on other platforms than XDG.
Laurent Montel
montel at kde.org
Sat May 2 08:37:19 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123588/#review79775
-----------------------------------------------------------
autotests/kdelibs4configmigratortest.cpp (line 85)
<https://git.reviewboard.kde.org/r/123588/#comment54638>
You test after copy file ?
It's perhaps more logical to test before no ?
- Laurent Montel
On mai 2, 2015, 8:17 matin, David Faure wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123588/
> -----------------------------------------------------------
>
> (Updated mai 2, 2015, 8:17 matin)
>
>
> Review request for KDE Frameworks and Laurent Montel.
>
>
> Repository: kcoreaddons
>
>
> Description
> -------
>
> Port kdelibs4configmigratortest to QStandardPaths, so it works on other platforms than XDG.
>
>
> Diffs
> -----
>
> autotests/kdelibs4configmigratortest.cpp 0affd2a6bc86f8e4cad04dd662e1298d34b1e7c1
>
> Diff: https://git.reviewboard.kde.org/r/123588/diff/
>
>
> Testing
> -------
>
> Still passes on Linux, we'll see if the CI says it fixed it for Mac.
>
> Failure before the fix:
> https://build.kde.org/view/Frameworks%20kf5-qt5/job/kcoreaddons%20master%20kf5-qt5/PLATFORM=OSX,compiler=clang/17/testReport/junit/%28root%29/TestSuite/kdelibs4configmigratortest/
>
>
> Thanks,
>
> David Faure
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150502/a1c06835/attachment.html>
More information about the Kde-frameworks-devel
mailing list