Skip to content

Conversation

@fanquake
Copy link
Member

@fanquake fanquake commented Jun 9, 2022

There might not be another 0.21.x release, however these are both straight forward changes. If this isn't merged, then the pulls can remain untagged for needing backport.

Backports:

joankaradimov and others added 2 commits June 9, 2022 12:53
... needed for std::numeric_limits<T>::max on WIN32

Github-Pull: bitcoin#23335
Rebased-From: 077a875
`crc32c`'s hardware accelerated code doesn't handle ARM 32-bit at all.
Make the check in `configure.ac` check for this architecture explicitly.

For the release binaries, the current `configure.ac` check happens
to work: it enables it on aarch64 but disables it for armhf. However
some combination of compiler version and settings might ostensibly cause
this check to succeed on armhf (as reported on IRC). So make the 64-bit
platform requirement explicit.

Github-Pull: bitcoin#23045
Rebased-From: f2747d1
@luke-jr
Copy link
Member

luke-jr commented Jun 9, 2022

nit: Shouldn't the milestone here be 0.21.3? We already released 0.21.2...

@fanquake fanquake removed this from the 0.21.2 milestone Jun 9, 2022
@fanquake
Copy link
Member Author

fanquake commented Jun 9, 2022

I'll just leave it unmilestoned. Have closed 0.21.2 given it wasn't already.

@laanwj
Copy link
Member

laanwj commented Jun 9, 2022

ACK efb9f00
Seems fine to backport these.

Copy link
Contributor

@LarryRuane LarryRuane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK efb9f00
I verified that these diffs match the source PRs' diffs.

@fanquake fanquake merged commit dca463b into bitcoin:0.21 Jun 10, 2022
@fanquake fanquake deleted the 0_21_backports branch June 10, 2022 10:59
@bitcoin bitcoin locked and limited conversation to collaborators Jun 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants