Review Request 120312: Port .cpp files from KDELibs4Support
Jakub Caban
kuba at whyblack.pl
Tue Sep 23 21:20:44 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120312/
-----------------------------------------------------------
(Updated Sept. 23, 2014, 9:20 p.m.)
Review request for KDevelop.
Changes
-------
Port away from deprecated KDevPlatform methods. In the meantime lear how awesome KDevelop::Path is and use where relevant for now.
Repository: kdev-upload
Description
-------
Mostly simple KUrl->QUrl changes with just a few harder corners like porting to QDialog or KUrl::relativePath(...). Removing KDELibs4support from CMakeList.txt is impossible now as some weird dependency forces profile dialog to link against it. Probably something with .ui files? I will investigate when we'll have this part done.
Diffs (updated)
-----
CMakeLists.txt 3428c80
kdevuploadplugin.cpp 5261978
profilesfiletree.h c595f32
profilesfiletree.cpp 2c19df0
uploadjob.cpp c58db45
uploadpreferences.cpp 0949c92
uploadprojectmodel.cpp 30db851
Diff: https://git.reviewboard.kde.org/r/120312/diff/
Testing
-------
Code compiles and works with 2 regressions:
- Directory chooser for profile settings always opens user home folder instead of remote view
- upload job always reports 'Upload cancelled' even after succesfull upload
Thanks,
Jakub Caban
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140923/aa4be01f/attachment.html>
More information about the KDevelop-devel
mailing list