From: Tanguy LE CARROUR <tanguy@bioneland.org>
To: Efraim Flashner <efraim@flashner.co.il>
Cc: 44628-done@debbugs.gnu.org
Subject: [bug#44628] [PATCH 2/2] gnu: Add astroid.
Date: Wed, 18 Nov 2020 09:23:19 +0100 [thread overview]
Message-ID: <1605687026.i9n46bask7.astroid@melmoth.none> (raw)
In-Reply-To: <20201117193755.GX987@E5400>
Hi Efraim,
Thanks for the time and efforts you put into reviewing my patch!
I really appreciate it!
Excerpts from Efraim Flashner's message of November 17, 2020 8:37 pm:
> 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
This has been merged, but there's been no release yet.
Is it a case where I could do a "version release", with a commit number,
like for instance what is done for `emacs-atom-one-dark-theme`, to make
the latest code available? If yes, should it be a different variable name,
like `astroid-latest`?
>> + (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.
Shame on me!! :-(
I added the "echo" to save some time, and forgot to remove it! Next
time, I'll put a comment alongside it!
But I thought `ninja test` was successfully running all the tests! Strange.
>
>> + (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.
The problem is that… the software is built, but does not work!
I had to put back `gsettings-desktop-schemas` and `adwaita-icon-theme`
back into the propagated inputs to make it work.
Without the first one, it does not event start. Without the second, one
cannot switch to the thread view.
I'll submit a new patch for that.
>> + (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.
Thaaaaaanks again for your time and effort!
--
Tanguy
next prev parent reply other threads:[~2020-11-18 8:24 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 ` bug#44628: " Efraim Flashner
2020-11-18 8:23 ` Tanguy LE CARROUR [this message]
2020-11-18 8:46 ` [bug#44628] " 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=1605687026.i9n46bask7.astroid@melmoth.none \
--to=tanguy@bioneland.org \
--cc=44628-done@debbugs.gnu.org \
--cc=efraim@flashner.co.il \
/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).