Author: svn-role
Date: Wed Dec  4 04:00:30 2019
New Revision: 1870786

URL: http://svn.apache.org/viewvc?rev=1870786&view=rev
Log:
Merge the r1870393 group from trunk:

 * r1870393, r1870395
   Fix issue #4837 'Cannot get info for a file that was
   inside of file-replaced directory'.
   Justification:
     Regression introduced in 1.10.0.
   Votes:
     +1: julianfoad, rhuijben, jamessan

Modified:
    subversion/branches/1.13.x/   (props changed)
    subversion/branches/1.13.x/STATUS
    subversion/branches/1.13.x/subversion/libsvn_client/info.c
    subversion/branches/1.13.x/subversion/tests/cmdline/info_tests.py

Propchange: subversion/branches/1.13.x/
------------------------------------------------------------------------------
  Merged /subversion/trunk:r1870393,1870395

Modified: subversion/branches/1.13.x/STATUS
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.13.x/STATUS?rev=1870786&r1=1870785&r2=1870786&view=diff
==============================================================================
--- subversion/branches/1.13.x/STATUS (original)
+++ subversion/branches/1.13.x/STATUS Wed Dec  4 04:00:30 2019
@@ -20,12 +20,3 @@ Veto-blocked changes:
 
 Approved changes:
 =================
-
- * r1870393, r1870395
-   Fix issue #4837 'Cannot get info for a file that was
-   inside of file-replaced directory'.
-   Justification:
-     Regression introduced in 1.10.0.
-   Votes:
-     +1: julianfoad, rhuijben, jamessan
-

Modified: subversion/branches/1.13.x/subversion/libsvn_client/info.c
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.13.x/subversion/libsvn_client/info.c?rev=1870786&r1=1870785&r2=1870786&view=diff
==============================================================================
--- subversion/branches/1.13.x/subversion/libsvn_client/info.c (original)
+++ subversion/branches/1.13.x/subversion/libsvn_client/info.c Wed Dec  4 
04:00:30 2019
@@ -268,6 +268,7 @@ same_resource_in_head(svn_boolean_t *sam
                                     ctx, pool);
   if (err &&
       ((err->apr_err == SVN_ERR_CLIENT_UNRELATED_RESOURCES) ||
+       (err->apr_err == SVN_ERR_FS_NOT_DIRECTORY) ||
        (err->apr_err == SVN_ERR_FS_NOT_FOUND)))
     {
       svn_error_clear(err);

Modified: subversion/branches/1.13.x/subversion/tests/cmdline/info_tests.py
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.13.x/subversion/tests/cmdline/info_tests.py?rev=1870786&r1=1870785&r2=1870786&view=diff
==============================================================================
--- subversion/branches/1.13.x/subversion/tests/cmdline/info_tests.py (original)
+++ subversion/branches/1.13.x/subversion/tests/cmdline/info_tests.py Wed Dec  
4 04:00:30 2019
@@ -818,6 +818,32 @@ def info_item_failures(sbox):
     sbox.ospath('iota'))
 
 
+@Issue(4837)
+def info_file_in_file_replaced_dir(sbox):
+  "info, file in file-replaced dir"
+
+  sbox.build(empty=True)
+  sbox.simple_mkdir('dir')
+  sbox.simple_add_text('text\n', 'dir/file')
+  sbox.simple_commit(message='Add file')
+
+  sbox.simple_copy('dir/file', 'file-moved')
+  sbox.simple_rm('dir')
+  sbox.simple_add_text('replaced\n', 'dir')
+  sbox.simple_commit(message='Replace dir with file')
+
+  sbox.simple_update()
+
+  expected = {'Relative URL' : r'\^/dir/file',
+              'Node Kind' : 'file',
+              'Revision': '1',
+              'Last Changed Rev': '1',
+             }
+
+  svntest.actions.run_and_verify_info([expected],
+                                      sbox.repo_url + '/dir/file@1')
+
+
 ########################################################################
 # Run the tests
 
@@ -842,6 +868,7 @@ test_list = [ None,
               info_item_size_repos,
               info_item_size_repos_recursive,
               info_item_failures,
+              info_file_in_file_replaced_dir,
              ]
 
 if __name__ == '__main__':


Reply via email to