Review Request 124544: Set a valid url for UDS_URL, basically one that includes files://

Aleix Pol Gonzalez aleixpol at kde.org
Fri Jul 31 09:50:24 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124544/#review83219
-----------------------------------------------------------

Ship it!


Ship It!

- Aleix Pol Gonzalez


On July 31, 2015, 11:45 a.m., Àlex Fiestas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124544/
> -----------------------------------------------------------
> 
> (Updated July 31, 2015, 11:45 a.m.)
> 
> 
> Review request for kdeconnect.
> 
> 
> Repository: kdeconnect-kde
> 
> 
> Description
> -------
> 
> In Qt5 QUrl need to be created by calling fromLocalFile to handle paths such /root/foo.txt.
> 
> So KIO apparently broke compatibility with setting such paths in UDS_URL and we have to pass instead files:///root/foo.txt
> 
> Altought this might be a bug in KIO, we want to work with all KF5 based versions of plasma and this patch will just do that.
> 
> 
> Diffs
> -----
> 
>   kio/kiokdeconnect.cpp 9d40ee5 
> 
> Diff: https://git.reviewboard.kde.org/r/124544/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Àlex Fiestas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150731/26263252/attachment.html>


More information about the KDEConnect mailing list