Re: [PATCH] svncopy.pl fails with spaces in file paths.

Michael W Thelen wrote:
> Martin Tomes wrote:
>> - SVNCall( "propset", "--file", $tmpfile, "svn:externals",
>> $work_dir );
>> + SVNCall( "propset", "--file", '"' . $tmpfile . '"',
>> "svn:externals",
>> + '"' . $work_dir . '"' );
> I have not used svncopy.pl, but it looks like anything that calls
> SVNCall with a pathname may fail for the same reason. Would it be
> better to guard the parameters inside the SVNCall subroutine itself,
> with something like this?
>
> ==================================================================
> --- contrib/client-side/svncopy.pl.in (revision 21885)
> +++ contrib/client-side/svncopy.pl.in (local)
> @@ -686,7 +686,7 @@
> {
> my ( $command, @options ) = @_;
>
> - my @commandline = ( $svn, $command, @svn_options, @options );
> + my @commandline = ( $svn, $command, @svn_options, map {"\"$_\""}
> @options );
>
> info( " > ", join( " ", @commandline ), "\n" );

I did think of doing that but wasn't brave enough to make that change!
If any of our readers understand svncopy.pl and are willing to endorse
this approach would they please step forward.

-- 
Martin Tomes
echo 'martin at tomes x org x uk'\
  | sed -e 's/ x /\./g' -e 's/ at /@/'
Visit http://www.subversionary.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org