Page MenuHomePhabricator

utils: Convert Windows args to utf-8 string

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



This is a backport of Core PR13883

Test Plan
make check

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
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.