From e682fa768efaa540326c4d8f514fb017f630c88d Mon Sep 17 00:00:00 2001
From: Wynn Wolf Arbor
Date: Wed, 16 Sep 2020 19:32:48 +0200
Subject: Make sure to return successfully if config/sync is missing

In sync_default, we test whether the config/sync file exists before
reading from it. If it does not exist, we return from the function using
the exit status of the last command (test), which in this case is 1.

For default setups that do not specify a config/sync file, this will
lead to the script as a whole exiting unsuccessfully even though no
error occurred. Make sure to set the exit status to 0 in order to signal
successful completion of the sync task.

Reported-by: Jan Rathner <jarainf@freenode>
---
 lifeboat | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lifeboat b/lifeboat
index 3546f5a..e84c916 100755
--- a/lifeboat
+++ b/lifeboat
@@ -44,7 +44,7 @@ clean_default() {
 }
 
 sync_default() {
-	test -r config/sync || return
+	test -r config/sync || return 0
 
 	while read -r target_spec; do
 		target=$(echo "$target_spec" | cut -f1)
-- 
cgit v1.2.3-2-gb3c3