[Asterisk-code-review] chan_sip.c: Don't build by default (asterisk[master])

George Joseph asteriskteam at digium.com
Tue Sep 22 09:03:35 CDT 2020


George Joseph has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/14923 )

Change subject: chan_sip.c: Don't build by default
......................................................................

chan_sip.c: Don't build by default

ASTERISK-29083 #close

Change-Id: I9ea25fba3ba8f63a47886894bd966e0f08d5e003
---
M channels/chan_sip.c
A doc/UPGRADE-staging/do-not-build-chan-sip-by-default.txt
2 files changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved; Approved for Submit



diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 78c0bb3..e506d35 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -176,6 +176,7 @@
 /*** MODULEINFO
 	<use type="module">res_crypto</use>
 	<use type="module">res_http_websocket</use>
+	<defaultenabled>no</defaultenabled>
 	<support_level>deprecated</support_level>
  ***/
 
diff --git a/doc/UPGRADE-staging/do-not-build-chan-sip-by-default.txt b/doc/UPGRADE-staging/do-not-build-chan-sip-by-default.txt
new file mode 100644
index 0000000..31790e4
--- /dev/null
+++ b/doc/UPGRADE-staging/do-not-build-chan-sip-by-default.txt
@@ -0,0 +1,5 @@
+Subject: chan_sip
+Master-Only: True
+
+chan_sip is no longer built by default. To build it, make sure to
+enable it when running 'make menuselect'

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/14923
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I9ea25fba3ba8f63a47886894bd966e0f08d5e003
Gerrit-Change-Number: 14923
Gerrit-PatchSet: 4
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200922/a9237ccb/attachment.html>


More information about the asterisk-code-review mailing list