Page MenuHomePhabricator

Remove check for thread_local in build systems
ClosedPublic

Authored by deadalnix on Dec 22 2018, 18:57.

Details

Summary

It is part of C++11, so has become useless to check for.

Test Plan
make check
ninja check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
threadlocaltest
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4358
Build 6781: Bitcoin ABC Buildbot (legacy)
Build 6780: arc lint + arc unit

Event Timeline

LGTM. Tested with make/cmake and gcc/clang.
Can you check why the build has failed ?

jasonbcox added a subscriber: jasonbcox.

I reran the build for you. Looks like it was a spurious failure with the build bot, not an issue with the codebase.

This revision is now accepted and ready to land.Dec 29 2018, 01:10
This revision was automatically updated to reflect the committed changes.