[Asterisk-code-review] Fixes to include signal.h (asterisk[master])

Timo Teräs asteriskteam at digium.com
Wed Jun 8 12:37:40 CDT 2016


Hello Anonymous Coward #1000019, Joshua Colp,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/2937

to look at the new patch set (#2).

Change subject: Fixes to include signal.h
......................................................................

Fixes to include signal.h

POSIX defines signal.h. sys/signal.h should not be used as it is
c-library internal header which may or may not exist. Notably with
musl it generates warning of being incorrect.

Change-Id: Ia56b0aa1d84b5c590114867b1b384a624f39a6fc
---
M addons/chan_ooh323.h
M apps/app_dahdiras.c
M apps/app_dial.c
M apps/app_queue.c
M channels/chan_console.c
M channels/chan_dahdi.c
M channels/chan_iax2.c
M channels/chan_mgcp.c
M channels/chan_motif.c
M channels/chan_sip.c
M channels/chan_skinny.c
M main/cli.c
M main/features.c
M main/http.c
M main/tcptls.c
M pbx/pbx_dundi.c
M res/res_musiconhold.c
17 files changed, 9 insertions(+), 23 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/37/2937/2
-- 
To view, visit https://gerrit.asterisk.org/2937
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia56b0aa1d84b5c590114867b1b384a624f39a6fc
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Timo Teräs <timo.teras at iki.fi>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list