unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: add msmtp
@ 2014-12-07  0:46 Tomáš Čech
  2014-12-07  1:13 ` David Thompson
  0 siblings, 1 reply; 5+ messages in thread
From: Tomáš Čech @ 2014-12-07  0:46 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/msmtp.scm: New file.
---
 gnu/packages/msmtp.scm | 46 ++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 46 insertions(+)
 create mode 100644 gnu/packages/msmtp.scm

diff --git a/gnu/packages/msmtp.scm b/gnu/packages/msmtp.scm
new file mode 100644
index 0000000..0cf01d4
--- /dev/null
+++ b/gnu/packages/msmtp.scm
@@ -0,0 +1,46 @@
+(define-module (gnu packages msmtp)
+  #:use-module (gnu packages)
+  #:use-module (gnu packages compression)
+  #:use-module (gnu packages libidn)
+  #:use-module (gnu packages gnutls)
+  #:use-module (gnu packages gsasl)
+  #:use-module (gnu packages pkg-config)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix build-system gnu)
+  #:use-module ((guix licenses) #:prefix license:)
+  )
+
+(define-public msmtp
+  (package
+   (name "msmtp")
+   (version "1.4.32")
+   (source
+    (origin
+     (method url-fetch)
+     (uri (string-append "http://downloads.sourceforge.net/project/msmtp/msmtp/"
+			 version "/msmtp-" version ".tar.bz2"))
+     (sha256 (base32
+	      "122z38pv4q03w3mbnhrhg4w85a51258sfdg2ips0b6cgwz3wbw1b"))))
+   (build-system gnu-build-system)
+   (inputs
+    `(("libidn" ,libidn)
+      ("gnutls" ,gnutls)
+      ("zlib" ,zlib)
+      ("gsasl" ,gsasl)))
+   (native-inputs
+    `(("pkg-config" ,pkg-config)))
+   (home-page "http://msmtp.sourceforge.net/")
+   (arguments
+    `(#:configure-flags (list "--with-libgsasl"
+			      "--with-libidn"
+			      "--with-ssl=gnutls")))
+   (synopsis
+    "SMTP client")
+   (description
+    "msmtp is an SMTP client.  In the default mode, it transmits a mail to
+an SMTP server (for example at a free mail provider) which takes care of further
+delivery.  To use this program with your mail user agent (MUA), create
+a configuration file with your mail account(s) and tell your MUA to call
+msmtp instead of /usr/sbin/sendmail.")
+   (license license:gpl3+)))
-- 
2.1.2

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

* Re: [PATCH] gnu: add msmtp
  2014-12-07  0:46 [PATCH] gnu: add msmtp Tomáš Čech
@ 2014-12-07  1:13 ` David Thompson
  2014-12-07  9:12   ` Tomáš Čech
  0 siblings, 1 reply; 5+ messages in thread
From: David Thompson @ 2014-12-07  1:13 UTC (permalink / raw)
  To: Tomáš Čech, guix-devel

Hey there!  Thanks for the patch!

Tomáš Čech <sleep_walker@suse.cz> writes:

> * gnu/packages/msmtp.scm: New file.
> ---
>  gnu/packages/msmtp.scm | 46 ++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 46 insertions(+)
>  create mode 100644 gnu/packages/msmtp.scm
>
> diff --git a/gnu/packages/msmtp.scm b/gnu/packages/msmtp.scm
> new file mode 100644
> index 0000000..0cf01d4
> --- /dev/null
> +++ b/gnu/packages/msmtp.scm
> @@ -0,0 +1,46 @@
> +(define-module (gnu packages msmtp)
> +  #:use-module (gnu packages)
> +  #:use-module (gnu packages compression)
> +  #:use-module (gnu packages libidn)
> +  #:use-module (gnu packages gnutls)
> +  #:use-module (gnu packages gsasl)
> +  #:use-module (gnu packages pkg-config)
> +  #:use-module (guix packages)
> +  #:use-module (guix download)
> +  #:use-module (guix build-system gnu)
> +  #:use-module ((guix licenses) #:prefix license:)
> +  )
> +
> +(define-public msmtp
> +  (package
> +   (name "msmtp")
> +   (version "1.4.32")
> +   (source
> +    (origin
> +     (method url-fetch)
> +     (uri (string-append "http://downloads.sourceforge.net/project/msmtp/msmtp/"
> +			 version "/msmtp-" version ".tar.bz2"))

Align version with the opening quote on the previous line.

> +     (sha256 (base32
> +	      "122z38pv4q03w3mbnhrhg4w85a51258sfdg2ips0b6cgwz3wbw1b"))))
> +   (build-system gnu-build-system)
> +   (inputs
> +    `(("libidn" ,libidn)
> +      ("gnutls" ,gnutls)
> +      ("zlib" ,zlib)
> +      ("gsasl" ,gsasl)))
> +   (native-inputs
> +    `(("pkg-config" ,pkg-config)))
> +   (home-page "http://msmtp.sourceforge.net/")
> +   (arguments
> +    `(#:configure-flags (list "--with-libgsasl"
> +			      "--with-libidn"
> +			      "--with-ssl=gnutls")))

Align "--with-libidn" and "--with-ssl=gnutls" with "--with-libgsasl".

> +   (synopsis
> +    "SMTP client")

Expand a bit.

> +   (description
> +    "msmtp is an SMTP client.  In the default mode, it transmits a mail to
> +an SMTP server (for example at a free mail provider) which takes care of further
> +delivery.  To use this program with your mail user agent (MUA), create
> +a configuration file with your mail account(s) and tell your MUA to call
> +msmtp instead of /usr/sbin/sendmail.")

The part about "/usr/sbin/sendmail" isn't relevant to Guix.

> +   (license license:gpl3+)))
> -- 
> 2.1.2

You forgot to add this new file to GNU_SYSTEM_MODULES in gnu-system.am,
but I think this package should go in gnu/packages/mail.scm instead of a
standalone module.  Could you move it?

Thanks!

-- 
David Thompson
Web Developer - Free Software Foundation - http://fsf.org
GPG Key: 0FF1D807
Support the FSF: https://fsf.org/donate

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

* Re: [PATCH] gnu: add msmtp
  2014-12-07  1:13 ` David Thompson
@ 2014-12-07  9:12   ` Tomáš Čech
  2014-12-07  9:12     ` Tomáš Čech
  0 siblings, 1 reply; 5+ messages in thread
From: Tomáš Čech @ 2014-12-07  9:12 UTC (permalink / raw)
  To: guix-devel

Thanks for kind review. Indentation issues weren't visible in emacs so I guess I
need to tune the configuration a bit. Now it looks in patch correctly.

changes since last attempt:
- instead of new module add into (gnu packages mail)
- fix overall indentation
- improve synopsis
- reduce description

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

* [PATCH] gnu: add msmtp
  2014-12-07  9:12   ` Tomáš Čech
@ 2014-12-07  9:12     ` Tomáš Čech
  2014-12-09 12:17       ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: Tomáš Čech @ 2014-12-07  9:12 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/mail.scm (msmtp): New variable.
---
 gnu/packages/mail.scm | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index 752cb51..6051dab 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -36,9 +36,11 @@
   #:use-module (gnu packages gnome)
   #:use-module (gnu packages gnupg)
   #:use-module (gnu packages gnutls)
+  #:use-module (gnu packages gsasl)
   #:use-module (gnu packages gtk)
   #:use-module (gnu packages guile)
   #:use-module (gnu packages libcanberra)
+  #:use-module (gnu packages libidn)
   #:use-module (gnu packages linux)
   #:use-module (gnu packages m4)
   #:use-module (gnu packages databases)
@@ -493,4 +495,37 @@ which can add many functionalities to the base client.")
     (home-page "http://www.claws-mail.org/")
     (license gpl3+))) ; most files are actually public domain or x11
 
+(define-public msmtp
+  (package
+    (name "msmtp")
+    (version "1.4.32")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "http://downloads.sourceforge.net/project/msmtp/msmtp/" version
+             "/msmtp-" version ".tar.bz2"))
+       (sha256 (base32
+                "122z38pv4q03w3mbnhrhg4w85a51258sfdg2ips0b6cgwz3wbw1b"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("libidn" ,libidn)
+       ("gnutls" ,gnutls)
+       ("zlib" ,zlib)
+       ("gsasl" ,gsasl)))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (home-page "http://msmtp.sourceforge.net/")
+    (arguments
+     `(#:configure-flags (list "--with-libgsasl"
+                               "--with-libidn"
+                               "--with-ssl=gnutls")))
+    (synopsis
+     "Simple and easy to use SMTP client with decent sendmail compatibility")
+    (description
+     "msmtp is an SMTP client.  In the default mode, it transmits a mail to
+an SMTP server (for example at a free mail provider) which takes care of further
+delivery.")
+    (license gpl3+)))
+
 ;;; mail.scm ends here
-- 
2.0.4

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

* Re: [PATCH] gnu: add msmtp
  2014-12-07  9:12     ` Tomáš Čech
@ 2014-12-09 12:17       ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2014-12-09 12:17 UTC (permalink / raw)
  To: Tomáš Čech; +Cc: guix-devel

Tomáš Čech <sleep_walker@suse.cz> skribis:

> * gnu/packages/mail.scm (msmtp): New variable.

I think this addresses the points that David raised, so I applied it.

Thanks!

Ludo’.

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

end of thread, other threads:[~2014-12-09 12:17 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-07  0:46 [PATCH] gnu: add msmtp Tomáš Čech
2014-12-07  1:13 ` David Thompson
2014-12-07  9:12   ` Tomáš Čech
2014-12-07  9:12     ` Tomáš Čech
2014-12-09 12:17       ` 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).