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

Re: automatic keyword un-expansion on 'svn {ps,pd,pe} svn:keywords'?

From: John Peacock <jpeacock_at_rowman.com>
Date: 2006-03-15 18:15:51 CET

kfogel@collab.net wrote:
> I think that's reasonable. Having a propdel result in a textual diff
> is rather unexpected, and in some sense the in-file expanded portion
> of the keyword is "part of" the property. We'll have to do it
> carefully, of course, because the working file might have local mods.

CAVEAT: Not a developer, but someone interested in keywords in general.

Personally I think it would be _much_ more surprising to automatically
unexpand the keyword when the matching property is deleted. If someone
sets one of the expanding properties and the unsets it, either they made
a mistake in the first place (and should revert that expanded text on
their own) or they meant to do that to freeze the expanded text (and
hence the diff is completely legit).

For example, a site policy is that all new files are to be created with
svn:keywords "Author" set and then committed. Then, the svn:keywords
"Author" is to be removed and the file committed again, thus orphaning
the original Author information in the file for all time. Sure this is
stupid, but it does correspond to some behavior that has been used in
CVS in the past (although there I believe it was freezing $Id$ on tags).

My 2 cents

John

-- 
John Peacock
Director of Information Research and Technology
Rowman & Littlefield Publishing Group
4501 Forbes Boulevard
Suite H
Lanham, MD  20706
301-459-3366 x.5010
fax 301-429-5748
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 15 18:16:26 2006

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.