[Asterisk-code-review] CI: Use .gitreview to default BRANCH NAME. (asterisk[16])

George Joseph asteriskteam at digium.com
Thu Sep 13 10:37:08 CDT 2018


George Joseph has submitted this change and it was merged. ( https://gerrit.asterisk.org/10129 )

Change subject: CI: Use .gitreview to default BRANCH_NAME.
......................................................................

CI: Use .gitreview to default BRANCH_NAME.

This ensures that binary modules are avoided in the master branch even
if BRANCH_NAME is not set.

Change-Id: I79162d2063f22fa9d6b31fde4827ace2dd5bf0da
---
M tests/CI/buildAsterisk.sh
1 file changed, 4 insertions(+), 0 deletions(-)

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



diff --git a/tests/CI/buildAsterisk.sh b/tests/CI/buildAsterisk.sh
index 992dc94..b974629 100755
--- a/tests/CI/buildAsterisk.sh
+++ b/tests/CI/buildAsterisk.sh
@@ -6,6 +6,10 @@
 DISABLE_BINARY_MODULES=0
 source $CIDIR/ci.functions
 
+if [ -z $BRANCH_NAME ]; then
+	BRANCH_NAME=$(git config -f .gitreview --get gerrit.defaultbranch)
+fi
+
 gen_cats() {
 	set +x
 	action=$1

-- 
To view, visit https://gerrit.asterisk.org/10129
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: I79162d2063f22fa9d6b31fde4827ace2dd5bf0da
Gerrit-Change-Number: 10129
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.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/20180913/2888c278/attachment-0001.html>


More information about the asterisk-code-review mailing list