From c688c23d465e19b15972df11f2d733779453eaba Mon Sep 17 00:00:00 2001
From: Laurence Tratt <laurie@tratt.net>
Date: Mon, 27 Apr 2026 09:19:44 +0100
Subject: [PATCH] Temporary failures do not mean that the queue is empty.

---
 src/jobrunner.rs | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/jobrunner.rs b/src/jobrunner.rs
index 6607dfb..7b31eaa 100644
--- a/src/jobrunner.rs
+++ b/src/jobrunner.rs
@@ -293,11 +293,10 @@ impl JobRunner {
                         check_queue = false;
                         tmp_failure = false;
                     }
-                    (true, true) => {
+                    (false | true, true) => {
                         check_queue = true;
                         tmp_failure = true;
                     }
-                    (false, true) => unreachable!(),
                 }
             }
         }
@@ -336,7 +335,7 @@ impl JobRunner {
                         Err(Some(qj)) => {
                             // The job couldn't be run for temporary reasons: we'll retry later.
                             queue.push_front(qj);
-                            return (true, true);
+                            return (false, true);
                         }
                         Err(None) => {
                             // The job couldn't be run for permanent reasons: it has been consumed