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

Re: svn commit: rev 3980 - in trunk/subversion: libsvn_client tests/clients/cmdline libsvn_ra_svn svnserve

From: <cmpilato_at_collab.net>
Date: 2002-12-04 18:57:58 CET

ghudson@tigris.org writes:

> * libsvn_client/commit.c
> (get_ra_editor, svn_client_import): If it looks like we'll need the
> head revision, get it before getting the commit editor. The
> previous order wasn't explicitly prohibited by the ra interface, but
> it doesn't work with ra_svn (once you get a commit editor the
> protocol has gone into editor-land and you can't make other requests
> until you're done editing), and this way is arguably more elegant.
> It's also the only case of this mis-ordering identified by the test
> suite.
> We accomplish this by adding an optional latest_rev argument to
> get_ra_editor; if it is set, get_ra_editor fills it in with the
> result of get-latest-rev.

This, unfortunately, causes an extra network round-trip for what is
probably the least-usual commit situation -- one that involves
changing dir properties. I'll fix this for you real quick-like.

C. Michael Pilato <cmpilato@collab.net>
CollabNet, Inc. | 312-922-2818 p | 312-922-2603 f
407 S Dearborn | Suite 1075 | Chicago, IL | 60605
Are you developing?
Attend a FREE Web seminar on January 22 - SourceCast Guided Tour.
Register now: http://collabnetevents.webex.com/
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Dec 4 19:02:19 2002

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