Review Request 123325: Drop dependency of KIO from kdeconnectcore
Aleix Pol Gonzalez
aleixpol at kde.org
Mon May 4 23:21:19 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123325/
-----------------------------------------------------------
(Updated May 4, 2015, 11:21 p.m.)
Status
------
This change has been marked as submitted.
Review request for kdeconnect.
Changes
-------
Submitted with commit 11996f56ca4f4bc214ebc4b895a519ff55573832 by Aleix Pol to branch master.
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 a2e2cfc
core/daemon.cpp a063589
core/filetransferjob.h 973bd3a
core/filetransferjob.cpp 27ed8e8
daemon/CMakeLists.txt 852e779
daemon/kdeconnectd.cpp c43bc9b
Diff: https://git.reviewboard.kde.org/r/123325/diff/
Testing
-------
Works for me.
Tested sending from my Android device to the laptop.
Thanks,
Aleix Pol Gonzalez
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150504/aeee92f5/attachment.html>
More information about the KDEConnect
mailing list