First of an off-topic question: Did you receive all the various editing
stages, too? Or is it just my mail-program mis-displaying something?<br><br><div class="gmail_quote">2009/3/27 Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Can you please use diff -u  for patches? Makes them easier to read - IMHO.<br>
</blockquote><div>Okay, I will prepare the full patch, and post it on
the reviewboard. I thought since this would become a fairly
wide-reaching patch<br>I'd rather distill the essence, as it otherwise could scare people.<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

While symmetry of API is good, I don't like a parameter that does nothing.<br>
If you remove a directory X from vcs control then that is always recursive.</blockquote><div> </div><div>Certainly, in that case the flag would be redundant. I will update the test-case to test that behaviour.<br> </div>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">No, the vcsmapping was introduced to support systems where one can have<br>
multiple sources checked out into different local locations. This is<br>
something perforce supports, so you'd say:<br>
<br>
put <a href="http://repo/foo/bar" target="_blank">http://repo/foo/bar</a> into /home/me/projectfoo/bar<br>
put <a href="http://repo/baz" target="_blank">http://repo/baz</a> into /home/me/projectfoo/baz<br>
<br>
in perforce. And the Matthew who back then created the interfaces with me<br>
is interested in writing perforce support at some point.<br>
</blockquote></div><br>This may be a reason why VcsMapping came into existence, but it doesn't negate the problems I cited.<br>What does one gain by batching all the checkouts? And is it sensible to require all but one VCS to re-implement said capability?<br>
<br><br>Fabian<br>