On Fri, Dec 18, 2020 at 12:50 AM <jcorvel_at_apache.org> wrote:
>
> Author: jcorvel
> Date: Thu Dec 17 23:50:27 2020
> New Revision: 1884579
>
> URL: http://svn.apache.org/viewvc?rev=1884579&view=rev
> Log:
> * 1.14.x/STATUS: Add -1 vote to r1880192 (issue #4862), because the test
> fails on Windows. Vote for the test fix (r1881985 group). If the test
> fix gets enough votes too, my vote for r1880192 can be converted to +1.
>
> Modified:
> subversion/branches/1.14.x/STATUS
>
> Modified: subversion/branches/1.14.x/STATUS
> URL: http://svn.apache.org/viewvc/subversion/branches/1.14.x/STATUS?rev=1884579&r1=1884578&r2=1884579&view=diff
> ==============================================================================
> --- subversion/branches/1.14.x/STATUS (original)
> +++ subversion/branches/1.14.x/STATUS Thu Dec 17 23:50:27 2020
> @@ -23,15 +23,17 @@ Candidate changes:
> r1878997, r1879192, r1879474, r1879959 (Fix issue #4859)
> Votes:
> +1: julianfoad, hartmannathan
> + -1: jcorvel (test FAIL on Windows;
> + +1 if r1881985 group (see below) is backported too)
>
> * r1881985, r1882105
> Follow up to r1880192: Fix an EOL issue in test on Windows.
> Justification:
> We should test correctly on Windows environment, too.
> Depends:
> - rr1880192
> + r1880192
> Votes:
> - +1: futatuki
> + +1: futatuki, jcorvel
>
> * r1875675
> Rewrite load_normalize_node_props to avoid literal trailing spaces.
>
Just as a heads up: I was ready to vote for an approve r1880192 (issue
#4862 -- Merge: the resulting mergeinfo is non-deterministic) for
1.14.x, but as Yasuhito also discovered: there is an EOL issue in the
test, which makes it fail on Windows. So it really needs to be
accompanied by the r1881985 group.
If anyone can add a third vote to the r1881985 group, they get the
other approval as a bonus :-).
(cc'ing Julian and Nathan personally, because they have voted already
for the first issue, so already know the context)
--
Johan
Received on 2020-12-18 01:00:54 CET