Re: Overriding mm-inline-override-types in notmuch-show

2022-08-11 Thread David Bremner
David Bremner  writes:

> Al Haji-Ali  writes:
>
>> The variable `mm-inline-override-types` is overridden in `notmuch-show` to 
>> "stop application/* parts from being displayed".
>>
>> This works well, however it means in other places the part *is* expanded. 
>> For example when `(notmuch-show-refresh-view)` is called (after any refresh) 
>> or even when `notmuch-show-reply-sender` is called.
>>
>> To see this behaviour make sure that `mm-inline-override-types` is nil and 
>> open a message with an `application/zip` part. The part is not expanded as 
>> expected.
>> Call `(notmuch-show-refresh-view)` and notice how the part is expanded (on 
>> my Emacs, I even get a warning: "These default coding systems were tried to 
>> encode the following problematic characters in the buffer ‘ *temp*-926949’").
>>
>> I solved this in my config by setting `mm-inline-override-types` to `(list 
>> "application/*")`, but I think notmuch could be more consistent.
>>
>
> Thanks for the report. I can duplicate the issue with
> notmuch-show-refresh-view. I believe with notmuch-show-reply-sender is
> fixed in git master (90a7c1af368a527700dcde9b0dcbd760afc7bd92).

The remaining issues should be fixed as of commit
bf64c48855f41833b57e0a770ad37f6a16a1193b
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


Re: Embed elisp.py from flycheck, use it in notmuch-emacs doc

2022-08-11 Thread David Bremner
Tomi Ollila  writes:

>
> This series is probably good; at least it does not break software
> (and if doc quality (or making of it) gets better then good :D)
>
> Tomi

Applied to master.

d
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


Re: [PATCH 3/3] doc: add some documentation about duplicate files.

2022-08-11 Thread David Bremner
David Bremner  writes:

> This is mainly motivated by the new emacs-UI features to select
> duplicates, but hopefully it will help demystify things for users of
> the CLI as well.

applied the remaining two patches to master (with a tiny adjustment of
wording for the man output in the second patch).

d
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


Re: [PATCH 2/2] emacs/show: restrict inlined mimetypes on refresh.

2022-08-11 Thread David Bremner
Tomi Ollila  writes:

> On Mon, Aug 01 2022, David Bremner wrote:
>
>> This fixes the bug reported by Al [1]. Essentially apply the same fix
>> as [2] in a different place.
>>
>> [1]: id:877d41nmr1@gmail.com
>> [2]: 90a7c1af368a527700dcde9b0dcbd760afc7bd92
>
> LGTM.
>
> Tomi

Applied to master.

d
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org