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

Re: [random line of thought] Hiding svn:mergeinfo

From: Paul Burba <ptburba_at_gmail.com>
Date: Wed, 18 May 2011 09:51:17 -0400

On Wed, May 18, 2011 at 5:41 AM, Hyrum K Wright <hyrum_at_hyrumwright.org> wrote:
> On Wed, May 18, 2011 at 9:32 AM, Bert Huijben <bert_at_qqmail.nl> wrote:
>> It would completely break mergetracking for about a dozen reasons :(
>>
>> I'll answer more detailed later, but this would just remove the
>> property everywhere.
>>
>> And if you could hide it in a better way while still storing it in the
>> wc, you would always forget to commit your mergeinfo only changes.
>>
>> Bert Huijben (Cell phone) From: Daniel Shahaf
>> Sent: woensdag 18 mei 2011 8:55
>> To: dev_at_subversion.apache.org
>> Subject: [random line of thought] Hiding svn:mergeinfo
>> Mike remarked that svn:mergeinfo should never have been user-visible...
>>
>> So: what if we made svn_property_kind(NULL, SVN_PROP_MERGEINFO) return
>> a value other than svn_prop_regular_kind, at least when both server and
>> client were ≥1.7?  Would that have any useful side-effects?

Hi Daniel,

I'm not sure of all the implications of doing this are, but quick look
reveals that this would either

1) (As Bert says) Break things, e.g. commit, diff

Or

2) Do nothing. Mergeinfo would still be dealt with normally, e.g. svn
merge, pl, pg, st

In fact I was hard pressed to find a *useful* outcome from this
(though I confess it was a brief look).

>> [I suspect it might, but want to hear others' thoughts before laying
>> out my own.]
>
> I don't think hiding it completely is the way to go, but there has
> been some half-implemented code to filter propchanges from various
> operations.  Completing this, and enabling it by default on selected
> commands may scratch the same itch.

I assume Hyrum is speaking of:

^/subversion/branches/ignore-mergeinfo
^/subversion/branches/ignore-mergeinfo-log

Paul
Received on 2011-05-18 15:51:46 CEST

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.