On 8/28/2016 13:31, Stefan wrote:
> Hi,
>
> attached is a new version of the patch which adds a regression test (the
> actual patch/fix was left unchanged).
> The regression test was tested against 1.9.4, 1.9.x and trunk r1743999.
>
> I also tried to run the test against 1.8.16 but there it fails (didn't
> investigate in detail).
> Trunk r1758069 caused some build issues on my machine. Therefore I
> couldn't validate/check the patch against the latest trunk (maybe it's
> just some local issue with my build machine rather than some actual
> problem on trunk - didn't look into that yet).
>
> [[[
> Fix issue #4647 by resolving the error case for binary file conflicts
> selecting
> to use the full local version, by allowing using the local file, if no
> merged
> file is present.
>
> * subversion/libsvn_wc/conflicts.c
> (build_text_conflict_resolve_items): in case mine_abspath is null, take
> local_abspath instead in case of
> svn_wc_conflict_choose_mine_full
>
> * subversion/tests/cmdline/resolve_tests.py
> (automatic_binary_conflict_resolution): add new regression test
>
> Suggested by: stsp
> ]]]
>
> Regards,
> Stefan
Got approved by Bert.
Separated the repro test from the actual fix in order to have the
possibility to selectively only backport the regression test to the 1.8
branch.
Committed to trunk in r1758128-30 and nominated for backport to 1.9 in
r1758131.
Regards,
Stefan
Received on 2016-08-28 18:34:12 CEST