Page MenuHomePhabricator

Partial build fix for WSL, disabling seeder

Authored by vermorel on Jul 26 2018, 12:31.


Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
rSTAGING96b53e289d03: Partial build fix for WSL, disabling seeder
rABC96b53e289d03: Partial build fix for WSL, disabling seeder

On the Windows Subsystem for Linux (WSL), the compilation
fails because of bitcoin_seeder. As this app is not a
must-have, the build instructions are updated with the
proper hint.

Still on WSL, a fix from Bitcoin Core is backported in order
to get the tests to compile. See:


Test Plan

Compiles and green tests on Win10 x64 under Ubuntu subsystem.

Diff Detail

rABC Bitcoin ABC
Lint OK
No Unit Test Coverage
Build Status
Buildable 2977
Build 4052: Bitcoin ABC Teamcity Staging
Build 4051: arc lint + arc unit

Event Timeline

vermorel created this revision.Jul 26 2018, 12:31
Owners added a reviewer: Restricted Owners Package.Jul 26 2018, 12:31
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 26 2018, 12:31
vermorel updated this revision to Diff 4436.Jul 26 2018, 12:39

Fix lint style.

deadalnix requested changes to this revision.Jul 26 2018, 17:11
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
117 ↗(On Diff #4436)

Also you added the prefix twice.

145 ↗(On Diff #4436)


69 ↗(On Diff #4436)

I think it'd be better to add a definition of setenv somewhere in /compat for win32, that way most of the code do not need to ifdef. setenv is also used in the seeder, and, while it isn't enough to get it to work on windows, that'd be a set in the right direction.

This revision now requires changes to proceed.Jul 26 2018, 17:11
vermorel updated this revision to Diff 4439.Jul 26 2018, 17:17

Fixed incorrect argument duplication.

jasonbcox accepted this revision.Jul 26 2018, 19:39
vermorel updated this revision to Diff 4444.Jul 26 2018, 20:03

Isolate compatibility fix for 'setenv()'.

deadalnix accepted this revision.Jul 26 2018, 20:07
deadalnix added inline comments.
6 ↗(On Diff #4444)
#include <cstdlib>
9 ↗(On Diff #4444)

This is not the exact behavior, but it'll do for now.

This revision is now accepted and ready to land.Jul 26 2018, 20:07
vermorel updated this revision to Diff 4446.Jul 26 2018, 20:12

Using suitable include.

This revision was automatically updated to reflect the committed changes.