[Asterisk-code-review] CI: Use brace expansion instead of calling out to seq (asterisk[13])

Sean Bright asteriskteam at digium.com
Wed Oct 3 09:06:37 CDT 2018


Sean Bright has uploaded this change for review. ( https://gerrit.asterisk.org/10403


Change subject: CI: Use brace expansion instead of calling out to seq
......................................................................

CI: Use brace expansion instead of calling out to seq

Also make the shebang in publishAsteriskDocs.sh the first line.

Change-Id: I3fdd6f22e652e4fb5b5fe85df46fa34eb6d0cf08
---
M tests/CI/publishAsteriskDocs.sh
M tests/CI/runUnittests.sh
2 files changed, 4 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/03/10403/1

diff --git a/tests/CI/publishAsteriskDocs.sh b/tests/CI/publishAsteriskDocs.sh
index d5c857a..ab91714 100755
--- a/tests/CI/publishAsteriskDocs.sh
+++ b/tests/CI/publishAsteriskDocs.sh
@@ -1,7 +1,7 @@
+#!/usr/bin/env bash
 #
 # Publish Asterisk documentation to the wiki
 #
-#!/usr/bin/env bash
 CIDIR=$(dirname $(readlink -fn $0))
 source $CIDIR/ci.functions
 ASTETCDIR=$DESTDIR/etc/asterisk
@@ -113,7 +113,7 @@
 rm -f ${OUTPUTDIR}/full-en_US.xml
 
 sudo $ASTERISK ${USER_GROUP:+-U ${USER_GROUP%%:*} -G ${USER_GROUP##*:}} -gn -C $CONFFILE
-for n in `seq 1 5` ; do
+for n in {1..5} ; do
 	sleep 3
 	$ASTERISK -rx "core waitfullybooted" -C $CONFFILE && break
 done
@@ -141,4 +141,4 @@
     --ast-version="${AST_VER}" \
     -v
 
-popd
\ No newline at end of file
+popd
diff --git a/tests/CI/runUnittests.sh b/tests/CI/runUnittests.sh
index 1d2656b..d451362 100755
--- a/tests/CI/runUnittests.sh
+++ b/tests/CI/runUnittests.sh
@@ -55,7 +55,7 @@
 
 set -x
 sudo $ASTERISK ${USER_GROUP:+-U ${USER_GROUP%%:*} -G ${USER_GROUP##*:}} -gn -C $CONFFILE
-for n in `seq 1 5` ; do
+for n in {1..5} ; do
 	sleep 3
 	$ASTERISK -rx "core waitfullybooted" -C $CONFFILE && break
 done

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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3fdd6f22e652e4fb5b5fe85df46fa34eb6d0cf08
Gerrit-Change-Number: 10403
Gerrit-PatchSet: 1
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181003/8f120805/attachment.html>


More information about the asterisk-code-review mailing list