Review Request 120312: Port .cpp files from KDELibs4Support

Jakub Caban kuba at whyblack.pl
Tue Sep 30 19:46:01 UTC 2014


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

(Updated Sept. 30, 2014, 7:46 p.m.)


Review request for KDevelop.


Changes
-------

Fix issues and re-enable tests directory.


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 
  tests/CMakeLists.txt f4bc1a8 
  tests/permissions.cpp 8735c8d 
  uploadjob.cpp c58db45 
  uploadpreferences.cpp 0949c92 
  uploadpreferences.ui ac4c679 
  uploadprofiledlg.h d363190 
  uploadprofilemodel.cpp 16847b2 
  uploadprojectmodel.h 8379810 
  uploadprojectmodel.cpp 30db851 
  uploadprofiledlg.cpp ff234bd 
  uploadprofileitem.h b17187a 
  uploadprofileitem.cpp ee4610e 

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/20140930/43146602/attachment.html>


More information about the KDevelop-devel mailing list