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

Re: [PATCH] Add --dry-run flag to "svn update" client command

From: Stefan Sperling <stsp_at_elego.de>
Date: Wed, 2 Mar 2011 15:35:48 +0100

On Wed, Mar 02, 2011 at 04:17:33PM +0200, Daniel Shahaf wrote:
> Bert Huijben wrote on Wed, Mar 02, 2011 at 11:14:24 +0100:
> > > -----Original Message-----
> > > From: Arwin Arni [mailto:arwin_at_collab.net]
> > > Please review this and share your thoughts.
> >
> > I don't think this is the way we should implement this.
>
> Which is precisely why one should contact the mailing list BEFORE
> writing a patch. Especially a large/long patch such as this one.

Well, Arwin said that he did this partly to learn about the update
editor. So there's still value in having written that patch, even
if it doesn't end up being applied.
I've written a lot of diffs that never went anywhere. I never
regretted this because I usually learned something new along the way.
Received on 2011-03-02 15:36:45 CET

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