[PATCH] gnu: Add emacs-consult-notmuch.

Done

Details

4 participants

Andrew Tropin
Kyle Meyer
Nicolas Goaziou
zimoun

Owner: unassigned

Submitted by: Andrew Tropin

Severity: normal

Andrew Tropin wrote on 7 Jul 2021 15:59

Recipients:(address . guix-patches@gnu.org)

Message-ID:87tul6w7g2.fsf@trop.in

From eba48a1058bc4ae7486e74ac30cf851a78ce7272 Mon Sep 17 00:00:00 2001

* gnu/packages/emacs-xyz.scm (emacs-consult-notmuch): New variable.

---

gnu/packages/emacs-xyz.scm | 30 ++++++++++++++++++++++++++++++

1 file changed, 30 insertions(+)

Toggle diff (43 lines)diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 9591a4e6e7..788a25a8e8 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -8084,6 +8084,36 @@ Emacs completion function completing-read, which allows quickly selecting from a
 list of candidates.")
     (license license:gpl3+)))
 
+(define-public emacs-consult-notmuch
+  (package
+    (name "emacs-consult-notmuch")
+    (version "0.3")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://codeberg.org/jao/consult-notmuch.git")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "19flyh3v1xm2zswzjkvjbijvpbq5r8isafza4fd0yicvqbjyklhx"))))
+    (build-system emacs-build-system)
+    (inputs
+     `(("emacs-consult" ,emacs-consult)
+       ("notmuch" ,notmuch)))
+    (license license:gpl3+)
+    (home-page "https://codeberg.org/jao/consult-notmuch")
+    (synopsis "Search and preview notmuch emails using consult")
+    (description "\
+This package provides two commands using consult to query notmuch
+emails and present results either as single emails @code{consult-notmuch}
+or full trees @code{consult-notmuch-tree}.
+
+The package also defines a narrowing source for @code{consult-buffer}, which
+can be activated with @code{(add-to-list 'consult-buffer-sources
+'consult-notmuch-buffer-source)}.")))
+
 (define-public emacs-marginalia
   (package
     (name "emacs-marginalia")
-- 
2.32.0

Andrew Tropin wrote on 7 Jul 2021 16:12

[PATCH v2] gnu: Add emacs-consult-notmuch.

Recipients:(address . guix-patches@gnu.org)

Message-ID:87r1gaw6u8.fsf@trop.in

v2 uses propagated-inputs instead of inputs, because using

consult-notmuch without consult and notmuch doesn't make much sense.

From 7ba18d68face76d8858ad67a6113f5773fa45cec Mon Sep 17 00:00:00 2001

* gnu/packages/emacs-xyz.scm (emacs-consult-notmuch): New variable.

---

gnu/packages/emacs-xyz.scm | 30 ++++++++++++++++++++++++++++++

1 file changed, 30 insertions(+)

Toggle diff (43 lines)diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 9591a4e6e7..492572322c 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -8084,6 +8084,36 @@ Emacs completion function completing-read, which allows quickly selecting from a
 list of candidates.")
     (license license:gpl3+)))
 
+(define-public emacs-consult-notmuch
+  (package
+    (name "emacs-consult-notmuch")
+    (version "0.3")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://codeberg.org/jao/consult-notmuch.git")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "19flyh3v1xm2zswzjkvjbijvpbq5r8isafza4fd0yicvqbjyklhx"))))
+    (build-system emacs-build-system)
+    (propagated-inputs
+     `(("emacs-consult" ,emacs-consult)
+       ("notmuch" ,notmuch)))
+    (license license:gpl3+)
+    (home-page "https://codeberg.org/jao/consult-notmuch")
+    (synopsis "Search and preview notmuch emails using consult")
+    (description "\
+This package provides two commands using consult to query notmuch
+emails and present results either as single emails @code{consult-notmuch}
+or full trees @code{consult-notmuch-tree}.
+
+The package also defines a narrowing source for @code{consult-buffer}, which
+can be activated with @code{(add-to-list 'consult-buffer-sources
+'consult-notmuch-buffer-source)}.")))
+
 (define-public emacs-marginalia
   (package
     (name "emacs-marginalia")
-- 
2.32.0

Nicolas Goaziou wrote on 7 Jul 2021 16:18

Recipients:(name . Andrew Tropin)(address . andrew@trop.in)(address . 49457@debbugs.gnu.org)

Message-ID:87czrutdf9.fsf@nicolasgoaziou.fr

Hello,

Andrew Tropin <andrew@trop.in> writes:

Toggle quote (3 lines)

> v2 uses propagated-inputs instead of inputs, because using

> consult-notmuch without consult and notmuch doesn't make much sense.

Thank you. However, notmuch should be an input, there's no reason to
propagate it into user's profile.

Regards,
-- 
Nicolas Goaziou

Andrew Tropin wrote on 7 Jul 2021 16:33

Recipients:(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)(address . 49457@debbugs.gnu.org)

Message-ID:87eecaqjlu.fsf@trop.in

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

Toggle quote (10 lines)> Hello,
>
> Andrew Tropin <andrew@trop.in> writes:
>
>> v2 uses propagated-inputs instead of inputs, because using
>> consult-notmuch without consult and notmuch doesn't make much sense.
>
> Thank you. However, notmuch should be an input, there's no reason to
> propagate it into user's profile.

There is a (require 'notmuch) in consult-notmuch code, so we need
notmuch.el on the load-path, sadly, but we don't have a separate
emacs-notmuch package for it and therefore we have to propagate the
whole notmuch package here :(

Nicolas Goaziou wrote on 8 Jul 2021 09:40

Recipients:(name . Andrew Tropin)(address . andrew@trop.in)(address . 49457-done@debbugs.gnu.org)

Message-ID:87czrtqmn6.fsf@nicolasgoaziou.fr

Hello,

Andrew Tropin <andrew@trop.in> writes:

Toggle quote (3 lines)

> v2 uses propagated-inputs instead of inputs, because using

> consult-notmuch without consult and notmuch doesn't make much sense.

I applied this patch with the following changes:

Toggle quote (3 lines)

> + (uri (git-reference

> + (url "https://codeberg.org/jao/consult-notmuch.git")

I removed the ".git" suffix.

Toggle quote (4 lines)

> + (license license:gpl3+)

> + (home-page "https://codeberg.org/jao/consult-notmuch")

> + (synopsis "Search and preview notmuch emails using consult")

I properly capitalized synopsis (Notmuch, Consult).

Toggle quote (9 lines)

> + (description "\

> +This package provides two commands using consult to query notmuch

> +emails and present results either as single emails @code{consult-notmuch}

> +or full trees @code{consult-notmuch-tree}.

> +

> +The package also defines a narrowing source for @code{consult-buffer}, which

> +can be activated with @code{(add-to-list 'consult-buffer-sources

> +'consult-notmuch-buffer-source)}.")))

I shortened the description as it looked like an introduction part of

the manual instead.

I also reordered licence, home-page, synopsis and description parts.

Thank you!

Regards,

Nicolas Goaziou

Closed

Nicolas Goaziou wrote on 8 Jul 2021 09:41

Recipients:(name . Andrew Tropin)(address . andrew@trop.in)(address . 49457@debbugs.gnu.org)

Message-ID:878s2hqmlp.fsf@nicolasgoaziou.fr

Hello,

Andrew Tropin <andrew@trop.in> writes:

Toggle quote (10 lines)

> Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

>> Thank you. However, notmuch should be an input, there's no reason to

>> propagate it into user's profile.

> There is a (require 'notmuch) in consult-notmuch code, so we need

> notmuch.el on the load-path, sadly, but we don't have a separate

> emacs-notmuch package for it and therefore we have to propagate the

> whole notmuch package here :(

Duh! You're right, of course.

Applied, as detailed in another message.

Regards,

Nicolas Goaziou

zimoun wrote on 8 Jul 2021 11:13

Recipients:(address . 49457@debbugs.gnu.org)

Message-ID:86bl7d2mny.fsf@gmail.com

Hi,

(Sorry for this naive question.)

On Thu, 08 Jul 2021 at 09:41, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:

Toggle quote (13 lines)> Andrew Tropin <andrew@trop.in> writes:
>> Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:
>
>>> Thank you. However, notmuch should be an input, there's no reason to
>>> propagate it into user's profile.
>>
>> There is a (require 'notmuch) in consult-notmuch code, so we need
>> notmuch.el on the load-path, sadly, but we don't have a separate
>> emacs-notmuch package for it and therefore we have to propagate the
>> whole notmuch package here :(
>
> Duh! You're right, of course.

For my personal understanding because I am not familiar with Emacs
packages and package propagation.  The package ’emacs-magit’ has ’git’
as inputs and AFAIK it does not work without Git installed in the
profile.  Why is it different?

Cheers,
simon

Nicolas Goaziou wrote on 8 Jul 2021 11:41

Recipients:(name . zimoun)(address . zimon.toutoune@gmail.com)

Message-ID:87h7h5p2gt.fsf@nicolasgoaziou.fr

Hello,

zimoun <zimon.toutoune@gmail.com> writes:

Toggle quote (5 lines)

> For my personal understanding because I am not familiar with Emacs

> packages and package propagation. The package ’emacs-magit’ has ’git’

> as inputs and AFAIK it does not work without Git installed in the

> profile. Why is it different?

It is different because Notmuch is both an external executable and an

Emacs library. Git, OTOH, does not ship with an Elisp library.

However, I think it is a bug if Emacs Magit does not work with git in

profile. We should modify the source so both magit-git-executable and

magit-gitk-executable point to those from inputs.

I suggest to open a new bug report about it, WDYT?

Regards,

Nicolas Goaziou

Andrew Tropin wrote on 8 Jul 2021 12:14

Recipients:(address . 49457@debbugs.gnu.org)

Message-ID:871r893yet.fsf@trop.in

zimoun <zimon.toutoune@gmail.com> writes:

Toggle quote (23 lines)> Hi,
>
> (Sorry for this naive question.)
>
> On Thu, 08 Jul 2021 at 09:41, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
>> Andrew Tropin <andrew@trop.in> writes:
>>> Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:
>>
>>>> Thank you. However, notmuch should be an input, there's no reason to
>>>> propagate it into user's profile.
>>>
>>> There is a (require 'notmuch) in consult-notmuch code, so we need
>>> notmuch.el on the load-path, sadly, but we don't have a separate
>>> emacs-notmuch package for it and therefore we have to propagate the
>>> whole notmuch package here :(
>>
>> Duh! You're right, of course.
>
> For my personal understanding because I am not familiar with Emacs
> packages and package propagation.  The package ’emacs-magit’ has ’git’
> as inputs and AFAIK it does not work without Git installed in the
> profile.  Why is it different?

The magit package itself will be loaded correctly, because its emacs
package requirements are satisfied and even some functions will work,
for example magit-dispatch will show a transient interface.  Other
functions that rely on git binary will throw an error and there are two
solutions for that: make a git a propagated input or set
`magit-git-executable` to a path to the git binary inside store, which
will make magit work with git as a regular input.

notmuch in contrast to git provides not only notmuch binary, but also
notmuch.el.  consult-notmuch requires only notmuch.el, which have to be
propagated and doesn't depend on notmuch binary, but because notmuch.el
is a part of notmuch package the binary will be propagated too.  The
possible solution here is to split notmuch into two packages: notmuch and
emacs-notmuch, so we will be able to set `notmuch-command` variable for
notmuch.el and have notmuch as an input, but not installed in the profile.

Andrew Tropin wrote on 8 Jul 2021 12:22

Recipients:(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)(address . 49457-done@debbugs.gnu.org)

Message-ID:87y2ah2jhw.fsf@trop.in

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

Toggle quote (32 lines)> Hello,
>
> Andrew Tropin <andrew@trop.in> writes:
>
>> v2 uses propagated-inputs instead of inputs, because using
>> consult-notmuch without consult and notmuch doesn't make much sense.
>
> I applied this patch with the following changes:
>
>> +       (uri (git-reference
>> +             (url "https://codeberg.org/jao/consult-notmuch.git")
>
> I removed the ".git" suffix.
>
>> +    (license license:gpl3+)
>> +    (home-page "https://codeberg.org/jao/consult-notmuch")
>> +    (synopsis "Search and preview notmuch emails using consult")
>
> I properly capitalized synopsis (Notmuch, Consult).
>
>> +    (description "\
>> +This package provides two commands using consult to query notmuch
>> +emails and present results either as single emails @code{consult-notmuch}
>> +or full trees @code{consult-notmuch-tree}.
>> +
>> +The package also defines a narrowing source for @code{consult-buffer}, which
>> +can be activated with @code{(add-to-list 'consult-buffer-sources
>> +'consult-notmuch-buffer-source)}.")))
>
> I shortened the description as it looked like an introduction part of
> the manual instead.

It was complete) but shorter version is also fine.

Toggle quote (5 lines)

> I also reordered licence, home-page, synopsis and description parts.

> Thank you!

Thank you!)

Closed

Nicolas Goaziou wrote on 8 Jul 2021 12:32

Recipients:(name . Andrew Tropin)(address . andrew@trop.in)(address . 49457-done@debbugs.gnu.org)

Message-ID:87v95lnljq.fsf@nicolasgoaziou.fr

Andrew Tropin <andrew@trop.in> writes:

Toggle quote (7 lines)

> Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

>> I shortened the description as it looked like an introduction part of

>> the manual instead.

> It was complete) but shorter version is also fine.

Yes, it was complete.

What I meant is the following: when I look at a description, I try to

see what kind of question it answers. If it answers the question: "Do

I want to install this package?", that's good. But if it answers: "How

can I start using this?", then the description belongs to a manual

instead.

For example,

which can be activated with @code{(add-to-list

'consult-buffer-sources 'consult-notmuch-buffer-source)}.

clearly belongs to the second question. So does the name of functions,

in general.

Closed

Kyle Meyer wrote on 8 Jul 2021 13:28

Recipients:(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)

Message-ID:87mtqx82or.fsf@kyleam.com

Nicolas Goaziou writes:

Toggle quote (6 lines)

> However, I think it is a bug if Emacs Magit does not work with git in

> profile. We should modify the source so both magit-git-executable and

> magit-gitk-executable point to those from inputs.

> I suggest to open a new bug report about it, WDYT?

Rewriting magit-git-executable to an absolute file name will break Magit

for those that use it over TRAMP.

Nicolas Goaziou wrote on 8 Jul 2021 13:38

Recipients:(name . Kyle Meyer)(address . kyle@kyleam.com)

Message-ID:87k0m1nihl.fsf@nicolasgoaziou.fr

Hello,

Kyle Meyer <kyle@kyleam.com> writes:

Toggle quote (11 lines)> Nicolas Goaziou writes:
>
>> However, I think it is a bug if Emacs Magit does not work with git in
>> profile. We should modify the source so both magit-git-executable and
>> magit-gitk-executable point to those from inputs.
>>
>> I suggest to open a new bug report about it, WDYT?
>
> Rewriting magit-git-executable to an absolute file name will break Magit
> for those that use it over TRAMP.

Point taken. However, such users can fix it by modifying the

`magit-git-executable' defcustom again, can't they?

So, the situation would arguably not be worse that what it is currently,

since it simply doesn't work for users relying on a local installation

of git, and those are probably more common.

Also, a third option: since we're not making use of git input at

runtime, why should it be an input at all? If it is needed at build

time, a native-input should suffice.

Regards,

Nicolas Goaziou

Kyle Meyer wrote on 9 Jul 2021 02:13

Recipients:(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)

Message-ID:87im1k8hv7.fsf@kyleam.com

Nicolas Goaziou writes:

Toggle quote (16 lines)> Kyle Meyer <kyle@kyleam.com> writes:
>
>> Nicolas Goaziou writes:
>>
>>> However, I think it is a bug if Emacs Magit does not work with git in
>>> profile. We should modify the source so both magit-git-executable and
>>> magit-gitk-executable point to those from inputs.
>>>
>>> I suggest to open a new bug report about it, WDYT?
>>
>> Rewriting magit-git-executable to an absolute file name will break Magit
>> for those that use it over TRAMP.
>
> Point taken. However, such users can fix it by modifying the
> `magit-git-executable' defcustom again, can't they?

Sure.

Given the number of "oops, some change didn't consider tramp

compatibility" bugs that crop up in Magit, my guess is that Magit is

widely used over TRAMP. So, potentially many users are affected, but I

agree it's an easy fix once they spend time to debug what the issue is.

(That collectively could amount to a lot of wasted time, so in my eyes

it depends on what the practical problem being fixed is.)

Toggle quote (4 lines)

> So, the situation would arguably not be worse that what it is currently,

> since it simply doesn't work for users relying on a local installation

> of git, and those are probably more common.

Hmm, I'm not clear on what the current problem is; a user will typically
have "git" on their path one way or the other.  Given Magit's heavily
used, I would have expected to see many Guix bug reports about about it
if it were a problem in practice.

But 1) perhaps I'm missing something or 2) perhaps for philosophical
reasons or to promote best practices rewriting magit-git-executable is
worth the breakage.  My drive-by comment was just to mention the
possible breakage [*].  (I don't have a personal stake in what's done
because I don't install Magit from Guix.)

Toggle quote (4 lines)

> Also, a third option: since we're not making use of git input at

> runtime, why should it be an input at all? If it is needed at build

> time, a native-input should suffice.

Yeah, that sounds right (though I haven't tested it).

[*] And it might not break things for much longer. On Magit's end,

Jonas recently floated the idea of not using magit-git-executable

when using TRAMP.

https://github.com/magit/magit/issues/4433#issuecomment-875964528

Your comment

This issue is archived.

To comment on this conversation send an email to 49457@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it

mumi current 49457

Then, you may apply the latest patchset in this issue (with sign off)

mumi am -- -s

Or, compose a reply to this issue

mumi compose

Or, send patches to this issue

mumi send-email *.patch

is:open	open issues
is:done	closed issues
submitter:<who>	search issue submitter
author:<who>	search by message author
date:yesterday..now	search by issue date
mdate:3m..2d	search by message date