[Asterisk-code-review] make_tarball: Fix --no-strip (repotools[master])

George Joseph asteriskteam at digium.com
Wed Oct 28 08:39:29 CDT 2020


George Joseph has submitted this change. ( https://gerrit.asterisk.org/c/repotools/+/15073 )

Change subject: make_tarball: Fix --no-strip
......................................................................

make_tarball: Fix --no-strip

Change-Id: Idd5496f949cb9b7420fccc330903ae28df7a3832
---
M build_tools/make_tarball
1 file changed, 3 insertions(+), 3 deletions(-)

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



diff --git a/build_tools/make_tarball b/build_tools/make_tarball
index 500fe61..7fdbe8a 100755
--- a/build_tools/make_tarball
+++ b/build_tools/make_tarball
@@ -29,8 +29,8 @@
 	--manifest-out=*)
 		manifest_out=${1#*=}
 		;;
-	--no_strip)
-		nostrip=1
+	--no-strip)
+		no_strip=1
 		;;
 	--version=*)
 		version=${1#*=}
@@ -49,7 +49,7 @@
 done
 
 if [[ ( -z "${package_name}" ) || ( -z "${module_list[@]}" ) ]] ; then
-	echo "Usage ${0} --package-name=<package name> [ --variant=<variant> ] [ --manifest-in=<manifest.xml.in> ] [ --arch=(x86_32 | x86_64) ] [ --no_strip ] [ --version=<x.x_y.y.y> ] <module list>"
+	echo "Usage ${0} --package-name=<package name> [ --variant=<variant> ] [ --manifest-in=<manifest.xml.in> ] [ --arch=(x86_32 | x86_64) ] [ --no-strip ] [ --version=<x.x_y.y.y> ] <module list>"
 	exit 1
 fi
 

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

Gerrit-Project: repotools
Gerrit-Branch: master
Gerrit-Change-Id: Idd5496f949cb9b7420fccc330903ae28df7a3832
Gerrit-Change-Number: 15073
Gerrit-PatchSet: 2
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell 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/20201028/92cc8b3d/attachment-0001.html>


More information about the asterisk-code-review mailing list