Page MenuHomePhabricator

Merge #12917: qa: Windows fixups for functional tests
ClosedPublic

Authored by markblundeberg on Feb 6 2020, 02:06.

Details

Summary

PR12917 backport.

fab9095d40 qa: Windows fixups for functional tests (MarcoFalke)

Pull request description:

Just two minor fixups to have less errors when the tests run on native windows.
* Strip whitespace from lines when reading from a notification file
* Instead of clumsily creating a file with weird permissions, just create a folder for the same effect in `mempool_persist.py`
Test Plan

ninja check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those Bitcoin Core PRs have been inserted into the summary for reference.

This revision is now accepted and ready to land.Feb 6 2020, 02:11

This is rerunning on the highperf3 machine...