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

RE: Re: [PATCH] new patch for asvn script

From: Scott Miller <Scott.Miller_at_prioritytech.com>
Date: 2006-10-31 20:28:23 CET

Actually, if I were to do something like this it would be best to check
which version of svn is being used, and if it is newer than X, use the
built in functionality and remove the symlink info etc., otherwise use
the updatedirsymlinks code.

So, what is the magic version X?

-Scott

-----Original Message-----
From: Peter Samuelson [mailto:peter@p12n.org]
Sent: Monday, October 30, 2006 5:52 PM
To: dev@subversion.tigris.org
Subject: Re: [PATCH] new patch for asvn script

[Scott Miller]
> o Commented out the code dealing with symbolic links since
subversion
> now has that capability built in.

Rather, you should change updatedirsymlinks() to look at existing
symlink properties and get rid of them in favor of just storing
symlinks directly. Otherwise there is no upgrade path for people
already using asvn.

As for commenting things out - in general, just remove code, don't
comment it out. That is why we have version control, so we can go back
and get code which is no longer in the tree.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Oct 31 20:27:34 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.