guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#62969] [PATCH] home: Add msmtp service.


From: Ludovic Courtès
Subject: [bug#62969] [PATCH] home: Add msmtp service.
Date: Sun, 30 Apr 2023 23:30:10 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)

Hi Tanguy! :-)

Tanguy Le Carrour <tanguy@bioneland.org> skribis:

> * gnu/home/services/mail.scm: New file.
> * gnu/local.mk (GNU_SYSTEM_MODULES): Add it.
> * doc/guix.texi (Mailing): New node.

I get:

--8<---------------cut here---------------start------------->8---
[ 94%] GUILEC   gnu/home/services/mail.go
gnu/home/services/mail.scm:52:0: warning: possibly unbound variable 
`serialize-string'
gnu/home/services/mail.scm:53:0: warning: possibly unbound variable 
`serialize-boolean'
gnu/home/services/mail.scm:54:0: warning: possibly unbound variable 
`serialize-integer'
--8<---------------cut here---------------end--------------->8---

I guess these procedures should be provided as well?

> +@node Mail Home Services
> +@subsection Mail Home Services
> +
> +@cindex msmtp
> +@uref{https://marlam.de/msmtp, MSMTP} is an SMTP client.

Could you give a bit more context, including @acronym for SMTP?  It
would also be nice to provide one commented example that readers might
copy/paste as a starting point.

You can take inspiration form the “Mcron Home Services”, “Secure Shell”,
or “Messaging Home Services” nodes, for example.

[...]

> +                (default-value (home-msmtp-configuration))
> +                (description "Configures msmtp.")))

Likewise, something like “Configure msmtp, a simple @acronym{SMTP, …}
client that can relay email to SMTP servers.”

Otherwise this looks great to me, thank you!

Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]