Merge #12447: test: Add missing signal.h header
ClosedPublic
Actions

Authored by jasonbcox on Aug 9 2019, 21:18.

Details

Reviewers

deadalnix
Fabien

Group Reviewers

Restricted Project

Commits

rSTAGINGf92090407be0: Merge #12447: test: Add missing signal.h header
rABCf92090407be0: Merge #12447: test: Add missing signal.h header

Summary

dd7e42cbb4 test: Add missing signal.h header (Wladimir J. van der Laan)

Pull request description:

util_tests.cpp needs to include the signal.h header on FreeBSD.

Reported by denis2342 on IRC.

Tree-SHA512: 0b946fc2770ca6043087f4fe22be96848d46efe44e3d4d45c5ee663e0f95b0e2e6145f4ade577a6257d0c0c4cfadf324111799e78b9e4e4be4da8a90f68e349f

Backport of Core PR12447
https://github.com/bitcoin/bitcoin/pull/12447/files

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jasonbcox created this revision.Aug 9 2019, 21:18

Herald added a reviewer: deadalnix. · View Herald TranscriptAug 9 2019, 21:18

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B7104: Diff 10708.Aug 9 2019, 21:24

FreeBSD tests build is OK without this, it should be included by some other dependency.

This revision is now accepted and ready to land.Aug 12 2019, 07:17

Closed by commit rABCf92090407be0: Merge #12447: test: Add missing signal.h header (authored by MarcoFalke <falke.marco@gmail.com>, committed by jasonbcox). · Explain WhyAug 13 2019, 16:34

This revision was automatically updated to reflect the committed changes.

jasonbcox added a commit: rABCf92090407be0: Merge #12447: test: Add missing signal.h header.

jasonbcox added a commit: rSTAGINGf92090407be0: Merge #12447: test: Add missing signal.h header.Sep 19 2019, 01:47

Revision Contents
Changeset List

Path

Size

src/

test/

util_tests.cpp

1 line

Diff 10758

View Options

Merge #12447: test: Add missing signal.h headerClosedPublicActions