Review Request 119830: Remove deprecated IProject methods
Miquel Sabaté Solà
mikisabate at gmail.com
Tue Mar 24 13:48:28 UTC 2015
> On March 24, 2015, 10:18 a.m., Kevin Funk wrote:
> > Would be nice if you could go through the other KDevelop-related repos and check if they still compile with this change.
>
> Dāvis Mosāns wrote:
> Any others should I check?
>
> Theese build fine:
>
> * kdevelop-pg-qt
> * kdev-clang
> * kdev-php
> * kdev-php-docs
> * kdev-python
> * kdev-qmljs
> * kdev-upload
> * kdev-valgrind
>
> These doesn't build
> because of this patch, I'll fix them:
>
> * kdev-cppcheck
> * kdev-qmake
>
> These doesn't build for me
> because of some other issue not related to this patch:
>
> * kdev-executebrowser
> * kdev-perforce
> * kdev-ruby
> * kdev-xdebug
>
> Miquel Sabaté Solà wrote:
> @D?vis kdev-ruby builds just fine for me with both gcc 4.9.2 and clang 3.6 (with and without your patch applied). What issues are you having ?
aaaand I was wrong :_) It should be ok now.
- Miquel
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119830/#review77992
-----------------------------------------------------------
On March 21, 2015, 4:19 a.m., Dāvis Mosāns wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119830/
> -----------------------------------------------------------
>
> (Updated March 21, 2015, 4:19 a.m.)
>
>
> Review request for KDevelop.
>
>
> Repository: kdevplatform
>
>
> Description
> -------
>
> Remove deprecated IProject methods. No need to keep them, when KUrl will be changed to QUrl signatures would be different anyway, so just remove now.
>
>
> Diffs
> -----
>
> interfaces/iproject.h bdfd66557dc9d9eebe4f8773e0428a8183ec1825
> plugins/execute/nativeappconfig.cpp 9c83db9e021201b0732ce186423a3d922458262d
> plugins/executescript/scriptappconfig.cpp 947cc464a42e3c18c38034287c93bdcb4ae8726c
> plugins/externalscript/externalscriptjob.cpp e65516d65e9315129bb8a9efc6d8897b4413cde6
> plugins/filetemplates/templateclassassistant.cpp 8681590b8e15ed1f3ceb15cba118733eb67c7e06
> plugins/grepview/grepdialog.cpp 4671c30bcdb75699d0cd474da78cfbf15eadf535
> plugins/projectmanagerview/projectmanagerview.cpp 95ae9a90273ec4f4be57fdc5e526e4416440788f
> plugins/projectmanagerview/vcsoverlayproxymodel.cpp 62e632c9c6e243d4a34626837f9232b3a3cc3f2a
> project/projectmodel.h e7513ff99cdb8f1724828e7e4d338c0fdcf1a7fa
> project/projectmodel.cpp 49cd984bdcbba661fedd8fbf42912b43fa28b733
> shell/project.h aec5be208c058a17540b3646dd4a8a0a12a29750
> shell/project.cpp e7e6efa462e8000c566af265d441464b5ab5cdce
> shell/projectcontroller.cpp 50bdce4b47c16a4e62c8b7014897a6ae12d6bba6
> tests/testproject.h 06757db24be0a356f4a0aee27f07899b6ea906b7
> tests/testproject.cpp 39a6d0be6505fcb7f5e130f27481fe02b89a7446
> vcs/models/brancheslistmodel.cpp 0737f8bcb8f855ac0901be6885428721218acefb
>
> Diff: https://git.reviewboard.kde.org/r/119830/diff/
>
>
> Testing
> -------
>
> Tests does pass :)
>
> but note that KDevelop must also be updated and currently it won't compile against this because it still uses these deprecated methods.
>
>
> Thanks,
>
> Dāvis Mosāns
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150324/4f122b1e/attachment.html>
More information about the KDevelop-devel
mailing list