guix-patches
[Top][All Lists]
Advanced

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

[bug#57021] [PATCH] gnu: packages: Add nmail.


From: Mathieu Othacehe
Subject: [bug#57021] [PATCH] gnu: packages: Add nmail.
Date: Fri, 12 Aug 2022 10:53:33 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux)

Hello,

> * gnu/packages/nmail.scm: Reorder module imports. Remove
>   duplicate imports.
> (nmail): New variable. Fix style.

Thanks for this first contribution. As you are adding a new package, the
preferred way to proceed is to amend your first patch with those changes
and submit a new revision (v2).

The module re-ordering stuff should also be part of a new patch. In
short, we would expect something like:

* gnu: Reorder mail module imports.
* gnu: nmail: New variable.

Do you think you could send those two patches in a v3? Don't hesitate to
ask for guidance here or on IRC if I'm unclear.

Mathieu





reply via email to

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