[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: Noorul Islam K M <noorul_at_collab.net>
Date: Tue, 15 Feb 2011 14:49:12 +0530

Daniel Becroft <djcbecroft_at_gmail.com> writes:

> 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.
>

That means we need to have two options, --ignore-properties and
--properties-only. Can I work on these two on a private branch?

Thanks and Regards
Noorul
Received on 2011-02-15 10:21:22 CET

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