unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: Tanguy Le Carrour <tanguy@bioneland.org>
Cc: 44628-done@debbugs.gnu.org
Subject: bug#44628: [PATCH 2/2] gnu: Add astroid.
Date: Tue, 17 Nov 2020 21:37:55 +0200	[thread overview]
Message-ID: <20201117193755.GX987@E5400> (raw)
In-Reply-To: <20201114092611.6225-2-tanguy@bioneland.org>

[-- Attachment #1: Type: text/plain, Size: 5110 bytes --]

This patch I went down a rabbit hole touching it up.

On Sat, Nov 14, 2020 at 10:26:11AM +0100, Tanguy Le Carrour wrote:
> * gnu/packages/mail.scm (astroid): New variable.
> ---
>  gnu/packages/mail.scm | 60 +++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 60 insertions(+)
> 
> diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
> index fcd593c356..d2af1ce10c 100644
> --- a/gnu/packages/mail.scm
> +++ b/gnu/packages/mail.scm
> @@ -63,6 +63,7 @@
>    #:use-module (gnu packages base)
>    #:use-module (gnu packages bash)
>    #:use-module (gnu packages bison)
> +  #:use-module (gnu packages boost)
>    #:use-module (gnu packages calendar)
>    #:use-module (gnu packages check)
>    #:use-module (gnu packages compression)
> @@ -107,6 +108,7 @@
>    #:use-module (gnu packages man)
>    #:use-module (gnu packages ncurses)
>    #:use-module (gnu packages networking)
> +  #:use-module (gnu packages ninja)
>    #:use-module (gnu packages openldap)
>    #:use-module (gnu packages onc-rpc)
>    #:use-module (gnu packages pcre)
> @@ -114,6 +116,7 @@
>    #:use-module (gnu packages perl-check)
>    #:use-module (gnu packages perl-web)
>    #:use-module (gnu packages pkg-config)
> +  #:use-module (gnu packages protobuf)
>    #:use-module (gnu packages python)
>    #:use-module (gnu packages python-check)
>    #:use-module (gnu packages python-crypto)
> @@ -703,6 +706,63 @@ mailpack.  What can alterMIME do?
>      (license (list (license:non-copyleft "file://LICENSE")
>                     license:bsd-3))))
>  
> +(define-public astroid
> +  (package
> +    (name "astroid")
> +    (version "0.15")
> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/astroidmail/astroid")
> +             (commit (string-append "v" version))))
> +       (file-name (git-file-name name version))
> +       (sha256
> +        (base32 "11cxbva9ni98gii59xmbxh4c6idcg3mg0pgdsp1c3j0yg7ix0lj3"))))

I added a snippet for the test suite to adjust for gmime@3.2.3+
https://github.com/astroidmail/astroid/pull/685

> +    (build-system cmake-build-system)
> +    (arguments
> +     `(#:configure-flags (list "-GNinja")
> +       #:phases
> +       (modify-phases %standard-phases
> +         (replace 'build
> +           (lambda _
> +             (invoke "ninja" "-j" (number->string (parallel-job-count)))))
> +         (replace 'check
> +           (lambda _
> +             (invoke "echo" "ninja" "test")))

This just echos "ninja test" but doesn't actually run anything. I
changed it to run (invoke "ctest" ".") Then I added a phase to start the
xorg display for the tests and another phase to work around not getting
the test suite to recognize the plugin modules.

> +         (replace 'install
> +           (lambda _
> +             (invoke "ninja" "install"))))))

The native-inputs should have a bunch moved to regular inputs. It was
also missing gnupg and w3m, needed for the test suite. I removed
adwaita-icon-theme from propagated inputs, we try not to propagate icon
sets. I also sorted them alphabetically.

> +    (native-inputs
> +     `(("gtkmm" ,gtkmm)
> +       ("gmime" ,gmime)
> +       ("gobject-introspection" ,gobject-introspection)
> +       ("webkitgtk" ,webkitgtk)
> +       ("pkg-config" ,pkg-config)
> +       ("ninja" ,ninja)
> +       ("ronn" ,ronn)
> +       ("libsass" ,libsass)
> +       ("libpeas" ,libpeas)
> +       ("boost" ,boost)
> +       ("glib-networking" ,glib-networking)
> +       ("protobuf" ,protobuf)))
> +    (inputs
> +     `(("python" ,python)
> +       ("notmuch" ,notmuch)))
> +    (propagated-inputs
> +     `(("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
> +       ("adwaita-icon-theme" ,adwaita-icon-theme)))
> +    (home-page "https://astroidmail.github.io/")
> +    (synopsis "GTK frontend to the notmuch mail system")
> +    (description
> +     "Astroid is a lightweight and fast Mail User Agent that provides
> +a graphical interface to searching, display and composing email,
> +organized in thread and tags.  Astroid uses the notmuch backend for blazingly
> +fast searches through tons of email.  Astroid searches, displays and
> +compose emails — and relies on other programs for fetching,
> +syncing and sending email. ")
> +    (license license:gpl3+)))

There's also a bit of borrowed code from geary and gmime, under the
lgpl2.1+, so I added that to the license list. It is true that the
entire work is under the gpl3+ but we try to be more specific if
possible.

> +
>  (define-public ripmime
>    ;; Upstream does not tag or otherwise provide any releases (only a version
>    ;; number in the source)
> -- 
> 2.29.2
> 

I didn't get a chance to test it since I don't use notmuch.

Patch pushed!

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2020-11-17 19:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-14  9:24 [bug#44628] [PATCH 0/2] gnu: Add astroid Tanguy Le Carrour
2020-11-14  9:26 ` [bug#44628] [PATCH 1/2] gnu: gmime: Add the "--enable-introspection" flag Tanguy Le Carrour
2020-11-14  9:26   ` [bug#44628] [PATCH 2/2] gnu: Add astroid Tanguy Le Carrour
2020-11-17 19:37     ` Efraim Flashner [this message]
2020-11-18  8:23       ` Tanguy LE CARROUR
2020-11-18  8:46         ` Tanguy LE CARROUR
2020-11-17 19:32   ` [bug#44628] [PATCH 1/2] gnu: gmime: Add the "--enable-introspection" flag Efraim Flashner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201117193755.GX987@E5400 \
    --to=efraim@flashner.co.il \
    --cc=44628-done@debbugs.gnu.org \
    --cc=tanguy@bioneland.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).