You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by sv...@apache.org on 2021/01/28 04:00:25 UTC
svn commit: r1885965 - in /subversion/branches/1.10.x: ./ STATUS
subversion/tests/cmdline/merge_tests.py
Author: svn-role
Date: Thu Jan 28 04:00:24 2021
New Revision: 1885965
URL: http://svn.apache.org/viewvc?rev=1885965&view=rev
Log:
Merge the r1881985 group from trunk:
* r1881985, r1882105
Follow up to r1880192: Fix an EOL issue in test on Windows.
Justification:
We should test correctly on Windows environment, too.
Depends:
r1880192
Votes:
+1: jcorvel, stsp, hartmannathan
Modified:
subversion/branches/1.10.x/ (props changed)
subversion/branches/1.10.x/STATUS
subversion/branches/1.10.x/subversion/tests/cmdline/merge_tests.py
Propchange: subversion/branches/1.10.x/
------------------------------------------------------------------------------
Merged /subversion/trunk:r1881985,1882105
Modified: subversion/branches/1.10.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.10.x/STATUS?rev=1885965&r1=1885964&r2=1885965&view=diff
==============================================================================
--- subversion/branches/1.10.x/STATUS (original)
+++ subversion/branches/1.10.x/STATUS Thu Jan 28 04:00:24 2021
@@ -48,12 +48,3 @@ Veto-blocked changes:
Approved changes:
=================
-
- * r1881985, r1882105
- Follow up to r1880192: Fix an EOL issue in test on Windows.
- Justification:
- We should test correctly on Windows environment, too.
- Depends:
- r1880192
- Votes:
- +1: jcorvel, stsp, hartmannathan
Modified: subversion/branches/1.10.x/subversion/tests/cmdline/merge_tests.py
URL: http://svn.apache.org/viewvc/subversion/branches/1.10.x/subversion/tests/cmdline/merge_tests.py?rev=1885965&r1=1885964&r2=1885965&view=diff
==============================================================================
--- subversion/branches/1.10.x/subversion/tests/cmdline/merge_tests.py (original)
+++ subversion/branches/1.10.x/subversion/tests/cmdline/merge_tests.py Thu Jan 28 04:00:24 2021
@@ -18669,11 +18669,18 @@ def merge_deleted_folder_with_mergeinfo_
)
# verify that mergeinfo is set/changed on A/D, A/D/G, A/D/G2.
+
+ # NOTE: When writing out multi-line prop values in svn:* props, the
+ # client converts to local encoding and local eol style.
+ # Therefore, the expected output must contain the right kind of eoln
+ # strings. That's why we use os.linesep in the tests below, not just
+ # plain '\n'.
+
expected_mergeinfo = [
('A', ['/branch_A:3-7']),
- ('A/D', ['/branch_A/D:5-7\n', '/branch_B/C:1*']),
- ('A/D/G', ['/branch_A/D/G:5-7\n', '/branch_B/C/G:1*']),
- ('A/D/G2', ['/branch_A/D/G2:5-7\n', '/branch_B/C/G2:1*']),
+ ('A/D', ['/branch_A/D:5-7'+os.linesep, '/branch_B/C:1*']),
+ ('A/D/G', ['/branch_A/D/G:5-7'+os.linesep, '/branch_B/C/G:1*']),
+ ('A/D/G2', ['/branch_A/D/G2:5-7'+os.linesep, '/branch_B/C/G2:1*']),
]
for path, mergeinfo in expected_mergeinfo:
svntest.actions.check_prop('svn:mergeinfo', sbox.ospath(path),