bug#45153: closed ([PATCH] gnu: rottlog: Fix mail reporting.)

[GNU bug Tracking System]

Your message dated Sun, 27 Dec 2020 16:07:50 +0100
with message-id <87ft3rthh5.fsf@gnu.org>
and subject line Re: [bug#45153] [PATCH v2] gnu: rottlog: Fix mail reporting.
has caused the debbugs.gnu.org bug report #45153,
regarding [PATCH] gnu: rottlog: Fix mail reporting.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
45153: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=45153
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] gnu: rottlog: Fix mail reporting. Date: Thu, 10 Dec 2020 04:41:29 +0100 User-agent: Evolution 3.34.2

Hello!

I was getting errors like this in my GNU Guix System installation:

# cat /var/log/mcron.log
/gnu/store/8q2i735qhdivn5pn87b6ay7z16xnf6py-rottlog-
0.72.2/sbin/rottlog: line 1596: sendmail: command not found
/gnu/store/8q2i735qhdivn5pn87b6ay7z16xnf6py-rottlog-
0.72.2/sbin/rottlog: line 1596: sendmail: command not found
/gnu/store/8q2i735qhdivn5pn87b6ay7z16xnf6py-rottlog-
0.72.2/sbin/rottlog: line 1596: sendmail: command not found
/gnu/store/8q2i735qhdivn5pn87b6ay7z16xnf6py-rottlog-
0.72.2/sbin/rottlog: line 1596: sendmail: command not found
/gnu/store/8q2i735qhdivn5pn87b6ay7z16xnf6py-rottlog-
0.72.2/sbin/rottlog: line 1596: sendmail: command not found

I am guessing this means sendmail needs to be added as propagated-
inputs to ensure it is available to rottlog at runtime.

Attached patch with these changes!

Thank you!

0001-gnu-rottlog-Fix-mail-reporting.patch
Description: Text Data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: [bug#45153] [PATCH v2] gnu: rottlog: Fix mail reporting.
>
>
>
>
> Date: 
>
> Sun, 27 Dec 2020 16:07:50 +0100
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)
>
>
>
>
> Hi,
>
> Leo Le Bouter <lle-bout@zaclys.net> skribis:
>
> >> I thought this was a typo and then realized that our ‘sendmail’
> >> package
> >> is this broken, due to an incorrect use of ‘DESTDIR’.
> >
> > Ah! Well I did not notice that as something wrong outright but now that
> > you say that yes I see the correct behavior :-)
> >
> >> The patch below is an attempt to fix that but some of the files are
> >> silently not getting installed.  Anyone has ideas here?
> >
> > Will take a look!
>
> Great.
>
> >> Anyhow, can we instead use ‘mail’ from ‘mailutils’ instead?  (I
> >> wrongfully mentioned Inetutils earlier, sorry for the confusion.)
> >
> > Is it going to be compatible? If so then I'll do it!
>
> Yes, I think so, again provided there’s a mail setup on the machine.
>
> I went ahead and made this change since it was a minor change compared
> to the patch you had sent earlier.
>
> Thanks!
>
> Ludo’.
>
>
> --- End Message ---