Page MenuHomePhabricator

[CMAKE] Add an option to enable the glibc compatibility features
ClosedPublic

Authored by Fabien on Tue, Apr 30, 09:57.

Details

Summary

The option ENABLE_GLIBC_BACK_COMPAT can be set to ON to configure
and include the functions from the compat/glibc_compat.cpp file. This
is a port of the configure option --enable-glibc-back-compat.

Test Plan

On a recent Linux with a glibc version > 2.17:

cmake -DENABLE_GLIBC_BACK_COMPAT=ON -GNinja ..
ninja
../contrib/devtools/symbol-check.py src/bitcoind | grep log2f

Before the patch, should return:

src/bitcoind: symbol log2f from unsupported version <glibc_version>

After the patch, should return nothing.

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

Fabien created this revision.Tue, Apr 30, 09:57
Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Apr 30, 09:57
Fabien updated this revision to Diff 8307.Tue, Apr 30, 09:59

Rebase

deadalnix requested changes to this revision.Sun, May 5, 16:48
deadalnix added inline comments.
src/CMakeLists.txt
162 ↗(On Diff #8307)

You should check for rt's presence. This is a miracle that it works at all, you probably have something in your build that requires rt transitively ?

This revision now requires changes to proceed.Sun, May 5, 16:48
Fabien updated this revision to Diff 8408.Mon, May 6, 08:03

Check for rt lib

Fabien updated this revision to Diff 8591.Fri, May 10, 10:09

Rebase

deadalnix accepted this revision.Thu, May 16, 23:07
This revision is now accepted and ready to land.Thu, May 16, 23:07
This revision was automatically updated to reflect the committed changes.