diff mbox series

[bug#51829] Better error message for mcron file loading

Message ID 87sfvztkxh.fsf@enzu.ru
State Accepted
Headers show
Series [bug#51829] Better error message for mcron file loading | expand

Checks

Context Check Description
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue

Commit Message

Ahmed Khanzada Nov. 14, 2021, 4:34 a.m. UTC
I was a bit puzzled why mcron was failing; based on the error, I assumed
file permissions, and did not realize the actual problem was my faulty
syntax within a file.

Until we can get more detailed error messages in place, I propose this
patch to make the current generic error we have slightly more helpful.

Comments

Ludovic Courtès Nov. 17, 2021, 4:09 p.m. UTC | #1
Hi,

Ahmed Khanzada <me@enzu.ru> skribis:

> I was a bit puzzled why mcron was failing; based on the error, I assumed
> file permissions, and did not realize the actual problem was my faulty
> syntax within a file.
>
> Until we can get more detailed error messages in place, I propose this
> patch to make the current generic error we have slightly more helpful.
>  
> diff --git a/src/mcron/scripts/mcron.scm b/src/mcron/scripts/mcron.scm

Could you send this patch to bug-mcron@gnu.org?

Thanks,
Ludo’.
diff mbox series

Patch

diff --git a/src/mcron/scripts/mcron.scm b/src/mcron/scripts/mcron.scm
index 4c5f7b5..3e581fa 100644
--- a/src/mcron/scripts/mcron.scm
+++ b/src/mcron/scripts/mcron.scm
@@ -93,7 +93,7 @@  $XDG_CONFIG_HOME is not defined uses ~/.config/cron instead)."
                               "/cron")))
     (when (eq? 2 errors)
       (mcron-error 13
-        "Cannot read files in your ~/.config/cron (or ~/.cron) directory."))))
+        "There was an error reading files in your ~/.config/cron (or ~/.cron) directory. Double-check the folder and file permissions and syntax."))))