Page MenuHomePhabricator

utils: Convert Windows args to utf-8 string
ClosedPublic

Authored by deadalnix on Fri, Sep 27, 20:34.

Details

Summary

This is a backport of Core PR13883

Test Plan
make check
./test/functional/test_runner.py

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

deadalnix created this revision.Fri, Sep 27, 20:34
Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, Sep 27, 20:34
deadalnix edited the test plan for this revision. (Show Details)Fri, Sep 27, 20:34
jasonbcox requested changes to this revision.Fri, Sep 27, 21:48
jasonbcox added a subscriber: jasonbcox.

The backport looks ok comparing to the original PR, but the test is broken.

This revision now requires changes to proceed.Fri, Sep 27, 21:48
deadalnix updated this revision to Diff 13266.Sat, Sep 28, 13:46

Try something

deadalnix updated this revision to Diff 13267.Sat, Sep 28, 14:17

try more shit

I have no idea what's happening on CI, none of this is reproducing outside of CI.

Fabien added a subscriber: Fabien.Mon, Sep 30, 17:08

I tried to reproduce on a fresh Debian Stretch, no success.
I will try to install the agent packages and run again.

jasonbcox accepted this revision.Tue, Oct 1, 20:26
This revision is now accepted and ready to land.Tue, Oct 1, 20:26
This revision was automatically updated to reflect the committed changes.