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

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

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

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

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

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

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

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

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

Path

Size

src/

random.cpp

44 lines

Diff 20586

View Options

random: Remove remaining OpenSSL calls and locking infrastructureClosedPublicActions