Page MenuHomePhabricator

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

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



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:

../contrib/devtools/ 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

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


deadalnix requested changes to this revision.Sun, May 5, 16:48
deadalnix added inline comments.
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


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.