I've created a temporary branch, "ood-status-info", so we have a place
to work on refining these fixes.
On Fri, 13 Oct 2006, Paul Burba wrote:
> Daniel Rall <firstname.lastname@example.org> wrote on 10/12/2006 05:37:45 PM:
> > [Focusing on the Java/test portion of this thread.]
> > On Wed, 11 Oct 2006, Paul Burba wrote:
> > > Daniel Rall <email@example.com> wrote on 10/06/2006 03:49:32 PM:
> > and have committed this code in an XFAIL-style in r21908.
> Though we'd need to detect XPASS no?
Yeah, we want to report an unexpected pass. I've done so on trunk in
> > Some of the Python status tests do check for "*", but don't test the
> > individual pieces of "out of date" information retrieved from
> > repository.
> You're making this statement for the benefit of others reading this yes?
> Or is there a question for me in here? :-)
> Our remaining issues(?):
> A) The performance of svn_repos_deleted_rev()
This is currently too slow to integrate into trunk. Let's write a
faster implementation -- hopefully we can speed things up by moving
the logic down inside the FS layer.
> B) Possible side effects of passing wrong revision to
> svn_delta_editor_t delete_entry() implementations.
Any findings here?
I've committed the following patch to the "ood-status-info" branch in
> Further improvements to status information on working copy items which
> are out of date compared to the repository.
> Follow-up to r16344 (and its subsequent follow-ups: r16494, 16784, 16796,
> 16829, 17844, and 21908).
> * subversion/bindings/java/javahl/src/org/tigris/
> (testOODStatus): Remove XFAIL-style catch.
> * subversion/include/svn_repos.h
> (svn_repos_deleted_rev): New function to find the revision a path was
> most recently deleted within a give revision range.
> * subversion/libsvn_repos/reporter.c
> (update_entry, delta_dirs): Use the new function svn_repos_deleted_rev()
> to determine the revision deleted paths were deleted and pass this to
> the delete_entry callback rather than defaulting to SVN_INVALID_REVNUM.
> * subversion/libsvn_repos/rev_hunt.c
> (svn_repos_deleted_rev): New function definition.
> * subversion/libsvn_wc/status.c
> (tweak_statushash): Add second baton argument which contains the out
> of date info for a dir baton when tweaking that baton's parent. Add
> another argument to identify the revision a path was deleted when
> handling deletes. When deleting paths: Construct the correct url for
> the path and record deleted path's deleted revision in the path's
> svn_wc_status2_t structure. For pre-1.5 servers, which don't provide
> the deleted revision, use the parent's last committed rev as a best
> (delete_entry, close_file): Supply new args to tweak_statushash()
> (close_directory): Tweak status for directories even when the only
> change is that they have and an out of date descendents. Supply new
> args to tweak_statushash() call.
Received on Wed Oct 25 20:56:31 2006
- application/pgp-signature attachment: stored