You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Stefan Sperling <st...@elego.de> on 2012/11/09 00:07:18 UTC

Re: svn commit: r1407279 - in /subversion/trunk/subversion: svnadmin/main.c tests/cmdline/svntest/main.py

On Thu, Nov 08, 2012 at 09:42:34PM -0000, cmpilato@apache.org wrote:
> Author: cmpilato
> Date: Thu Nov  8 21:42:34 2012
> New Revision: 1407279
> 
> URL: http://svn.apache.org/viewvc?rev=1407279&view=rev
> Log:
> Rather than continue the trend of accumulating new --pre-X.Y-compatible
> options to 'svnadmin create', make use of the new version parsing and
> comparison private functions to add a single new option that should
> work henceforth.

Nice!!!

> +  /* In 1.8, we figured out that we didn't have to keep extending this
> +     madness indefinitely. */

xactly! :)

Re: svn commit: r1407279 - in /subversion/trunk/subversion: svnadmin/main.c tests/cmdline/svntest/main.py

Posted by Mat Booth <ma...@wandisco.com>.
On 9 November 2012 11:13, Daniel Shahaf <d....@daniel.shahaf.name> wrote:
> Mat Booth wrote on Fri, Nov 09, 2012 at 10:07:25 +0000:
>> On 8 November 2012 23:07, Stefan Sperling <st...@elego.de> wrote:
>> >
>> > On Thu, Nov 08, 2012 at 09:42:34PM -0000, cmpilato@apache.org wrote:
>> > > Author: cmpilato
>> > > Date: Thu Nov  8 21:42:34 2012
>> > > New Revision: 1407279
>> > >
>> > > URL: http://svn.apache.org/viewvc?rev=1407279&view=rev
>> > > Log:
>> > > Rather than continue the trend of accumulating new --pre-X.Y-compatible
>> > > options to 'svnadmin create', make use of the new version parsing and
>> > > comparison private functions to add a single new option that should
>> > > work henceforth.
>> >
>> > Nice!!!
>> >
>> > > +  /* In 1.8, we figured out that we didn't have to keep extending this
>> > > +     madness indefinitely. */
>> >
>> > xactly! :)
>>
>>
>> Hmm, I just tried to look at this diff in the source viewer at
>> svn.apache.org because I am not subscribed to the commits list, but it
>> seems to crash. Does anyone else see a stack trace instead of a diff?
>>
>
> Fixed.  Something broke when we upgraded viewvc on svn.eu
>
>

Nice one, thanks.



-- 
Mat Booth
Software Engineer
WANdisco, Inc.
http://www.wandisco.com

Re: svn commit: r1407279 - in /subversion/trunk/subversion: svnadmin/main.c tests/cmdline/svntest/main.py

Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
Mat Booth wrote on Fri, Nov 09, 2012 at 10:07:25 +0000:
> On 8 November 2012 23:07, Stefan Sperling <st...@elego.de> wrote:
> >
> > On Thu, Nov 08, 2012 at 09:42:34PM -0000, cmpilato@apache.org wrote:
> > > Author: cmpilato
> > > Date: Thu Nov  8 21:42:34 2012
> > > New Revision: 1407279
> > >
> > > URL: http://svn.apache.org/viewvc?rev=1407279&view=rev
> > > Log:
> > > Rather than continue the trend of accumulating new --pre-X.Y-compatible
> > > options to 'svnadmin create', make use of the new version parsing and
> > > comparison private functions to add a single new option that should
> > > work henceforth.
> >
> > Nice!!!
> >
> > > +  /* In 1.8, we figured out that we didn't have to keep extending this
> > > +     madness indefinitely. */
> >
> > xactly! :)
> 
> 
> Hmm, I just tried to look at this diff in the source viewer at
> svn.apache.org because I am not subscribed to the commits list, but it
> seems to crash. Does anyone else see a stack trace instead of a diff?
> 

Fixed.  Something broke when we upgraded viewvc on svn.eu


> http://svn.apache.org/viewvc/subversion/trunk/subversion/svnadmin/main.c?r1=1407279&r2=1407278&pathrev=1407279
> 
> 
> 
> --
> Mat Booth
> Software Engineer
> WANdisco, Inc.
> http://www.wandisco.com

Re: svn commit: r1407279 - in /subversion/trunk/subversion: svnadmin/main.c tests/cmdline/svntest/main.py

Posted by Mat Booth <ma...@wandisco.com>.
On 8 November 2012 23:07, Stefan Sperling <st...@elego.de> wrote:
>
> On Thu, Nov 08, 2012 at 09:42:34PM -0000, cmpilato@apache.org wrote:
> > Author: cmpilato
> > Date: Thu Nov  8 21:42:34 2012
> > New Revision: 1407279
> >
> > URL: http://svn.apache.org/viewvc?rev=1407279&view=rev
> > Log:
> > Rather than continue the trend of accumulating new --pre-X.Y-compatible
> > options to 'svnadmin create', make use of the new version parsing and
> > comparison private functions to add a single new option that should
> > work henceforth.
>
> Nice!!!
>
> > +  /* In 1.8, we figured out that we didn't have to keep extending this
> > +     madness indefinitely. */
>
> xactly! :)


Hmm, I just tried to look at this diff in the source viewer at
svn.apache.org because I am not subscribed to the commits list, but it
seems to crash. Does anyone else see a stack trace instead of a diff?

http://svn.apache.org/viewvc/subversion/trunk/subversion/svnadmin/main.c?r1=1407279&r2=1407278&pathrev=1407279



--
Mat Booth
Software Engineer
WANdisco, Inc.
http://www.wandisco.com