Re: deprecate all of svn_wc.h ?

On Fri, Apr 9, 2010 at 13:49, Stefan Küng <tortoisesvn_at_gmail.com> wrote:
>...
> Going through the code of TSVN, I found a few places where we use functions
> defined in svn_wc.h:

Ooh! Thanks!

> * svn_wc_adm_probe_open3, used to get the svn_wc_adm_access_t required for
> svn_client_uuid_from_path()

Looks like we have an access-baton-free replacement
(svn_client_uuid_from_path2).

> * svn_wc_translated_file2
> * svn_wc_prop_list
> * svn_wc_is_adm_dir

These should be fine.

> * svn_wc_dup_status2

I think we're going to be revamping the status structure pretty hard. It may

> * svn_wc_get_pristine_copy_path (we don't use svn_wc_get_pristine_contents
> because that returns a stream, and we need a file)

Ugh. This one is particularly problematic.

Can I ask what you use it for, so that we can come up with a suitable
replacement?

> * svn_wc_set_adm_dir

Set? Is this for when you detect a working copy that uses _svn ?

> * svn_wc_match_ignore_list
>
>
> I hope I got all the svn_wc_ functions we use in TSVN listed here. A code
> search for this also returns hundreds of hits where we use the status struct
> and other defines/enums/structs which are returned and used by svn_client_
> functions, so I might have missed some functions in those hits.

Gotcha. I'll go grab a copy of the sources, and get a feel for those
struct bits. That will help to guide how we revamp the status system.

Thanks,
-g
Received on 2010-04-09 21:14:53 CEST