fix backup source handling in case sanitizing fails

Signed-off-by: simonsan <[email protected]>
rustic-rs · Dec 4, 2024 · 84c2cb6 · 84c2cb6
1 parent f565eea
commit 84c2cb6
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/src/commands/backup.rs b/src/commands/backup.rs
@@ -208,7 +208,10 @@ impl BackupCmd {
                 })
                 .collect();
 
-            let snapshot_sources = match (self.cli_sources.is_empty(), snapshot_opts.is_empty()) {
+            let snapshot_sources = match (
+                self.cli_sources.is_empty(),
+                config_snapshot_sources.is_empty(),
+            ) {
                 (false, _) => {
                     let item = PathList::from_iter(&self.cli_sources).sanitize()?;
                     vec![item]
@@ -218,12 +221,9 @@ impl BackupCmd {
                     config_snapshot_sources.clone()
                 }
                 (true, true) => {
-                    bail!("no backup source given.");
+                    bail!("No usable backup source. Please make sure that the given sources are valid.");
                 }
             };
-            if snapshot_sources.is_empty() {
-                return Ok(());
-            }
 
             let mut is_err = false;
             for sources in snapshot_sources {
@@ -239,6 +239,7 @@ impl BackupCmd {
                     is_err = true;
                 }
             }
+
             if is_err {
                 Err(anyhow!("Not all snapshots were generated successfully!"))
             } else {