On Thu, Feb 10 2022, Utkarsh Singh wrote: > -------------------- Start of forwarded message -------------------- > From: Utkarsh Singh <utkarsh190...@gmail.com> > To: Tomi Ollila <tomi.oll...@iki.fi> > Subject: Re: [PATCH] emacs: Add more front ends for address completion > Date: Thu, 10 Feb 2022 08:58:49 +0530 > > Hello Tomi, > > On 2022-02-09, 23:59 +0200, Tomi Ollila <tomi.oll...@iki.fi> wrote: > >> On Tue, Feb 08 2022, Utkarsh Singh wrote: >> >>> Hello maintainers, >>> >>> Emacs Lisp Package Archive (ELPA) now includes a package called 'corfu', >>> according to its documentation: >>> >>> Corfu enhances the default completion in region function with a >>> completion overlay. The current candidates are shown in a popup >>> below or above the point. Corfu is the minimalistic >>> ~completion-in-region~ counterpart of the >>> [[https://github.com/minad/vertico][Vertico]] minibuffer UI. >>> >>> Hence, this patch tries to add support for `completion-in-region' in >>> `notmuch-address-expand-name'. By default, this behaviour is turned off >>> so that existing users can enjoy existing completion techniques. >> >> The current "default" (i.e. w/o any notmuch emacs mua configuration) is to >> use completing-read to do the completion. If "company" is available, then >> company is used by default (w/ all address harvesting and so on...). >> >> This is "messy" enough ;( (i.e the notmuch-address-selection-function >> is called if company mode is not available or notmuch-address-command >> is a string instead of 'internal or 'as-is (or whatnot, too tired to do >> deep investigation there ;/) >> >> This change, contributes even more "complexity" there. To keep the >> complexity to the same level would be adding more >> notmuch-address-selection-functions and have the defcustom there list >> the options (also probably the name of notmuch-address-selection-function >> would need to be changed to notmuch-fallback-address-selection-function >> ;/) >> > > I think, in general, you're right about the complexity and we should try > minimize it. But this patch was originally derived from a bug I was > experiencing with `(global-corfu-mode 1)' in `notmuch-message-mode'.
Is global-corfu-mode changing how completing-read works ? > Here are the steps to reproduce the bug: > > 1. [Install](https://notmuchmail.org/#index7h2) `notmuch`. > 2. In emacs -Q session, evaluate the following When doing in notmuch repo, one can execute ./devel/try-emacs-mua -Q ... which probably uses the same package database (under $HOME) as any other emacs invocation would do -- what is the way to change the location of that in order to have chance to have "empty" set of externally installed packages, and messing with it would not touch the "default" packages... (so that I can try the steps below) Tomi > > ```elisp > (progn > (add-to-list 'load-path "/usr/share/emacs/site-lisp") ; Notmuch Emacs > interface > (package-initialize) > (package-install 'corfu) > (package-install 'vertico) > > (require 'notmuch) > (require 'corfu) > (require 'vertico) > > (vertico-mode 1) > (corfu-global-mode 1) > (notmuch-mua-new-mail)) > ``` > > 3. Press `TAB` to generate address completion, then press `RET` to > select address of your choice. After selection, you will notice that > `notmuch-address-expand-name` fails to insert desired address, that is, > you will get the following: > > ``` > To: > ``` > > Note: Check the trailing spaces. > >> All this said, I think this is not simple to solve, as this otherwise fine >> change would indicate :/ > > No problem, I'm willing to help the maintainers on this matter. > > -- > Utkarsh Singh > https://utkarshsingh.xyz/ > -------------------- End of forwarded message -------------------- > > -- > Utkarsh Singh > https://utkarshsingh.xyz/ > _______________________________________________ > notmuch mailing list -- firstname.lastname@example.org > To unsubscribe send an email to notmuch-le...@notmuchmail.org _______________________________________________ notmuch mailing list -- email@example.com To unsubscribe send an email to notmuch-le...@notmuchmail.org