Page MenuHomePhabricator
Differential D6242

random: Remove remaining OpenSSL calls and locking infrastructure
ClosedPublic
Actions

Authored by Fabien on May 25 2020, 07:49.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCea9e88b8c6fb: random: Remove remaining OpenSSL calls and locking infrastructure
Summary

Partial backport of core PR17265:
https://github.com/bitcoin/bitcoin/pull/17265/commits/b49b6b0f7090cc15860d815fb0ef306ddfc718ba

Depends on D6240 and D6241.

Test Plan
ninja all check-all

Run the Gitian builds.

Diff Detail

Event Timeline

Fabien created this revision.May 25 2020, 07:49
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 25 2020, 07:49
Fabien requested review of this revision.May 25 2020, 07:49
teamcity edited the summary of this revision. (Show Details)May 25 2020, 07:50

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

teamcity added a comment.May 25 2020, 07:50

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

teamcity added a comment.May 25 2020, 07:50

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

teamcity added a comment.May 25 2020, 07:50

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

teamcity added a comment.May 25 2020, 07:51

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

teamcity added a comment.May 25 2020, 07:52

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

teamcity added a comment.May 25 2020, 08:02

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

Harbormaster completed remote builds in B11009: Diff 20581.May 25 2020, 08:02
Fabien added a child revision: D6243: [BUILD] Search and include OpenSSL only where required.May 25 2020, 12:34
deadalnix accepted this revision.May 25 2020, 13:20
This revision is now accepted and ready to land.May 25 2020, 13:20
Closed by commit rABCea9e88b8c6fb: random: Remove remaining OpenSSL calls and locking infrastructure (authored by fanquake <fanquake@gmail.com>, committed by Fabien). · Explain WhyMay 25 2020, 14:45
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCea9e88b8c6fb: random: Remove remaining OpenSSL calls and locking infrastructure.
Fabien mentioned this in D6243: [BUILD] Search and include OpenSSL only where required.May 25 2020, 19:22
Fabien removed a child revision: D6243: [BUILD] Search and include OpenSSL only where required.
Fabien mentioned this in rABC617ddd5be3c0: [BUILD] Search and include OpenSSL only where required.May 26 2020, 13:29

Revision Contents
Changeset List

PathSize
src/
44 lines

Diff 20581

View Options

src/random.cpp

Loading...