From b46d8c0a3952e123a3901e19edaecf60aaf187f3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Butz?=
 <67107050+MAButz@users.noreply.github.com>
Date: Tue, 18 Oct 2022 16:21:43 +0200
Subject: [PATCH] Update proxmox-zfs-postinstall.sh

Suggested from teissler:
-change (y/n)? to (y/N)?   ;This way it is easier to understand that everything except "y" doesn't enables the pve-no-subscription repo.
-$response to lowercase  ;would also improve the scripts failure proofness.
---
 proxmox-zfs-postinstall.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/proxmox-zfs-postinstall.sh b/proxmox-zfs-postinstall.sh
index da83dd1..51705a8 100644
--- a/proxmox-zfs-postinstall.sh
+++ b/proxmox-zfs-postinstall.sh
@@ -167,9 +167,9 @@ fi
 # disable pve-enterprise repo and add pve-no-subscription repo
 
 #Not tested, yet!
-read -p "Do you want to disable pve-enterprise repo and add pve-no-subscription repo (y/n)? " response
+read -p "Do you want to disable pve-enterprise repo and add pve-no-subscription repo (y/N)? " response
 
-if [ "$response" == "y" ]; then
+if [ "${response,,}" == "y" ]; then
     if [[ "$(uname -r)" == *"-pve" ]]; then
         echo "Deactivating pve-enterprise repository"
         mv /etc/apt/sources.list.d/pve-enterprise.list /etc/apt/sources.list.d/pve-enterprise.list.bak > /dev/null 2>&1