[Asterisk-code-review] download externals: Fix issue with re-install (asterisk[master])

George Joseph asteriskteam at digium.com
Tue Sep 27 16:13:13 CDT 2016


George Joseph has uploaded a new change for review.

  https://gerrit.asterisk.org/3995

Change subject: download_externals: Fix issue with re-install
......................................................................

download_externals: Fix issue with re-install

Needed to ignore an xmlstarlet return code for optional element.

Change-Id: I6a96f709b4b38c9a3f3dda4e8b07903787e16873
Reported-by: Dan Jenkins
---
M build_tools/download_externals
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/95/3995/1

diff --git a/build_tools/download_externals b/build_tools/download_externals
index 9b2b841..2bc357c 100755
--- a/build_tools/download_externals
+++ b/build_tools/download_externals
@@ -134,7 +134,7 @@
 		sums=$(${XMLSTARLET} sel -t -m "//file" -v "@md5sum" -n ${DESTDIR}${ASTMODDIR}/${module_name}.manifest.xml)
 		for sum in ${sums} ; do
 			install_path=$(${XMLSTARLET} sel -t -v "//file[@md5sum = '${sum}']/@install_path" ${DESTDIR}${ASTMODDIR}/${module_name}.manifest.xml )
-			executable=$(${XMLSTARLET} sel -t -v "//file[@md5sum = '${sum}']/@executable" ${DESTDIR}${ASTMODDIR}/${module_name}.manifest.xml )
+			executable=$(${XMLSTARLET} sel -t -v "//file[@md5sum = '${sum}']/@executable" ${DESTDIR}${ASTMODDIR}/${module_name}.manifest.xml || : )
 			f=${DESTDIR}$(eval echo ${install_path})
 			if [[ ! -f ${f} ]] ; then
 				echo Not found: ${f}

-- 
To view, visit https://gerrit.asterisk.org/3995
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6a96f709b4b38c9a3f3dda4e8b07903787e16873
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: George Joseph <gjoseph at digium.com>



More information about the asterisk-code-review mailing list