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

Re: Patches pending review

From: Daniel Becroft <djcbecroft_at_gmail.com>
Date: Tue, 15 Feb 2011 19:04:43 +1000

On Tue, Feb 15, 2011 at 9:48 AM, Hyrum K Wright <hyrum_at_hyrumwright.org>wrote:

> On Mon, Feb 14, 2011 at 11:33 AM, Noorul Islam K M <noorul_at_collab.net>
> wrote:
> >
> > See below in line more information about the patches.
> >
> > Noorul Islam K M <noorul_at_collab.net> writes:
> ...
> >> 3. Issue 3690
> >>
> >> http://svn.haxx.se/dev/archive-2011-01/0414.shtml
> >>
> >> This is actually an enhancement and there were two approaches. One
> >> that I initially submitted and another one suggested by Hyrum. This
> >> thread has three to four patches which are pending review. May be it
> >> is not getting reviewed because it is not yet finalized how are are
> >> we going to proceed.
> >>
> >
> > Above thread contains patches for issue 3690 which adds an option
> > "--ignore-properties" to log command so that user can ignore revisions
> > which has only property changes.
>
> I've said multiple times (both in regards to this, as well as the
> ignore-mergeinfo-log branch) that I'd appreciate some discussion
> surrounding the applicability of these features before committing
> them. Part of me sees the use, as they solve a real usability problem
> in my own life, but another part of me wonders if these are specific
> fixes for which other (non-core) solutions would be appropriate.
>
> Maybe I should spearhead that discussion, since I'm the guy who wants
> it so badly, but unfortunately my supply of tuits has been quite low
> lately. I'm just not in any rush to get this work on trunk, as if we
> do add this functionality, I'd like to do it for as many subcommands
> as is reasonable, all in the same release. And that ain't happening
> before 1.7 (imho).
>
> -Hyrum
>

My 0.02c - there was a question raised on the users@ list, regarding the
ability to see the changes that *only* relate to properties. The use case
was seeing the log for svn:externals changes on a directory.

I guess this is the opposite of the above request: one where we can exclude
all property changes, the other where we want nothing but property changes.

Cheers,
Daniel B.
Received on 2011-02-15 10:05:40 CET

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.