[Asterisk-code-review] releaser.sh - Fix wrong 'current' links (repotools[master])

Joshua Colp asteriskteam at digium.com
Thu Apr 28 05:28:01 CDT 2016


Joshua Colp has submitted this change and it was merged.

Change subject: releaser.sh - Fix wrong 'current' links
......................................................................


releaser.sh - Fix wrong 'current' links

When adding 'current' links the releaser script would create links without the
branch. This patch removes the errant code.

Change-Id: Idf4964aafb357e2746705ac8d5e98c615f171268
---
M releaser.sh
1 file changed, 12 insertions(+), 27 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, approved; Verified



diff --git a/releaser.sh b/releaser.sh
index 951dacf..6d8e46f 100755
--- a/releaser.sh
+++ b/releaser.sh
@@ -40,33 +40,18 @@
 
 add_links () {
     echo "Adding current links for $1 $2."
-    if [ -z ${var+x} ]; then
-	ln -s releases/$1-$2.tar.gz ./$1-current.tar.gz
-	ln -s releases/$1-$2.tar.gz.asc ./$1-current.tar.gz.asc
-	ln -s releases/$1-$2.sha1 ./$1-current.sha1
-	ln -s releases/$1-$2.sha256 ./$1-current.sha256
-	ln -s releases/$1-$2.md5 ./$1-current.md5
-	ln -s releases/$1-$2-patch.tar.gz ./$1-current-patch.tar.gz
-	ln -s releases/$1-$2-patch.tar.gz.asc ./$1-current-patch.tar.gz.asc
-	ln -s releases/README-$2 ./README-current
-	ln -s releases/ChangeLog-$2 ./ChangeLog-current
-	ln -s releases/$1-$2-summary.html ./$1-current-summary.html
-	ln -s releases/$1-$2-summary.txt ./$1-current-summary.txt
-	svn add *-current*
-    else
-	ln -s releases/$1-$2.tar.gz ./$1-$3-current.tar.gz
-	ln -s releases/$1-$2.tar.gz.asc ./$1-$3-current.tar.gz.asc
-	ln -s releases/$1-$2.sha1 ./$1-$3-current.sha1
-	ln -s releases/$1-$2.sha256 ./$1-$3-current.sha256
-	ln -s releases/$1-$2.md5 ./$1-$3-current.md5
-	ln -s releases/$1-$2-patch.tar.gz ./$1-$3-current-patch.tar.gz
-	ln -s releases/$1-$2-patch.tar.gz.asc ./$1-$3-current-patch.tar.gz.asc
-	ln -s releases/README-$2 ./README-$3-current
-	ln -s releases/ChangeLog-$2 ./ChangeLog-$3-current
-	ln -s releases/$1-$2-summary.html ./$1-$3-current-summary.html
-	ln -s releases/$1-$2-summary.txt ./$1-$3-current-summary.txt
-	svn add *$3-current*
-    fi
+    ln -s releases/$1-$2.tar.gz ./$1-$3-current.tar.gz
+    ln -s releases/$1-$2.tar.gz.asc ./$1-$3-current.tar.gz.asc
+    ln -s releases/$1-$2.sha1 ./$1-$3-current.sha1
+    ln -s releases/$1-$2.sha256 ./$1-$3-current.sha256
+    ln -s releases/$1-$2.md5 ./$1-$3-current.md5
+    ln -s releases/$1-$2-patch.tar.gz ./$1-$3-current-patch.tar.gz
+    ln -s releases/$1-$2-patch.tar.gz.asc ./$1-$3-current-patch.tar.gz.asc
+    ln -s releases/README-$2 ./README-$3-current
+    ln -s releases/ChangeLog-$2 ./ChangeLog-$3-current
+    ln -s releases/$1-$2-summary.html ./$1-$3-current-summary.html
+    ln -s releases/$1-$2-summary.txt ./$1-$3-current-summary.txt
+    svn add *$3-current*
 }
 
 if [ "$TYPE" = "release" ]; then

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idf4964aafb357e2746705ac8d5e98c615f171268
Gerrit-PatchSet: 1
Gerrit-Project: repotools
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list