* [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable.
@ 2023-03-13 19:02 Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 1/3] services: mcron: Restyle mcron-configuration Bruno Victal
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Bruno Victal @ 2023-03-13 19:02 UTC (permalink / raw)
To: 62169; +Cc: Bruno Victal
Tested with 'make check-system TESTS=mcron'.
Bruno Victal (3):
services: mcron: Restyle mcron-configuration.
services: mcron: Use match-record.
services: mcron: Add log-file and date-format fields.
doc/guix.texi | 8 +++-
gnu/services/mcron.scm | 104 +++++++++++++++++++++++++----------------
2 files changed, 71 insertions(+), 41 deletions(-)
--
2.39.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#62169] [PATCH 1/3] services: mcron: Restyle mcron-configuration.
2023-03-13 19:02 [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable Bruno Victal
@ 2023-03-13 19:30 ` Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 2/3] services: mcron: Use match-record Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 3/3] services: mcron: Add log-file and date-format fields Bruno Victal
2 siblings, 0 replies; 6+ messages in thread
From: Bruno Victal @ 2023-03-13 19:30 UTC (permalink / raw)
To: 62169; +Cc: Bruno Victal
* doc/guix.texi (Scheduled Job Execution): Sync doc with source.
* gnu/services/mcron.scm (mcron-configuration): Restyle.
[log-format]: Fix incorrectly formatted text.
---
doc/guix.texi | 2 +-
gnu/services/mcron.scm | 15 +++++++++++----
2 files changed, 12 insertions(+), 5 deletions(-)
diff --git a/doc/guix.texi b/doc/guix.texi
index 39932d5aad..45d96b8fbb 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -19349,7 +19349,7 @@ Scheduled Job Execution
@item @code{log-format} (default: @code{"~1@@*~a ~a: ~a~%"}) (type: string)
@code{(ice-9 format)} format string for log messages. The default value
-produces messages like "@samp{@var{pid} @var{name}: @var{message}"}
+produces messages like @samp{@var{pid} @var{name}: @var{message}}
(@pxref{Invoking mcron, Invoking,, mcron,GNU@tie{}mcron}). Each message
is also prefixed by a timestamp by GNU Shepherd.
diff --git a/gnu/services/mcron.scm b/gnu/services/mcron.scm
index 52332d6123..6ee333f253 100644
--- a/gnu/services/mcron.scm
+++ b/gnu/services/mcron.scm
@@ -56,18 +56,25 @@ (define list-of-gexps?
(list-of gexp?))
(define-configuration/no-serialization mcron-configuration
- (mcron (file-like mcron) "The mcron package to use.")
+ (mcron
+ (file-like mcron)
+ "The mcron package to use.")
+
(jobs
(list-of-gexps '())
"This is a list of gexps (@pxref{G-Expressions}), where each gexp
corresponds to an mcron job specification (@pxref{Syntax, mcron job
specifications,, mcron, GNU@tie{}mcron}).")
- (log? (boolean #t) "Log messages to standard output.")
+
+ (log?
+ (boolean #t)
+ "Log messages to standard output.")
+
(log-format
(string "~1@*~a ~a: ~a~%")
"@code{(ice-9 format)} format string for log messages. The default value
-produces messages like \"@samp{@var{pid} @var{name}:
-@var{message}\"} (@pxref{Invoking mcron, Invoking,, mcron, GNU@tie{}mcron}).
+produces messages like @samp{@var{pid} @var{name}: @var{message}}
+(@pxref{Invoking mcron, Invoking,, mcron, GNU@tie{}mcron}).
Each message is also prefixed by a timestamp by GNU Shepherd."))
(define (job-files mcron jobs)
--
2.39.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [bug#62169] [PATCH 2/3] services: mcron: Use match-record.
2023-03-13 19:02 [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 1/3] services: mcron: Restyle mcron-configuration Bruno Victal
@ 2023-03-13 19:30 ` Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 3/3] services: mcron: Add log-file and date-format fields Bruno Victal
2 siblings, 0 replies; 6+ messages in thread
From: Bruno Victal @ 2023-03-13 19:30 UTC (permalink / raw)
To: 62169; +Cc: Bruno Victal
* gnu/services/mcron.scm (mcron-shepherd-services): Use match-record.
---
gnu/services/mcron.scm | 68 ++++++++++++++++++++----------------------
1 file changed, 33 insertions(+), 35 deletions(-)
diff --git a/gnu/services/mcron.scm b/gnu/services/mcron.scm
index 6ee333f253..9f3afecf62 100644
--- a/gnu/services/mcron.scm
+++ b/gnu/services/mcron.scm
@@ -143,41 +143,39 @@ (define (shepherd-schedule-action mcron files)
(display line)
(loop)))))))))
-(define mcron-shepherd-services
- (match-lambda
- (($ <mcron-configuration> mcron ()) ;nothing to do!
- '())
- (($ <mcron-configuration> mcron jobs log? log-format)
- (let ((files (job-files mcron jobs)))
- (list (shepherd-service
- (provision '(mcron))
- (requirement '(user-processes))
- (modules `((srfi srfi-1)
- (srfi srfi-26)
- (ice-9 popen) ;for the 'schedule' action
- (ice-9 rdelim)
- (ice-9 match)
- ,@%default-modules))
- (start #~(make-forkexec-constructor
- (list (string-append #$mcron "/bin/mcron")
- #$@(if log?
- #~("--log" "--log-format" #$log-format)
- #~())
- #$@files)
-
- ;; Disable auto-compilation of the job files and set a
- ;; sane value for 'PATH'.
- #:environment-variables
- (cons* "GUILE_AUTO_COMPILE=0"
- "PATH=/run/current-system/profile/bin"
- (remove (cut string-prefix? "PATH=" <>)
- (environ)))
-
- #:log-file "/var/log/mcron.log"))
- (stop #~(make-kill-destructor))
-
- (actions
- (list (shepherd-schedule-action mcron files)))))))))
+(define (mcron-shepherd-services config)
+ (match-record config <mcron-configuration> (mcron jobs log? log-format)
+ (if (eq? jobs '())
+ '() ; nothing to do
+ (let ((files (job-files mcron jobs)))
+ (list (shepherd-service
+ (provision '(mcron))
+ (requirement '(user-processes))
+ (modules `((srfi srfi-1)
+ (srfi srfi-26)
+ (ice-9 popen) ;for the 'schedule' action
+ (ice-9 rdelim)
+ (ice-9 match)
+ ,@%default-modules))
+ (start #~(make-forkexec-constructor
+ (list (string-append #$mcron "/bin/mcron")
+ #$@(if log?
+ #~("--log" "--log-format" #$log-format)
+ #~())
+ #$@files)
+
+ ;; Disable auto-compilation of the job files and
+ ;; set a sane value for 'PATH'.
+ #:environment-variables
+ (cons* "GUILE_AUTO_COMPILE=0"
+ "PATH=/run/current-system/profile/bin"
+ (remove (cut string-prefix? "PATH=" <>)
+ (environ)))
+
+ #:log-file "/var/log/mcron.log"))
+ (stop #~(make-kill-destructor))
+ (actions
+ (list (shepherd-schedule-action mcron files)))))))))
(define mcron-service-type
(service-type (name 'mcron)
--
2.39.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [bug#62169] [PATCH 3/3] services: mcron: Add log-file and date-format fields.
2023-03-13 19:02 [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 1/3] services: mcron: Restyle mcron-configuration Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 2/3] services: mcron: Use match-record Bruno Victal
@ 2023-03-13 19:30 ` Bruno Victal
2023-03-21 18:45 ` [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable Maxim Cournoyer
2 siblings, 1 reply; 6+ messages in thread
From: Bruno Victal @ 2023-03-13 19:30 UTC (permalink / raw)
To: 62169; +Cc: Bruno Victal
* doc/guix.texi (Scheduled Job Execution): Document it.
* gnu/services/mcron.scm (mcron-configuration)[log-file, date-format]: New field.
(mcron-shepherd-services): Add log-file and date-format support.
Use file-append instead of string-append.
---
doc/guix.texi | 6 ++++++
gnu/services/mcron.scm | 31 +++++++++++++++++++++++++------
2 files changed, 31 insertions(+), 6 deletions(-)
diff --git a/doc/guix.texi b/doc/guix.texi
index 45d96b8fbb..6d47d36d22 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -19347,12 +19347,18 @@ Scheduled Job Execution
@item @code{log?} (default: @code{#t}) (type: boolean)
Log messages to standard output.
+@item @code{log-file} (default: @code{"/var/log/mcron.log"}) (type: string)
+Log file location.
+
@item @code{log-format} (default: @code{"~1@@*~a ~a: ~a~%"}) (type: string)
@code{(ice-9 format)} format string for log messages. The default value
produces messages like @samp{@var{pid} @var{name}: @var{message}}
(@pxref{Invoking mcron, Invoking,, mcron,GNU@tie{}mcron}). Each message
is also prefixed by a timestamp by GNU Shepherd.
+@item @code{date-format} (type: maybe-string)
+@code{(srfi srfi-19)} format string for date.
+
@end table
@end deftp
@c %end of fragment
diff --git a/gnu/services/mcron.scm b/gnu/services/mcron.scm
index 9f3afecf62..dac213358c 100644
--- a/gnu/services/mcron.scm
+++ b/gnu/services/mcron.scm
@@ -1,6 +1,7 @@
;;; GNU Guix --- Functional package management for GNU
;;; Copyright © 2016, 2017, 2018, 2019, 2020 Ludovic Courtès <ludo@gnu.org>
;;; Copyright © 2022 Maxim Cournoyer <maxim.cournoyer@gmail.com>
+;;; Copyright © 2023 Bruno Victal <mirai@makinata.eu>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -33,7 +34,9 @@ (define-module (gnu services mcron)
mcron-configuration-mcron
mcron-configuration-jobs
mcron-configuration-log?
+ mcron-configuration-log-file
mcron-configuration-log-format
+ mcron-configuration-date-format
mcron-service-type))
@@ -55,6 +58,8 @@ (define-module (gnu services mcron)
(define list-of-gexps?
(list-of gexp?))
+(define-maybe/no-serialization string)
+
(define-configuration/no-serialization mcron-configuration
(mcron
(file-like mcron)
@@ -70,12 +75,20 @@ (define-configuration/no-serialization mcron-configuration
(boolean #t)
"Log messages to standard output.")
+ (log-file
+ (string "/var/log/mcron.log")
+ "Log file location.")
+
(log-format
(string "~1@*~a ~a: ~a~%")
"@code{(ice-9 format)} format string for log messages. The default value
produces messages like @samp{@var{pid} @var{name}: @var{message}}
(@pxref{Invoking mcron, Invoking,, mcron, GNU@tie{}mcron}).
-Each message is also prefixed by a timestamp by GNU Shepherd."))
+Each message is also prefixed by a timestamp by GNU Shepherd.")
+
+ (date-format
+ maybe-string
+ "@code{(srfi srfi-19)} format string for date."))
(define (job-files mcron jobs)
"Return a list of file-like object for JOBS, a list of gexps."
@@ -144,7 +157,9 @@ (define (shepherd-schedule-action mcron files)
(loop)))))))))
(define (mcron-shepherd-services config)
- (match-record config <mcron-configuration> (mcron jobs log? log-format)
+ (match-record config <mcron-configuration> (mcron jobs
+ log? log-file log-format
+ date-format)
(if (eq? jobs '())
'() ; nothing to do
(let ((files (job-files mcron jobs)))
@@ -158,10 +173,14 @@ (define (mcron-shepherd-services config)
(ice-9 match)
,@%default-modules))
(start #~(make-forkexec-constructor
- (list (string-append #$mcron "/bin/mcron")
+ (list #$(file-append mcron "/bin/mcron")
#$@(if log?
- #~("--log" "--log-format" #$log-format)
- #~())
+ `("--log" "--log-format" ,log-format
+ ,@(if (maybe-value-set? date-format)
+ (list "--date-format"
+ date-format)
+ '()))
+ '())
#$@files)
;; Disable auto-compilation of the job files and
@@ -172,7 +191,7 @@ (define (mcron-shepherd-services config)
(remove (cut string-prefix? "PATH=" <>)
(environ)))
- #:log-file "/var/log/mcron.log"))
+ #:log-file #$log-file))
(stop #~(make-kill-destructor))
(actions
(list (shepherd-schedule-action mcron files)))))))))
--
2.39.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable.
2023-03-13 19:30 ` [bug#62169] [PATCH 3/3] services: mcron: Add log-file and date-format fields Bruno Victal
@ 2023-03-21 18:45 ` Maxim Cournoyer
2023-03-21 18:51 ` bug#62169: " Maxim Cournoyer
0 siblings, 1 reply; 6+ messages in thread
From: Maxim Cournoyer @ 2023-03-21 18:45 UTC (permalink / raw)
To: Bruno Victal; +Cc: 62169
Hello,
Bruno Victal <mirai@makinata.eu> writes:
> * doc/guix.texi (Scheduled Job Execution): Document it.
> * gnu/services/mcron.scm (mcron-configuration)[log-file, date-format]: New field.
> (mcron-shepherd-services): Add log-file and date-format support.
> Use file-append instead of string-append.
I've installed the first 2 patches, but was wondering about the use case
for having a configurable log-file for mcron? It seems it'd be very
niche to me, so perhaps it's not necessary?
--
Thanks,
Maxim
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#62169: [PATCH 0/3] services: mcron: Make log-file and date-format configurable.
2023-03-21 18:45 ` [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable Maxim Cournoyer
@ 2023-03-21 18:51 ` Maxim Cournoyer
0 siblings, 0 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2023-03-21 18:51 UTC (permalink / raw)
To: Bruno Victal; +Cc: 62169-done
Hello,
Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
> Hello,
>
> Bruno Victal <mirai@makinata.eu> writes:
>
>> * doc/guix.texi (Scheduled Job Execution): Document it.
>> * gnu/services/mcron.scm (mcron-configuration)[log-file, date-format]: New field.
>> (mcron-shepherd-services): Add log-file and date-format support.
>> Use file-append instead of string-append.
>
> I've installed the first 2 patches, but was wondering about the use case
> for having a configurable log-file for mcron? It seems it'd be very
> niche to me, so perhaps it's not necessary?
Thanks for confirming the use case on #guix (see:
https://logs.guix.gnu.org/guix/2023-03-21.log#194137).
Installed that one as well.
--
Thanks,
Maxim
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-03-21 18:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-13 19:02 [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 1/3] services: mcron: Restyle mcron-configuration Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 2/3] services: mcron: Use match-record Bruno Victal
2023-03-13 19:30 ` [bug#62169] [PATCH 3/3] services: mcron: Add log-file and date-format fields Bruno Victal
2023-03-21 18:45 ` [bug#62169] [PATCH 0/3] services: mcron: Make log-file and date-format configurable Maxim Cournoyer
2023-03-21 18:51 ` bug#62169: " Maxim Cournoyer
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.