Re: svn bisect

2011-06-21 Thread Arwin Arni
On Wednesday 22 June 2011 02:43 AM, Erik Huelsmann wrote: Hi Arwin, On Tue, Jun 21, 2011 at 9:45 AM, Arwin Arni wrote: 3. Will this feature be considered at all (if it is any good) or am I simply doing something to exercise my brain cells? Actually, I think it'd be a good idea to have a stan

Re: svn bisect

2011-06-21 Thread Erik Huelsmann
Hi Arwin, On Tue, Jun 21, 2011 at 9:45 AM, Arwin Arni wrote: > 3. Will this feature be considered at all (if it is any good) or am I simply > doing something to exercise my brain cells? Actually, I think it'd be a good idea to have a standardized command to have where all clients work alike. W

Re: svn bisect

2011-06-21 Thread Daniel Shahaf
So, to summarize, the motivations are * doesn't require perl/python on Windows * doesn't require third party scripts / extra packages on Unix * useable by API clients * canonical implementation Stefan Sperling wrote on Tue, Jun 21, 2011 at 12:03:22 +0200: > On Tue, Jun 21, 2011 at 12:55:15PM +030

Re: svn bisect

2011-06-21 Thread Daniel Shahaf
Stefan Sperling wrote on Tue, Jun 21, 2011 at 10:29:24 +0200: > On Tue, Jun 21, 2011 at 01:15:06PM +0530, Arwin Arni wrote: > > Hi All, > > > > I am currently trying to implement "svn bisect" subcommand. Yes, I > > know there are some good scripts out there that work, but it's not > > part of our

Re: svn bisect

2011-06-21 Thread Arwin Arni
On Tuesday 21 June 2011 03:11 PM, Julian Foad wrote: On Tue, 2011-06-21 at 11:37 +0200, Johan Corveleyn wrote: On Tue, Jun 21, 2011 at 11:34 AM, Arwin Arni wrote: On Tuesday 21 June 2011 02:51 PM, Daniel Shahaf wrote: Please convince me why having svn_client_bisect() is needed. There are alr

Re: svn bisect

2011-06-21 Thread Stefan Sperling
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 s

Re: svn bisect

2011-06-21 Thread Daniel Shahaf
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 CPA

Re: svn bisect

2011-06-21 Thread Julian Foad
On Tue, 2011-06-21 at 11:37 +0200, Johan Corveleyn wrote: > On Tue, Jun 21, 2011 at 11:34 AM, Arwin Arni wrote: > > On Tuesday 21 June 2011 02:51 PM, Daniel Shahaf wrote: > >> > >> Please convince me why having svn_client_bisect() is needed. > >> > >> There are already THREE scripts out there that

Re: svn bisect

2011-06-21 Thread Johan Corveleyn
On Tue, Jun 21, 2011 at 11:34 AM, Arwin Arni wrote: > On Tuesday 21 June 2011 02:51 PM, 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 Deb

Re: svn bisect

2011-06-21 Thread Stefan Sperling
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

Re: svn bisect

2011-06-21 Thread Arwin Arni
On Tuesday 21 June 2011 02:51 PM, 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

Re: svn bisect

2011-06-21 Thread Daniel Shahaf
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 feat

Re: svn bisect

2011-06-21 Thread Stefan Sperling
On Tue, Jun 21, 2011 at 02:40:10PM +0530, Arwin Arni wrote: > On Tuesday 21 June 2011 02:35 PM, Stefan Sperling wrote: > >On Tue, Jun 21, 2011 at 02:26:28PM +0530, Arwin Arni wrote: > >>>Stefan Sperling wrote: > I'd like to offer you commit access to a branch in our repository > so you work

Re: svn bisect

2011-06-21 Thread Arwin Arni
On Tuesday 21 June 2011 02:35 PM, Stefan Sperling wrote: On Tue, Jun 21, 2011 at 02:26:28PM +0530, Arwin Arni wrote: Stefan Sperling wrote: I'd like to offer you commit access to a branch in our repository so you work on this there if you like. I'd really appreciate that. Thanks for the suppor

Re: svn bisect

2011-06-21 Thread Stefan Sperling
On Tue, Jun 21, 2011 at 02:26:28PM +0530, Arwin Arni wrote: > >Stefan Sperling wrote: > >>I'd like to offer you commit access to a branch in our repository > >>so you work on this there if you like. > I'd really appreciate that. Thanks for the support. Good, I'll fire off an account request. Do yo

Re: svn bisect

2011-06-21 Thread Stefan Sperling
On Tue, Jun 21, 2011 at 10:00:40AM +0100, Julian Foad wrote: > My patch retrieves the list of operative revisions on the specified > subtree, and binary-searches through that list, not through all possible > revision numbers. Nice. I didn't read it thoroughly enough so :-)

Re: svn bisect

2011-06-21 Thread Julian Foad
Stefan Sperling wrote: > On Tue, Jun 21, 2011 at 09:35:54AM +0100, Julian Foad wrote: > > Here's one I wrote a few months back. I didn't send it here because I > > didn't think it was a serious proposition, just something fun to > > exercise my brain cells. :-) > > > > One thing "git bisect" doe

Re: svn bisect

2011-06-21 Thread Arwin Arni
On Tuesday 21 June 2011 02:05 PM, Julian Foad wrote: Here's one I wrote a few months back. I didn't send it here because I didn't think it was a serious proposition, just something fun to exercise my brain cells. :-) One thing "git bisect" does that's not implemented here is to support a third

Re: svn bisect

2011-06-21 Thread Stefan Sperling
On Tue, Jun 21, 2011 at 09:35:54AM +0100, Julian Foad wrote: > Here's one I wrote a few months back. I didn't send it here because I > didn't think it was a serious proposition, just something fun to > exercise my brain cells. :-) > > One thing "git bisect" does that's not implemented here is to

Re: svn bisect

2011-06-21 Thread Julian Foad
Here's one I wrote a few months back. I didn't send it here because I didn't think it was a serious proposition, just something fun to exercise my brain cells. :-) One thing "git bisect" does that's not implemented here is to support a third response, "unable to test", and skip such revisions.

Re: svn bisect

2011-06-21 Thread Stefan Sperling
On Tue, Jun 21, 2011 at 01:15:06PM +0530, Arwin Arni wrote: > Hi All, > > I am currently trying to implement "svn bisect" subcommand. Yes, I > know there are some good scripts out there that work, but it's not > part of our API. I figured this would be a decent addition to our > code. Here are a f