[SVN-3937] 'svn log' of nonexistent path pegged at existing rev succeeds over ra_serf

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: trunk
• Fix Version/s: 1.7.0
• Component/s: unknown
• Assignee: Unassigned
• Reporter: Paul Burba
• Created: 23/Jun/11 19:07
• Updated: 27/Oct/18 16:28
• Resolved: 03/May/12 07:01

Description

Running 'svn log' on a nonexistent path pegged at an existing revision raises an
error over every RA provider except serf.

For example, given our basic greek tree:

  >svn log -q ^^/
  ------------------------------------------------------------------------
  r1 | jrandom | 2011-06-23 14:45:46 -0400 (Thu, 23 Jun 2011)
  ------------------------------------------------------------------------

Log requests for paths, existent or otherwise, pegged at a nonexistent rev fail:

  >svn log ^^/A@2
  ..\..\..\subversion\svn\log-cmd.c:758: (apr_err=160006)
  ..\..\..\subversion\libsvn_client\log.c:622: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\log.c:672: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:681: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:1789: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:1789: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:827: (apr_err=160006)
  svn: E160006: No such revision 2

  >svn log ^^/Z@2
  ..\..\..\subversion\svn\log-cmd.c:758: (apr_err=160006)
  ..\..\..\subversion\libsvn_client\log.c:622: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\log.c:672: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:681: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:1789: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:1789: (apr_err=160006)
  ..\..\..\subversion\libsvn_ra_serf\util.c:827: (apr_err=160006)
  svn: E160006: No such revision 2

Nonexistent paths pegged at an existing revision fail over the other 3 RA
providers result in an error, but over ra_serf the result is an empty log:

ra_local:

  >svn log ^^/Z@1
  ..\..\..\subversion\svn\log-cmd.c:758: (apr_err=160013)
  ..\..\..\subversion\libsvn_client\log.c:622: (apr_err=160013)
  ..\..\..\subversion\libsvn_repos\log.c:1699: (apr_err=160013)
  ..\..\..\subversion\libsvn_repos\log.c:1254: (apr_err=160013)
  ..\..\..\subversion\libsvn_fs_fs\tree.c:2790: (apr_err=160013)
  ..\..\..\subversion\libsvn_fs_fs\tree.c:2790: (apr_err=160013)
  svn: E160013: File not found: revision 1, path '/Z'

ra_neon:

  >svn log ^^/Z@1
  ..\..\..\subversion\svn\log-cmd.c:758: (apr_err=160013)
  ..\..\..\subversion\libsvn_client\log.c:622: (apr_err=160013)
  ..\..\..\subversion\libsvn_ra_neon\util.c:1323: (apr_err=160013)
  ..\..\..\subversion\libsvn_ra_neon\util.c:596: (apr_err=160013)
  svn: E160013: '/svn-test-work/repositories/log_tests-34/!svn/rvr/1/Z' path not
found

ra_svn:

  >svn log ^^/Z@1
  ..\..\..\subversion\svn\log-cmd.c:758: (apr_err=160013)
  ..\..\..\subversion\libsvn_client\log.c:622: (apr_err=160013)
  ..\..\..\subversion\libsvn_repos\log.c:1699: (apr_err=160013)
  svn: E160013: File not found: revision 1, path '/Z'
  ..\..\..\subversion\libsvn_repos\log.c:1254: (apr_err=160013)
  svn: E160013: File not found: revision 1, path '/Z'
  ..\..\..\subversion\libsvn_fs_fs\tree.c:2790: (apr_err=160013)
  svn: E160013: File not found: revision 1, path '/Z'
  ..\..\..\subversion\libsvn_fs_fs\tree.c:2790: (apr_err=160013)
  svn: E160013: File not found: revision 1, path '/Z'


ra_serf:

  >svn log ^^/Z@1
  ------------------------------------------------------------------------

  >

Issue Links

• blocks: SVN-3936 ra_serf returns a spurious error from svn_ra_get_locations()
• is depended upon by: SVN-3936 ra_serf returns a spurious error from svn_ra_get_locations()