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

Re: [TSVN] can't set svn:externals property to reference a directorywith a space

From: Peter McNab <mcnab_p_at_melbpc.org.au>
Date: 2005-08-19 04:31:06 CEST

Stefan Küng wrote:

> Simon Large wrote:
>
>> Stefan Küng wrote:
>>
>>> URLs in the svn:externals property *must* be properly URI encoded.
>>> So instead of a space, you have to write %20. Then it will work.
>>
>>
>>
>> This also came up recently (5/8/2005) with the relocate command.
>>
>> In both cases we could improve matters by doing a conditional escape
>> of the URL before using it (before relocate, or before setting
>> svn:externals)
>>
>> Two alternatives:
>>
>> 1. Escape the URL *unless* it already contains '%'.
>> 2. Escape the URL if it contains an illegal char *except* '%'.
>
>
> We already do exactly that in various other places. But for externals,
> you must know the URL you're pointing to anyway (and we would have to
> parse the entered string first to find the URL, and doing that in the
> shell extension without the MFC classes...)
> And for the relocate command: we do it there already.
>
> Stefan
>
Maybe an addition to the *Help* examples would be advisable to make this
clear. e.g.

third-party/sounds http://sounds.red-bean.com/repos
third-party/skins http://skins.red-bean.com/repositories/skinproj

Note. Replace spaces in path/filenames with %20
third-party/quick%20graphs http://graphics.red-bean.com/repositories/fast%20graphics

Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Fri Aug 19 04:31:47 2005

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