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

[PATCH] Make svn clients indicate their operation name to backend(right now only to DAV) Version 2

From: Kamesh Jayachandran <kamesh_at_collab.net>
Date: Tue, 12 Jan 2010 18:49:10 +0530

Hi All,

Last week I posted the patch to implement 'svn client' to identify the
svn operation they are about to do with a given ra_session.

Following thread has the detailed discussion.

http://mail-archives.apache.org/mod_mbox/subversion-dev/201001.mbox/%3C4B448959.1070400@collab.net%3E

Below is the summary:

Concern/Suggestion 1:
Michael Pilato and Philip Martin were suggesting to tweak
libsvn_ra_(serf|neon) to detect the operation rather than making a
change across all layers from the simplicity point of view.

My answer to 1:
I feel it would be too hackish to tweak one general API inside these
modules to flag 'commit or write' operation to the server when the
solution I propose handles this in a transparent way.

Concern/Suggestion 2:
There was a request to change the 'high_level_svn_operation' name at a
later stage after opening the ra_session.

My answer to 2:
I have introduced a new API 'svn_ra_change_operation_name' to handle this.

Concern/Suggestion 3:
There was a request to change the header name from SVN-ACTION to
something else as it will be a confusing with svn operational log's
SVN-ACTION.

My answer to 3:
I have changed that request header to SVN-OPERATION.

I am attaching my new patch if there are no objections will commit it in
Jan 14 my time.

With regards
Kamesh Jayachandran

  • text/plain attachment: stored
Received on 2010-01-12 14:21:06 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.