On Thu, Sep 11, 2008 at 1:33 PM, Julian Foad <julianfoad_at_btopenworld.com> wrote:
> On Thu, 2008-09-11 at 11:38 -0500, Hyrum K. Wright wrote:
>> As for the generality, I can imagine, though I don't know of any personally,
>> circumstances where projects use custom properties on files for some purpose,
>> use like we use properties for mergeinfo. Why *not* provide the ability for
>> users to filter arbitrary properties, especially since it doesn't require any
>> additional work in the implementation?
>
> Why not? Only reasons like: If there's no real gain (which isn't yet
> known), then the growth in out API and UI and implementation would be
> just a drain. If this is to be a real feature, it needs to be "complete"
> like, I suggest, having the converse operation of "show only properties
> P1, P2, ...". If this is to be frequently used for ignoring mergeinfo,
> there ought to be an easier-to-type command than "svn st
> --ignore-prop=svn:mergeinfo ...". We need to consider whether to provide
> a corresponding configuration option.
>
> None of these are showstoppers, but they need considering.
I am not 100% sold on this either, but there is one benefit to the
generic approach Hyrum is taking. My argument against this, is that I
feel like we are putting a band-aid on the problem instead of fixing
it. The fix would be to look for a better way to store this
information than properties. If we did that, then it would hopefully
remove the need for this change. So getting back to my first
sentence, if the UI is generic, then at least the general feature
still has some value in terms of being in the UI, even if no one has a
need for it any longer.
Does --ignore-prop=* or ALL or something like that work? Maybe
--ignore-prop could even have VAL be optional and mean ALL if none is
specified?
--
Thanks
Mark Phippard
http://markphip.blogspot.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-11 19:45:53 CEST