From 5e544c4a89a8698df0719f7bc181b41939bc73cf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?E=CC=81ric=20Gaspar?=
 <46165813+ericgaspar@users.noreply.github.com>
Date: Mon, 2 Sep 2024 09:00:26 +0200
Subject: [PATCH] cleaning

---
 manifest.toml   | 2 +-
 scripts/backup  | 8 --------
 scripts/install | 6 ++----
 scripts/upgrade | 6 ++----
 4 files changed, 5 insertions(+), 17 deletions(-)

diff --git a/manifest.toml b/manifest.toml
index 9a4d5ae..7354456 100644
--- a/manifest.toml
+++ b/manifest.toml
@@ -18,7 +18,7 @@ admindoc = "https://borgwarehouse.com/docs/admin-manual/debian-installation/"
 code = "https://github.com/ravinou/borgwarehouse"
 
 [integration]
-yunohost = ">= 11.2.18"
+yunohost = ">= 11.2.29"
 helpers_version = "2.1"
 architectures = "all"
 multi_instance = true
diff --git a/scripts/backup b/scripts/backup
index a2d2ecd..ea3bda7 100755
--- a/scripts/backup
+++ b/scripts/backup
@@ -25,16 +25,8 @@ ynh_backup "/home/yunohost.app/$app/repos"
 
 ynh_backup "/etc/nginx/conf.d/$domain.d/$app.conf"
 
-#=================================================
-# BACKUP SYSTEMD
-#=================================================
-
 ynh_backup "/etc/systemd/system/$app.service"
 
-#=================================================
-# BACKUP VARIOUS FILES
-#=================================================
-
 ynh_backup "/etc/cron.d/$app"
 
 #=================================================
diff --git a/scripts/install b/scripts/install
index 508a04e..b62ce55 100755
--- a/scripts/install
+++ b/scripts/install
@@ -55,8 +55,6 @@ ynh_config_add_nginx
 ynh_config_add_systemd
 
 ynh_config_add --template="cron" --destination="/etc/cron.d/$app"
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown root: "/etc/cron.d/$app"
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod 644 "/etc/cron.d/$app"
 
 yunohost service add $app --description="WebUI for BorgBackup" --log="/var/log/$app/$app.log"
 
@@ -79,8 +77,8 @@ ynh_script_progression "Installing $app..."
 
 pushd /home/yunohost.app/$app/app
 
-	ynh_hide_warnings ynh_exec_as_app node_load_PATH npm ci
-	ynh_hide_warnings ynh_exec_as_app node_load_PATH NODE_ENV=production npm run build
+	ynh_hide_warnings ynh_exec_as_app npm ci
+	ynh_hide_warnings ynh_exec_as_app NODE_ENV=production npm run build
 popd
 
 #=================================================
diff --git a/scripts/upgrade b/scripts/upgrade
index 7b80e92..e625dbf 100755
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -41,8 +41,6 @@ ynh_config_add_systemd
 
 # Update cron in case of changes
 ynh_config_add --template="cron" --destination="/etc/cron.d/$app"
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown root: "/etc/cron.d/$app"
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod 644 "/etc/cron.d/$app"
 
 yunohost service add $app --description="WebUI for BorgBackup" --log="/var/log/$app/$app.log"
 
@@ -64,8 +62,8 @@ chown $app:$app "/home/yunohost.app/$app/app/.env.local"
 
 pushd /home/yunohost.app/$app/app
 
-	ynh_hide_warnings ynh_exec_as_app node_load_PATH npm ci
-	ynh_hide_warnings ynh_exec_as_app node_load_PATH NODE_ENV=production npm run build
+	ynh_hide_warnings ynh_exec_as_app npm ci
+	ynh_hide_warnings ynh_exec_as_app NODE_ENV=production npm run build
 popd
 
 #=================================================