[Asterisk-code-review] core: AST DEVMODE no longer affects ABI. (asterisk[16])

Joshua Colp asteriskteam at digium.com
Tue Jul 24 05:28:47 CDT 2018


Joshua Colp has submitted this change and it was merged. ( https://gerrit.asterisk.org/9613 )

Change subject: core: AST_DEVMODE no longer affects ABI.
......................................................................

core: AST_DEVMODE no longer affects ABI.

Remove AST_DEVMODE from the AST_BUILDOPTS list and the AST_BUILDOPTS_SUM
calculation as it no longer affects API/ABI compatibility.

Change-Id: Id5bd6dfade173a53b3a49f715586b86e3fb24acb
---
M build_tools/make_buildopts_h
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Corey Farrell: Looks good to me, but someone else must approve
  Benjamin Keith Ford: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved; Approved for Submit



diff --git a/build_tools/make_buildopts_h b/build_tools/make_buildopts_h
index c47d508..57df27d 100755
--- a/build_tools/make_buildopts_h
+++ b/build_tools/make_buildopts_h
@@ -14,7 +14,8 @@
 if ${GREP} "AST_DEVMODE" makeopts | ${GREP} -q "yes"
 then
 	echo "#define AST_DEVMODE 1"
-	BUILDOPTS="AST_DEVMODE"
+	# AST_DEVMODE is no longer an API/ABI affecting option so it no longer
+	# gets added to BUILDOPTS.
 fi
 
 TMP=`${GREP} -e "^MENUSELECT_CFLAGS" menuselect.makeopts | sed 's/MENUSELECT_CFLAGS\=//g' | sed 's/-D//g'`

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-MessageType: merged
Gerrit-Change-Id: Id5bd6dfade173a53b3a49f715586b86e3fb24acb
Gerrit-Change-Number: 9613
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180724/e79b55cd/attachment.html>


More information about the asterisk-code-review mailing list