diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2019-10-10 13:18:54 -0400 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2019-10-10 13:19:10 -0400 |
| commit | d5a770b70d55a5874fbd7796a171de22abbbd342 (patch) | |
| tree | 5b425a7be1d3284445e42f7f30814884c1daa7c0 /src | |
| parent | befdef8aee899dcf7e40aa5ea4bc1b0256381cdc (diff) | |
| parent | d478a472eb0d666e8a762ed8d24fafbabc5f94f3 (diff) | |
Merge #16973: test: Fix combine_logs.py for AppVeyor build
d478a472eb0d666e8a762ed8d24fafbabc5f94f3 test: Fix combine_logs.py for AppVeyor build (Martin Zumsande)
Pull request description:
Fixes #16894
This fixes the problem of AppVeyor builds not showing `debug.log` if a functional test fails, because the windows separator `\` doesn't work together with the regex in `combine_logs.py`.
A fix was already attempted in #16896, however, that PR became inactive and was marked "up for grabs", plus it's a really small change.
As suggested by jamesob, this PR uses `pathlib`: For the glob and to convert the path to a posix-style string, it leaves the regex as is (in contrast to #16896 which adjusted the regex).
I tested this locally on Windows and Ubuntu.
Top commit has no ACKs.
Tree-SHA512: 603b4359b6009b6da874c30f69759acda03730ee5747898a0fe957a5fc37ee9ba07858c6aa2169bf4c40521f37e47138e8314d698652ea2760fa0a3f76b890bd
Diffstat (limited to 'src')
0 files changed, 0 insertions, 0 deletions