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

Semantics of --depth: should define WC-depth for omitted-items?

From: Rui, Guo <timmyguo_at_mail.ustc.edu.cn>
Date: Thu, 10 Apr 2008 00:37:01 +0800

Hi all,
This is once again about the following examples in sparse-directories.txt.
<==============QUOTE================>
   svn up Awc/B:

       Since B is not yet checked out, add it at depth infinity.

       Awc/.svn/entries this_dir depth-empty
                                       B
Awc/B/.svn/entries this_dir depth-infinity
                                       ...
       Awc/B/E/.svn/entries this_dir depth-infinity
...
       ...

   svn up --depth=immediates Awc/D

       Since D is not yet checked out, add it at depth-immediates.

       Awc/.svn/entries this_dir depth-empty
                                       B
                                       D
       Awc/D/.svn/entries this_dir depth-immediates
                                       ...
       Awc/D/G/.svn/entries this_dir depth-empty
       ...

<===========END QUOTE================>

It shows that the --depth option will define WC-depth for those not exist in
the wc yet (aka omitted-items). I wrote a test case for it, which is
gracefully passed. I expect this behavior is consistent in other subcommands
and thus add a verify_depth(None, "files", new1_path) to check it in the
"add_tree_with_depth_files" test case. To my surprise, the depth of new
added
directory is still infinity. What do you think about this? Do I made a wrong
assumption or is it actually a bug?

Anyway, I suggest the description of --depth option in the commandline help
messages be improved to notify this behavior, at least for the checkout,
update and switch commands. The new description should looks like this:

  --depth ARG : limit operation by depth ARG ('empty', 'files',
                            'immediates', or 'infinity'), also defines wc
                            depth for the items not yet exist in wc

Also, the sparse-directories.txt should be updated accordingly.

Karl: I'm looking forward to see your comments about this. My previous post
seems to be buried in the sea of mails and got no reply. :(

Thank you
Rui, Guo

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-09 18:35:25 CEST

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