Re: [PATCH] 'svn st --ignore-prop' (was Re: Proposal: --no-props switch for 'log', 'st' and 'diff')

On Thu, Sep 4, 2008 at 7:37 PM, Julian Foad <julianfoad_at_btopenworld.com> wrote:
> On Thu, 2008-09-04 at 19:07 +0400, Ivan Zhakov wrote:
>> On Thu, Sep 4, 2008 at 5:55 PM, Hyrum K. Wright
>> <hyrum_wright_at_mail.utexas.edu> wrote:
>> > Hyrum K. Wright wrote:
>> >> I'm currently investigating a generic '--ignore-prop' switch, initially for 'svn
>> >> st'. It is being written in such a way that we could easily alias
>> >> '--ignore-mergeinfo' or '--forget-merges' or
>> >> '--dont-get-in-my-way-i'm-trying-to-do-real-work' to mean '--ignore-prop
>> >> svn:mergeinfo'. I hope to have a proof-of-concept patch done in a few days.
>> >
>> > As promised, here's a patch which implements the above. It completely
>> > implements the functionality which it claims to, but there is still future work
>> > to be done to flesh out the feature. This does not add a '--ignore-mergeinfo'
>> > alias as discussed above, but adding such would be trivial.
>> >
>> > I'd appreciate any feedback, but if there aren't serious objections, I'll commit
>> > the patch in the next day or two.
>> >
>> Hi, sorry I've missed discussion in original thread, but I'd like to
>> share my thoughts about this issue. My opinion that Subversion should
>> handle svn:mergeinfo as special property by default and never show
>> them in svn status and svn commit, except --show-mergeinfo option
>> used.
>>
>> My idea that svn:mergeinfo is really special property, we use
>> properties for storing mergeinfo just to do not invent another storage
>> mechanism in libsvn_wc. Please correct me if I'm wrong.
>
> Please refer to the earlier messages in the thread, starting here:
> <http://svn.haxx.se/dev/archive-2008-09/0008.shtml>, where questions
> were raised about how users would forget to check in changes to their
> mergeinfo if they aren't told that such changes exist. We need answers
> to those concerns before you can say Subversion should never show them.
>
Hi Julian,

Sorry I missed this point. Thanks for pointing it. In that case I'm +1
on concept of --ignore-prop switch and alias --ignore-mergeinfo

I've also reviewed patch itself.

-- 
Ivan Zhakov
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org