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


From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2001-09-18 02:26:13 CEST

so in the process of making svn_path_canonicalize actually do more
than remove a trailing /, i've run into a small snag.

my new code removes redundant components (// and /./ at the moment,
well, dirsep dirsep and dirsep . dirsep actually, but anyway), and tha
tworks fine, but the problem is that it seems that in several places
we're passing url's to svn_path_canonicalize. of course, it's finding
the // after http: and turning it into a single /. so later on when we
parse the url we get screwed up results and crash.

now it doesn't seem right to have svn_path_canonicalize try to deal
with url's, since url's aren't really paths. would anyone mind the
addition of svn_url_canonicalize (or something similar) in libsvn_subr
which can call svn_path_canonicalize on the path portion?

attached is the change to svn_path_canonicalize, just so you can see
what i've got. don't use it though, or you'll crash trying to run
'svn update'.

garrett rooney                     Unix was not designed to stop you from 
rooneg@electricjellyfish.net       doing stupid things, because that would  
http://electricjellyfish.net/      stop you from doing clever things.

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:41 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.