Re: svn client api comments

Philip Martin wrote:
> Martin Hauner <hauner@web.de> writes:
>
>>Is there any reason why svn_client_add does take a const char* and
>>not a const apr_array_header_t* like revert and delete?
> Historical reasons; they all took single paths originally, add has
> never been updated to multiple paths.

I will take a look at it, if i understand enough of the code, i'll try
to provide a patch.

>>svn_copy: takes a single input file
>>svn_move(2): takes a single input file
>>
>>It would be nice if those two took a list of files to copy/move as input.
>>If i move or copy source files in a working copy i typically want to move
>>a header and its source file together.
>
> That's harder. There is 'destination logic' such that
>
> svn_client_copy "src" "dst"
>
> will create "dst/src" if "dst" is an existing directory, and "dst" if
> "dst" does not exist.
> [..]

If that's the only special case it would be possible to say that dst
has to exist if the input is more than one file.

-- 
Martin
Subcommander, http://subcommander.tigris.org
a x-platform Win32/Unix svn gui (qt) client & a text diff/merge tool.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org