[Differential] [Updated] D1497: [kioslave/desktop] Ensure there are no multiple / when creating the file path
dfaure (David Faure)
noreply at phabricator.kde.org
Tue Apr 26 19:32:10 UTC 2016
dfaure added a comment.
I'm pretty sure it should work. What changed in QUrl is that setPath used to normalize the path and doesn't do that anymore.
(qtbase commit 2e1de7f3c4ca). So in 5.5 the double slash got simplified to a single slash, while in 5.6 it stays and breaks the test.
Not putting a double slash in the first place will work with both Qt versions.
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
BRANCH
fix-kioslave-desktop
REVISION DETAIL
https://phabricator.kde.org/D1497
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: graesslin, Plasma, hein, dfaure
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160426/e782f4af/attachment.html>
More information about the Plasma-devel
mailing list