[Asterisk-code-review] Fix #include poll.h and sys/cdefs.h (asterisk[master])

Alexander Traud asteriskteam at digium.com
Sat Jun 23 08:53:09 CDT 2018


Alexander Traud has posted comments on this change. ( https://gerrit.asterisk.org/2934 )

Change subject: Fix #include poll.h and sys/cdefs.h
......................................................................


Patch Set 1:

Time, why exactly did you add sys/cdefs.h  – is a symbol used which requires this? Which one exactly?

According to the musl FAQ <https://wiki.musl-libc.org/faq.html>, this is a bug and should not be done. Because of this inclusion, I am not able to compile main/ast_expr2 in Oracle Solaris.


-- 
To view, visit https://gerrit.asterisk.org/2934
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I142930df53fe7585a06b854b6faddc5301e024be
Gerrit-Change-Number: 2934
Gerrit-PatchSet: 1
Gerrit-Owner: Timo Teräs <timo.teras at iki.fi>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Comment-Date: Sat, 23 Jun 2018 13:53:09 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180623/694aacf2/attachment.html>


More information about the asterisk-code-review mailing list