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

Re: [PATCH] propset can't delete properties

From: Karl Fogel <kfogel_at_collab.net>
Date: 2001-07-07 20:54:13 CEST

Ben Collins-Sussman <sussman@collab.net> writes:
> > I think the best solution is to not use propset for deleting properties,
> > and add a 'propdel' command instead, or a 'proprm' or whatever. Sound
> > okay?
>
> I'm -0 on this, but I guess it's a bikeshed UI issue.
>
> It feels a bit inconsistent, since according to all our C APIs, the
> way to remove a property is to "set it to NULL" -- I believe the same
> idea is present in WebDAV. So a 'propdel' command feels like a
> philosophical mismatch to me.
>
> Then again, most users don't know the API, so probably won't care.

I'm +1 on Joe's suggestion, but whether one is for or against his
solution, the problem he points out cannot be ignored. The `propset'
command as it is currently defined is ambiguous and cannot be used for
both setting and deleting properties.

In CVS, one could do "propset -d", analogous to "tag -d". But we
don't like to overload option letters the way CVS does. Seems to me
that propdel is the best available solution.

I'd like to apply the patch, but am holding off, so we have a chance
to consider other possible interfaces.

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:33 2006

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