~rjarry/aerc-devel

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
2 2

[PATCH] Document filter input charset and add w3m filter example to default config

Details
Message ID
<20220801113354.2523-1-q3cpma@posteo.net>
DKIM signature
missing
Download raw message
Patch: +4 -0
---
 config/aerc.conf      | 1 +
 doc/aerc-config.5.scd | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/config/aerc.conf b/config/aerc.conf
index 3b7b5d6..fc6479a 100644
--- a/config/aerc.conf
+++ b/config/aerc.conf
@@ -292,6 +292,7 @@ reply-to-self=true
#subject,~^\[PATCH=colordiff
text/plain=sed 's/^>\+.*/\x1b[36m&\x1b[0m/'
#text/html=pandoc -f html -t plain
#text/html=w3m -dump -I UTF-8 -T text/html
#image/*=catimg -w $(tput cols) -

[triggers]
diff --git a/doc/aerc-config.5.scd b/doc/aerc-config.5.scd
index c712c58..aaf15b8 100644
--- a/doc/aerc-config.5.scd
+++ b/doc/aerc-config.5.scd
@@ -497,6 +497,9 @@ aerc ships with some default filters installed in the share directory (usually
_/usr/share/aerc/filters_). Note that these may have additional dependencies
that aerc does not have alone.

Note that said email body is converted into UTF-8 before being passed to
filters.

## TRIGGERS

Triggers specify commands to execute when certain events occur.
-- 
2.35.1
Details
Message ID
<CLUO3IBO16X1.2AC4RDCF7LP0W@marty>
In-Reply-To
<20220801113354.2523-1-q3cpma@posteo.net> (view parent)
DKIM signature
missing
Download raw message
q3cpma, Aug 01, 2022 at 13:33:
> ---
>  config/aerc.conf      | 1 +
>  doc/aerc-config.5.scd | 3 +++
>  2 files changed, 4 insertions(+)

The title (first line) should be: "filters: make it explicit that
encoding is UTF-8" Also, please add an extended description in the
commit message body.

Finally, add the following git trailers:

Fixes: https://todo.sr.ht/~rjarry/aerc/65
Signed-off-by: q3cpma <q3cpma@posteo.net>

>
> diff --git a/config/aerc.conf b/config/aerc.conf
> index 3b7b5d6..fc6479a 100644
> --- a/config/aerc.conf
> +++ b/config/aerc.conf
> @@ -292,6 +292,7 @@ reply-to-self=true
>  #subject,~^\[PATCH=colordiff
>  text/plain=sed 's/^>\+.*/\x1b[36m&\x1b[0m/'
>  #text/html=pandoc -f html -t plain
> +#text/html=w3m -dump -I UTF-8 -T text/html
>  #image/*=catimg -w $(tput cols) -
>  
>  [triggers]
> diff --git a/doc/aerc-config.5.scd b/doc/aerc-config.5.scd
> index c712c58..aaf15b8 100644
> --- a/doc/aerc-config.5.scd
> +++ b/doc/aerc-config.5.scd
> @@ -497,6 +497,9 @@ aerc ships with some default filters installed in the share directory (usually
>  _/usr/share/aerc/filters_). Note that these may have additional dependencies
>  that aerc does not have alone.
>  
> +Note that said email body is converted into UTF-8 before being passed to
> +filters.
> +

Could you also update the html and html-unsafe filters to force the
UTF-8 encoding?

Thanks.
Details
Message ID
<CLUP7IFEZD53.1IR3R0XQ1Z1IF@gentoo-zen5900x>
In-Reply-To
<CLUO3IBO16X1.2AC4RDCF7LP0W@marty> (view parent)
DKIM signature
missing
Download raw message
> The title (first line) should be: "filters: make it explicit that
> encoding is UTF-8" Also, please add an extended description in the
> commit message body.
>
> Finally, add the following git trailers:
>
> Fixes: https://todo.sr.ht/~rjarry/aerc/65
> Signed-off-by: q3cpma <q3cpma@posteo.net>
>

Done, sorry I'm not very familiar with the sr.ht/git mail workflow.

>
> Could you also update the html and html-unsafe filters to force the
> UTF-8 encoding?
>

Oh I see, will do so in v3.
Reply to thread Export thread (mbox)