[helm-plugin] Fix bash issues with scripts/install.sh

charted-dev · Apr 12, 2024 · 8831d31 · 8831d31
1 parent 9ef905d
commit 8831d31
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/tools/helm-plugin/plugin.yaml b/tools/helm-plugin/plugin.yaml
@@ -14,7 +14,7 @@
 # limitations under the License.
 
 name: charted
-version: 0.1.0-beta
+version: '0.1.0-beta'
 description: 🐻‍❄️🌺 Helm plugin to help you push your Helm charts into charted-server easily!~
 command: '$HELM_PLUGIN_DIR/bin/charted-helm-plugin'
 useTunnel: false

diff --git a/tools/helm-plugin/scripts/update.sh b/tools/helm-plugin/scripts/update.sh
@@ -23,7 +23,7 @@ if [ -n "${CHARTED_HELM_NO_INSTALL_HOOK:-}" ]; then
 fi
 
 function charted::helm::os {
-    case "$(uname -s)"; do
+    case "$(uname -s)" in
         Darwin)
             echo "darwin"
             ;;
@@ -44,7 +44,7 @@ function charted::helm::os {
 }
 
 function charted::helm::architecture {
-    case "$(uname -m)"; do
+    case "$(uname -m)" in
         x86_64|amd64)
             echo "x86_64"
             ;;
@@ -110,7 +110,7 @@ function charted::helm::on_exit {
 
 trap charted::helm::on_exit EXIT
 
-version="$(grep 'version' plugin.yaml | cut -d '"' -f 2)"
+version="$(grep 'version' plugin.yaml | cut -d "'" -f 2)"
 BINARY_URL=$(charted::helm::binary_url "$version")
 CHECKSUM_URL=$(charted::helm::checksum_url "$version")