Review Request 123325: Drop dependency of KIO from kdeconnectcore
Albert Vaca Cintora
albertvaka at gmail.com
Sat Apr 11 04:53:43 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123325/#review78781
-----------------------------------------------------------
I love this approach. It doesn't work though :P
- Albert Vaca Cintora
On abr. 10, 2015, 9:16 a.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123325/
> -----------------------------------------------------------
>
> (Updated abr. 10, 2015, 9:16 a.m.)
>
>
> Review request for kdeconnect.
>
>
> Repository: kdeconnect-kde
>
>
> Description
> -------
>
> Use QNetworkAccessManager for storing the files we receive instead of directly KIO, then provide the KIO-based QNetworkAccessManager from the daemon. This way we'll still get the KIO network-transparency, but the library only depenends on QtNetwork, which will be easier to get in some platforms.
>
> After this change, we only depend on KConfig, KI18n and KCoreAddons, which are tier1 and really easy to work with on any platform.
>
>
> Diffs
> -----
>
> CMakeLists.txt dbaa039
> core/CMakeLists.txt f465959
> core/daemon.h 654441f
> core/filetransferjob.h 973bd3a
> core/filetransferjob.cpp 27ed8e8
> daemon/CMakeLists.txt 852e779
> daemon/kdeconnectd.cpp 25632fd
>
> Diff: https://git.reviewboard.kde.org/r/123325/diff/
>
>
> Testing
> -------
>
> None, don't give a ship it. I only worked on it on the airplane, so I couldn't test it. If somebody wants to help with the testing, I'll welcome it.
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150411/7581c5f3/attachment.html>
More information about the KDEConnect
mailing list