[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: Stefan Sperling <stsp_at_elego.de>
Date: Tue, 15 Feb 2011 16:27:16 +0100

On Tue, Feb 15, 2011 at 02:49:12PM +0530, Noorul Islam K M wrote:
> 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

I haven't been following this closely. But as Hyrum points out,
it seems that more design work is needed before much coding can be done.

Branch or not, you'll need to find a full committer willing to help
with the design and review the implementation.

The problem with that is that most developers are currently focused
on working towards the 1.7 release. There is little room at the
moment for designing new features that aren't planned to appear in 1.7.

So maybe we can postpone work on this feature for later?

In the meantime, there are quite a number of issues with milestones
1.7.0 and 1.7-consider. Those are likely to catch more attention at
the moment, since everyone is focused on getting the release done.

See these two links:
http://subversion.tigris.org/issues/buglist.cgi?issue_status=UNCONFIRMED&issue_status=NEW&issue_status=STARTED&issue_status=REOPENED&target_milestone=1.7.0

http://subversion.tigris.org/issues/buglist.cgi?issue_status=UNCONFIRMED&issue_status=NEW&issue_status=STARTED&issue_status=REOPENED&target_milestone=1.7-consider

It would help tremendously to get any of these issues closed.

Stefan
Received on 2011-02-15 16:28:06 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.