[Asterisk-code-review] CI: Update buildAsterisk.sh to do a "make full" (asterisk[17])

Friendly Automation asteriskteam at digium.com
Thu Jan 9 14:29:31 CST 2020


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/13576 )

Change subject: CI: Update buildAsterisk.sh to do a "make full"
......................................................................

CI: Update buildAsterisk.sh to do a "make full"

If you do a "make all" when building Asterisk the xml documentation
produced will be missing certain AMI events where their
documentation is located not at the top of the c source file but
embedded further down next to the event's manager_event()
registration call.  See main/manager_mwi.c for an example.

"make full" does produce the correct documentation so we're changing
it in the build script.  A separate commit/issue will address the
problem with "make all".

ASTERISK-28507
Reported by: David Lee

Change-Id: I4a22635d6eef99eacecc0efb69e28360eebdb86c
---
M tests/CI/buildAsterisk.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Sean Bright: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/tests/CI/buildAsterisk.sh b/tests/CI/buildAsterisk.sh
index 6278a47..2deb549 100755
--- a/tests/CI/buildAsterisk.sh
+++ b/tests/CI/buildAsterisk.sh
@@ -165,7 +165,7 @@
 fi
 
 if [ $NO_MAKE -eq 0 ] ; then
-	runner ${MAKE} -j8 || runner ${MAKE} -j1 NOISY_BUILD=yes
+runner ${MAKE} -j8 full || runner ${MAKE} -j1 NOISY_BUILD=yes full
 fi
 
 runner rm -f ${LCOV_DIR}/*.info

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

Gerrit-Project: asterisk
Gerrit-Branch: 17
Gerrit-Change-Id: I4a22635d6eef99eacecc0efb69e28360eebdb86c
Gerrit-Change-Number: 13576
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200109/5ab9591e/attachment.html>


More information about the asterisk-code-review mailing list