subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Becroft <djcbecr...@gmail.com>
Subject Re: Patches pending review
Date Tue, 15 Feb 2011 09:04:43 GMT
On Tue, Feb 15, 2011 at 9:48 AM, Hyrum K Wright <hyrum@hyrumwright.org>wrote:

> On Mon, Feb 14, 2011 at 11:33 AM, Noorul Islam K M <noorul@collab.net>
> wrote:
> >
> > See below in line more information about the patches.
> >
> > Noorul Islam K M <noorul@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.

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message