[cmake] Use builtin target for OpenSSL rather than build one ourselves
ClosedPublic
Actions

Authored by deadalnix on Sep 27 2018, 18:17.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING2ff18c6f90fd: [cmake] Use builtin target for OpenSSL rather than build one ourselves
rABC2ff18c6f90fd: [cmake] Use builtin target for OpenSSL rather than build one ourselves

Summary

As it turns out, cmake provides a target for it.

Test Plan

ninja

Diff Detail

Repository

rABC Bitcoin ABC

Branch

qtcmake

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3421
Build 4918: Bitcoin ABC Buildbot (legacy)
Build 4917: arc lint + arc unit

Event Timeline

deadalnix created this revision.Sep 27 2018, 18:17

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 27 2018, 18:17

jasonbcox accepted this revision.Sep 27 2018, 18:19

This revision is now accepted and ready to land.Sep 27 2018, 18:19

Closed by commit rABC2ff18c6f90fd: [cmake] Use builtin target for OpenSSL rather than build one ourselves (authored by deadalnix). · Explain WhySep 27 2018, 18:20

This revision was automatically updated to reflect the committed changes.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B3421: Diff 5138.Sep 27 2018, 18:42

Revision Contents
Changeset List

Path

Size

src/

CMakeLists.txt

5 lines

crypto/

CMakeLists.txt

2 lines

qt/

CMakeLists.txt

2 lines

Diff 5138

View Options

src/CMakeLists.txt

View Options

src/crypto/CMakeLists.txt

View Options

[cmake] Use builtin target for OpenSSL rather than build one ourselvesClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 5138

src/CMakeLists.txt

src/crypto/CMakeLists.txt

src/qt/CMakeLists.txt

[cmake] Use builtin target for OpenSSL rather than build one ourselves
ClosedPublic
Actions

Revision Contents
Changeset List