build: Fix #include sys/poll.h to just poll.h (without sys/) #17084

pull carnhofdaki wants to merge 1 commits into bitcoin:master from carnhofdaki:cdk/sys_poll changing 1 files +1 −1

carnhofdaki commented at 10:36 am on October 9, 2019: contributor

http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html http://man7.org/linux/man-pages/man2/poll.2.html
laanwj added the label Linux/Unix on Oct 9, 2019
laanwj commented at 10:38 am on October 9, 2019: member

ACK 0a996375c165293535ebf27c8103767963f1572e does not need backport to any releases, this code is new in master (#13716)
laanwj renamed this:
~~Trivial: Fix #include sys/poll.h to just poll.h (without sys/)~~
build: Fix #include sys/poll.h to just poll.h (without sys/)
on Oct 9, 2019
theStack commented at 10:56 am on October 9, 2019: member

ACK https://github.com/bitcoin/bitcoin/commit/0a996375c165293535ebf27c8103767963f1572e

build: Fix #include sys/poll.h to just poll.h (without sys/)

http://pubs.opengroup.org/onlinepubs/009695399/functions/poll.html
http://man7.org/linux/man-pages/man2/poll.2.html

4de0bde7bc

carnhofdaki force-pushed on Oct 9, 2019
Empact commented at 11:37 am on October 9, 2019: member

ACK 4de0bde7bc546638cbbbb8bc4eb0d840d81cb953
fanquake commented at 12:05 pm on October 9, 2019: member

@carnhofdaki were you seeing a particular compile failure, or just making this align with the spec?
carnhofdaki commented at 12:41 pm on October 9, 2019: contributor

@fanquake no failure, just a warning.
MarcoFalke added this to the milestone 0.20.0 on Oct 9, 2019
laanwj referenced this in commit 97e05817d4 on Oct 9, 2019
laanwj merged this on Oct 9, 2019
laanwj closed this on Oct 9, 2019
fanquake approved
fanquake commented at 1:03 pm on October 9, 2019: member

ACK 4de0bde7bc546638cbbbb8bc4eb0d840d81cb953 - Tested on macOS 10.14.6, Debian 10.1 and OpenBSD 6.5.
MarkLTZ referenced this in commit beb88e1920 on Nov 17, 2019
jasonbcox referenced this in commit 939736aaca on Oct 26, 2020
DrahtBot locked this on Dec 16, 2021

Milestone
0.20.0