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

Re: [PATCH][v2] Sticky depths should work for an excluded dir

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Tue, 05 Jan 2010 13:37:55 +0000

Kannan <kannanr_at_collab.net> writes:

> Make `--set-depth (empty|files|immediates) DIR' work, where DIR's depth
> had already been set as `exclude'. Only `infinity' works as of now.
>
> * subversion/libsvn_wc/crop.c
> (svn_wc_crop_tree2): Check the 'base status' of the node here and
> remove the check done using `svn_wc__db_node_hidden()' as it
> marks a node as 'hidden' if status is set as
> `svn_wc__db_status_excluded'.
>
> * update_editor.c
> (complete_directory): Call `svn_wc__set_depth' with the requested
> depth rather than `infinity' by default.
>
> * entries.c
> (svn_wc__set_depth): Set the entry's depth to the requested one rather
> than `infinity' by default if the depth is not `exclude'.
>
> * wc_db.c
> (svn_wc__db_temp_op_dir_set_depth): Remove the code which sets the
> depth as `infinity' if the depth is not `exclude'.
>
> Patch by: Kannan R <kannanr_at_collab.net>

It looks plausible, but it would be much better if you included a
regression test that exercised the new code. If you fix something
like this without adding a test there is no guarantee that some future
change won't break it.

-- 
Philip
Received on 2010-01-05 14:38:34 CET

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