Make kdev-executebrowser work with KDevelop 4.7
AbandonedPublic

Authored by kfunk on Oct 18 2015, 1:11 PM.

Details

Reviewers
mwolff
zhigalin
Group Reviewers
KDevelop
Summary

As there was no API changes all we need is an updated .desktop file to make KDevelop sure that he can load this plugin

Diff Detail

Repository
R32 KDevelop
Lint
Lint Skipped
Unit
Unit Tests Skipped
zhigalin updated this revision to Diff 1016.Oct 18 2015, 1:11 PM
zhigalin retitled this revision from to Make kdev-executebrowser work with KDevelop 4.7.
zhigalin updated this object.
zhigalin edited the test plan for this revision. (Show Details)
zhigalin added a reviewer: KDevelop.
zhigalin set the repository for this revision to R32 KDevelop.
zhigalin added a project: KDevelop.
Restricted Application added a subscriber: kdevelop-devel. ยท View Herald TranscriptOct 18 2015, 1:11 PM
zhigalin updated this object.Oct 18 2015, 1:12 PM
mwolff accepted this revision.Oct 25 2015, 3:37 PM
mwolff added a reviewer: mwolff.
mwolff added a subscriber: mwolff.

lgtm. Are you maybe interested in maintaining this plugin? Porting it to KDevelop 5 and having a release for that would be awesome, I think.

This revision is now accepted and ready to land.Oct 25 2015, 3:37 PM
In D429#8741, @mwolff wrote:

lgtm. Are you maybe interested in maintaining this plugin? Porting it to KDevelop 5 and having a release for that would be awesome, I think.

I'm interested, but now I can't.
Maybe in 2016, if no one else will do it.

the master branch already contains this change btw. so this can be closed.

kfunk requested changes to this revision.Nov 2 2015, 9:37 PM
kfunk added a reviewer: kfunk.
kfunk added a subscriber: kfunk.

@mwolff: Please push master as 1.7 branch in kdev-executebrowser.git for consistency (I can't b/c of permissions).

@zhigalin: Please abandon this patch, it's already in master.

This revision now requires changes to proceed.Nov 2 2015, 9:37 PM
kfunk commandeered this revision.Nov 5 2015, 12:12 PM
kfunk abandoned this revision.
kfunk edited reviewers, added: zhigalin; removed: kfunk.

Closing, already committed