[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1698359 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

From: Stefan Sperling <stsp_at_elego.de>
Date: Wed, 2 Sep 2015 16:49:23 +0200

On Wed, Sep 02, 2015 at 04:01:08PM +0200, Johan Corveleyn wrote:
> ISTR that in this community we try to treat a veto as a signal that
> further discussion is needed, or that more work is needed to resolve
> the question / issue. You may ask / suggest a revert if you think
> that's best, but it's mainly up for discussion how to best resolve
> things (and other avenues, such as further commits, may also resolve
> the issue).
>
> Of course, in some circumstances there is more pressure to revert
> quickly (e.g. when the build is broken, or other developer's work is
> blocked for other reasons, or ...). But usually we try to work our way
> towards consensus without "forcing" a revert.
>
> On the other hand, it might have been better for Stefan to wait for
> your reaction first, and discuss the issue and a possible solution,
> before making further changes ...
> It's not like this is such an urgent matter that it cannot wait for a
> couple of days ...
>
> Just my 2 cents,
> --
> Johan

+1

Thank you for writing this!
Received on 2015-09-02 16:49:48 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.