From 8e96d8b29e9f55b3df8f68b07fbd099d6c5e09e8 Mon Sep 17 00:00:00 2001
From: Adhithya Srinivasan <adhithya.srinivasan@walmart.com>
Date: Sun, 31 Jul 2022 13:35:19 +0530
Subject: [PATCH] updating the existing function take parameter for compression

Signed-off-by: Adhithya Srinivasan <adhithya.srinivasan@walmart.com>
---
 scripts/package-deploy.sh | 52 ++++++++++++++++-----------------------
 1 file changed, 21 insertions(+), 31 deletions(-)

diff --git a/scripts/package-deploy.sh b/scripts/package-deploy.sh
index eee8ae51dcb..55de936719e 100755
--- a/scripts/package-deploy.sh
+++ b/scripts/package-deploy.sh
@@ -31,36 +31,26 @@ function stage-platform-files {
 # package pulls built files for the platform ($1). If you pass in a file
 # extension ($2) it will be used on the binaries
 function package {
-    local PLATFORM=$1
-    local FILE_EXTENSION=$2
-    # script start
-    local PACKAGE_STAGING_DIR=jaeger-$VERSION-$PLATFORM
-
-    mkdir $PACKAGE_STAGING_DIR
-
-    stage-platform-files $PLATFORM $PACKAGE_STAGING_DIR $FILE_EXTENSION
-
-    local ARCHIVE_NAME="$PACKAGE_STAGING_DIR.tar.gz"
-    echo "Packaging into $ARCHIVE_NAME:"
-    tar -czvf ./deploy/$ARCHIVE_NAME $PACKAGE_STAGING_DIR
-    rm -rf $PACKAGE_STAGING_DIR
-}
-
-# package pulls built files for the platform ($1). If you pass in a file
-# extension ($2) it will be used on the binaries
-function package_zip {
-    local PLATFORM=$1
-    local FILE_EXTENSION=$2
+    
+    local COMPRESSION=$1
+    local PLATFORM=$2
+    local FILE_EXTENSION=$3
     # script start
     local PACKAGE_STAGING_DIR=jaeger-$VERSION-$PLATFORM
 
     mkdir $PACKAGE_STAGING_DIR
 
     stage-platform-files $PLATFORM $PACKAGE_STAGING_DIR $FILE_EXTENSION
-
-    local ARCHIVE_NAME="$PACKAGE_STAGING_DIR.zip"
-    echo "Packaging into $ARCHIVE_NAME:"
-    zip -r ./deploy/$ARCHIVE_NAME $PACKAGE_STAGING_DIR
+    if [ "$COMPRESSION" == "zip" ]
+    then
+        local ARCHIVE_NAME="$PACKAGE_STAGING_DIR.zip"
+        echo "Packaging into $ARCHIVE_NAME:"
+        zip -r ./deploy/$ARCHIVE_NAME $PACKAGE_STAGING_DIR
+    else
+        local ARCHIVE_NAME="$PACKAGE_STAGING_DIR.tar.gz"
+        echo "Packaging into $ARCHIVE_NAME:"
+        tar -czvf ./deploy/$ARCHIVE_NAME $PACKAGE_STAGING_DIR
+    fi
     rm -rf $PACKAGE_STAGING_DIR
 }
 
@@ -75,10 +65,10 @@ rm -rf deploy $DEPLOY_STAGING_DIR
 mkdir deploy
 mkdir $DEPLOY_STAGING_DIR
 
-package linux-amd64
-package darwin-amd64
-package darwin-arm64
-package_zip windows-amd64 .exe
-package linux-s390x
-package linux-arm64
-package linux-ppc64le
+package tar linux-amd64
+package tar darwin-amd64
+package tar darwin-arm64
+package zip windows-amd64 .exe 
+package tar linux-s390x
+package tar linux-arm64
+package tar linux-ppc64le