<div class="gmail_quote">On Fri, Aug 20, 2010 at 2:29 PM, David Nolden <span dir="ltr"><<a href="mailto:zwabel@googlemail.com">zwabel@googlemail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I think the git support was already quite useful after my last fixes a<br>
few weeks ago, the only problem was that there were some menu-entries<br>
that simply didn't work (I didn't update yet to see if you fixed<br>
those). If there is still such menu-entries, then I think they should<br>
simply be removed (also there were 2 redundant "Revision History"<br>
entries of which only one was functional). The vcs-support should<br>
allow the vcs-plugins to specify what functions they properly support,<br>
and only show menu-actions for those.<br>
<br>
Anyway all in all, I'm all for moving the git plugin into<br>
kdevplatform, as it is very important.<br>
<br>
Greetings, David<br>
<div><div></div><div class="h5"><br>
--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</div></div></blockquote></div><br><div>Yep, they were Update/Pull/Push that weren't working. Now they are (Note: behaviour of update is a little weird, depending on what revision we're updating it pulls... hmmm).</div>
<div><br></div><div>Also there's this Revision History. I didn't remove them because it provides a little more information than the regular history view, probably they should be merged, or something. If nobody likes the view we can nuke it of course, but I'd prefer you to look at it. I'm not pronouncing myself on that because the code behind that feature is so shitty I would send it to mars, but well if we like the feature we can clean it up.</div>
<div><br></div><div>And yes, it should work like that but the guy who made that support made some (wrong) assumptions on how these plugins should be implemented and on plugins inheriting DVcsPlugin it's not possible. I've spent most of my time fixing that and it's not over yet. More to come.</div>
<div><br></div><div>Cheers!</div><div>Aleix</div>