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

Re: [PATCH] issue 2154 - svn blame on a directory over DAV does not return gracefully

From: VK Sameer <sameer_at_collab.net>
Date: 2005-03-01 10:43:27 CET

Attached is a patch based on Justin's comment.

'make davcheck' gave some errors in different test cases, merge_tests.py
once, but they didn't recur when run separately, leading me to think it
may have been due to my httpd performance configuration. 'make check'
passed.

Regards
Sameer

Change report functions to check for interim errors before calling
ap_fflush(). This avoids creating the HTTP packet header with the
default (and currently invalid) HTTP 200 status.

* subversion/mod_dav_svn/file_revs.c
  (svn_repos_get_file_revs):
     Call ap_fflush only if there is no existing error

* subversion/mod_dav_svn/log.c
  (dav_svn__log_report): Ditto

* subversion/mod_dav_svn/version.c
  (dav_svn__drev_report,
   dav_svn__get_locations_report): Ditto

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Received on Tue Mar 1 10:44:41 2005

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.