On Tue, Jun 21, 2011 at 12:55:15PM +0300, Daniel Shahaf wrote: > Stefan Sperling wrote on Tue, Jun 21, 2011 at 11:34:46 +0200: > > On Tue, Jun 21, 2011 at 12:21:27PM +0300, Daniel Shahaf wrote: > > > Please convince me why having svn_client_bisect() is needed. > > > > > > There are already THREE scripts out there that implement cmdline 'svn > > > bisect': > > > one on CPAN, one in Peter's Debian package, one that Julian just posted. > > > (And we got one sent to the issue tracker a while ago, but the CPAN one > > > was more featureful) > > > > Julian's implementation is, by his own account, incomplete. > > > > There is precedence here. Let me provide two examples: > > We have svnmerge.py. Why is there merge-tracking in svn now? > > We have 'svn patch' now. Wasn't UNIX patch good enough? > > > > No, because it won't handle properties and renames.
Another primary reason was that Windows users now have a patch feature withouth installting third party software. > What value does > having 'svn bisect' in the core add over the existing external > 'svn-bisect' implementations? It's handy for Windows users or even Unixy types who don't want to install zillions of helper scripts. I am not even going to consider ever trying to fix a bug in an svn-bisect implementation that was written in Perl. git-svn was enough experience of that, thank you very much. I suppose Arwin could share his ideas in detail once they have developed, and explain advantages over the existing scripts.
