These headers are already included in a default set which are checked early during configure.
We already use at least sys/types.h and unistd.h unconditionally in configure.
build: remove already tested headers from AC_CHECK_HEADERS #26825
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:remove_already_tested_headers changing 1 files +1 −1-
fanquake commented at 7:30 PM on January 5, 2023: member
-
94d0c418c5
build: remove already tested headers from AC_CHECK_HEADERS
These headers are already included in a default set which are checked early during configure. We already use at least sys/types.h and unistd.h unconditionally in configure.
-
DrahtBot commented at 7:30 PM on January 5, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK TheCharlatan If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Build system on Jan 5, 2023
- TheCharlatan approved
-
TheCharlatan commented at 10:27 AM on January 11, 2023: contributor
Code review ACK 94d0c418c51223f67e50cfa7fa4f4d838f733914
These are e.g. in
ac_includes_default. - fanquake merged this on Jan 12, 2023
- fanquake closed this on Jan 12, 2023
- fanquake deleted the branch on Jan 12, 2023
- sidhujag referenced this in commit f828f1d980 on Jan 12, 2023
- bitcoin locked this on Jan 12, 2024
Contributors
Labels