It's not useful at all and looks really like he was in a rush to get all plugins done, for the moment I'll just rename DVcsJob to CmdVcsJob, it annoys to me to see that D' (standing for distributed) whereas the class has nothing to do with its distributedness..<div>
<br><br><div class="gmail_quote">On Fri, Aug 20, 2010 at 3:08 PM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 20.08.10 14:46:45, Aleix Pol wrote:<br>
> And yes, it should work like that but the guy who made that support made<br>
> some (wrong) assumptions on how these plugins should be implemented and on<br>
> plugins inheriting DVcsPlugin it's not possible. I've spent most of my time<br>
> fixing that and it's not over yet. More to come.<br>
<br>
</div>For 4.1 its too late, but feel free to completely drop the dvcsplugin<br>
uselessness in master once 4.1 has branched off. I don't think this<br>
abstraction layer is necessary or even useful, at least not in its current<br>
form.<br>
<br>
Andreas<br>
<br>
--<br>
Don't hate yourself in the morning -- sleep till noon.<br>
<font color="#888888"><br>
--<br>
</font><div><div></div><div class="h5">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>