On Mon, Feb 15, 2010 at 6:10 PM, Vladimir Prus <span dir="ltr"><<a href="mailto:ghost@cs.msu.su">ghost@cs.msu.su</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On Monday 15 February 2010 20:06:27 David Nolden wrote:<br>
<br>
> Am Montag 15 Februar 2010 17:26:58 schrieb Aleix Pol:<br>
> > Hi,<br>
> > As I said I have been trying to get the git plugin to work today.<br>
> > I didn't know what was the state of it. I have to say it's not that bad. I<br>
> > didn't try any weird cases but at least it's something to work on.<br>
> ><br>
> > The biggest problem I have found it's that the reviewpatch infrastructure<br>
> > doesn't cope properly with git's diff format. I'm not sure how to address<br>
> > that, any idea?<br>
> ><br>
> > Thanks,<br>
> > Aleix<br>
> I guess the main problem is that git usually prepends "a/" and "b/" to the<br>
> paths within the patch, so one needs to use "-p2" or something like that when<br>
> using the patch utility.<br>
><br>
> However I think the used diff-library doesn't support such kinds of prefixes.<br>
><br>
> I guess the simplest solution would be processing the patches from git,<br>
> changing the paths in a way so that they equal the paths used by svn.<br>
<br>
</div>Or pass the --no-prefix option to 'git diff'?<br>
<br>
- Volodya<br>
<div><div></div><div class="h5"><br>
<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>Ok, I managed to port libdiff2 back from kdesdk and now we can read 
patches from git. yay!<br>
<br>
now: any reason why the VcsDiff class doesnt provide the patch base?<br>
makes sense to me...<br>
<br>
I'll add that, just to report :P<br>
<br>
Thanks for the answers!<br><br>PD: Btw, this prefix stuff is not a problem for kompare but is for patchreview, maybe this should be sorted out... -.- we'll see.<br><br>