Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: Avoid qt-specific @GLIBC_2.28 symbols for compatibility #22321

Closed
wants to merge 2 commits into from

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Jun 23, 2021

As noted in #22244, new symbols were introduced in bitcoin-qt during the current release cycle, particularly:

  • statx@GLIBC_2.28
  • renameat2@GLIBC_2.28

This PR gets rid of them.

Removing of statx is pretty hacky because it is integrated into Qt code tightly, and just passing -no-feature-statx don't work.

See:

Closes #22280.


Note for reviewers. While testing please post kernel and glib versions of systems you run tested binaries on.

It would be nice to test systems with pretty old ones (e.g., pre-4.11 kernel).

@DrahtBot
Copy link
Contributor

DrahtBot commented Jun 23, 2021

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@hebasto
Copy link
Member Author

hebasto commented Jul 1, 2021

Closing in favor of #22365.

@hebasto hebasto closed this Jul 1, 2021
@hebasto hebasto mentioned this pull request Jul 8, 2021
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

build: bitcoin-qt built against Qt 5.12 requires glibc 2.28+
2 participants