unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#51829] Better error message for mcron file loading
@ 2021-11-14  4:34 Ahmed Khanzada
  2021-11-17 16:09 ` Ludovic Courtès
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmed Khanzada @ 2021-11-14  4:34 UTC (permalink / raw)
  To: 51829

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
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."))))
 
 
 




^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [bug#51829] Better error message for mcron file loading
  2021-11-14  4:34 [bug#51829] Better error message for mcron file loading Ahmed Khanzada
@ 2021-11-17 16:09 ` Ludovic Courtès
  0 siblings, 0 replies; 2+ messages in thread
From: Ludovic Courtès @ 2021-11-17 16:09 UTC (permalink / raw)
  To: Ahmed Khanzada; +Cc: 51829

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’.




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-11-17 16:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-14  4:34 [bug#51829] Better error message for mcron file loading Ahmed Khanzada
2021-11-17 16:09 ` Ludovic Courtès

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).