Daniel Rall <email@example.com> wrote on 10/25/2006 02:55:09 PM:
> 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
> > Or is there a question for me in here? :-)
'Yup' to which question? "Would you like coffee or tea?", "YES!" :-)
> > 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.
I'll post a separate message to catch the attention of FS gurus and get
> > B) Possible side effects of passing wrong revision to
> > svn_delta_editor_t delete_entry() implementations.
> Any findings here?
Short answer is no. I gave the long answer earlier in this thread:
If my answer there isn't satisfactory let me know.
> 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,
> > 16829, 17844, and 21908).
> > * subversion/bindings/java/javahl/src/org/tigris/
> > subversion/javahl/tests/BasicTests.java
> > (testOODStatus): Remove XFAIL-style catch.
> > * subversion/include/svn_repos.h
> > (svn_repos_deleted_rev): New function to find the revision a path
> > most recently deleted within a give revision range.
> > * subversion/libsvn_repos/reporter.c
> > (update_entry, delta_dirs): Use the new function
> > to determine the revision deleted paths were deleted and pass this
> > the delete_entry callback rather than defaulting to
> > * 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
> > 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
> > the deleted revision, use the parent's last committed rev as a best
> > guess.
> > (delete_entry, close_file): Supply new args to tweak_statushash()
> > calls.
> > (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.
> > ]]]
To unsubscribe, e-mail: firstname.lastname@example.org
For additional commands, e-mail: email@example.com
Received on Wed Oct 25 21:56:25 2006