~emersion

France

https://emersion.fr

~emersion/public-inbox

Last active 42 minutes ago

~emersion/alps-dev

Last active 5 days ago

~emersion/soju-dev

Last active 10 days ago

~emersion/mrsh-dev

Last active 19 days ago
View more

Recent activity

Re: Getting "Invalid charset specified" 51 minutes ago

From Simon Ser to ~sircmpwn/aerc

Hmm, I don't find this string in go-message, nor in the Go standard
library...

Re: [PATCH gamja v4] Add error reporting on connect and main page 2 days ago

From Simon Ser to ~emersion/public-inbox

Thanks for this new version! Looks a lot better, just a few remaining
issues and it'll be good to go :)

On Monday, July 27, 2020 9:17 PM, bbworld1 <vwangsf@gmail.com> wrote:

> ---
>
> The unrelated changes should be gone. Apologies.
> I accidentally sent another email without the gamja tag in the header.
> The patches are identical, but please disregard that one.
>
>  components/app.js     | 22 ++++++++++++++++------
>  components/connect.js |  1 +
>  lib/client.js         | 11 +++++------

Re: Variable substitution for the exec command 3 days ago

From Simon Ser to ~sircmpwn/aerc

On Tuesday, July 28, 2020 11:52 AM, Nicolai Dagestad <nicolai@dagestad.fr> wrote:

> Hi,
> I'm adding a simple variable substitution for the exec command so I can sync my
> maildirs with a shortcut from aerc.
> What would be the prefered format, something like `{acount}` and `{folder}`
> or %a and %f.

These solutions would have the downside of not allowing proper quoting.
If the folder contains a space or a quote, I don't think it's possible
to come up with a solution that works.

An alternative would be to set env variables and let the shell perform
the expansion. So it would be something like $account and $folder.

Re: [PATCH v3] Add error reporting on connect and main page 6 days ago

From Simon Ser to ~emersion/public-inbox

Thanks for sending this new version, however it still seems like you'll
need to send a new one! :P

This patch includes unrelated changes, I think the changes are coming
from recent commits pushed to master. Maybe this can help:

    git fetch
    # Delete the broken commit, but keeps the local changes:
    git reset origin/master
    # And then create a new commit as usual

See [1] for a good resource about editing Git history.

[1]: https://git-rebase.io/

Re: [PATCH soju v2] Fix typo in man page. 10 days ago

From Simon Ser to ~emersion/public-inbox

LGTM and pushed, thanks!

Re: [PATCH soju] Fix typo in man page. 10 days ago

From Simon Ser to ~emersion/public-inbox

> -	overwrite the remote and local connection addresses (via the X-Forwarded-*
> +	overwrite the remote and local connection addresses (via the X-Forwarded-

Ah, I guess this messes up with man page rendering. The intention was
to use * as a wildcard (because it refers to e.g. X-Forwarded-For and
X-Forwarded-Port).

Can we escape the * instead?

Re: [PATCH gamja] Add error reporting on connect and main page 11 days ago

From Simon Ser to ~emersion/public-inbox

Thanks for sending a new version, but it seems like this is the exact
same patch as the first one?

(Tip: you can give the -v2 flag to git-send-email to label a patch as
version 2.)

[PATCH core.sr.ht 3/3] Switch to SMTP.send_message 11 days ago

From Simon Ser to ~sircmpwn/sr.ht-dev

Let smtplib take care of the message formatting, because its encoding may
change depending on the capabilities of the SMTP server.
---
 srht/email.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srht/email.py b/srht/email.py
index d2e548aefc8a..1566b77834b7 100644
--- a/srht/email.py
+++ b/srht/email.py
@@ -123,7 +123,7 @@ def send_email(body, to, subject, encrypt_key=None, **headers):
        print(message)
        return
    smtp = start_smtp()
[message trimmed]

[PATCH core.sr.ht 1/3] Never use unixfrom in email.send_email 11 days ago

From Simon Ser to ~sircmpwn/sr.ht-dev

The "From line" is only meant to be used in e-mail archives (eml or
mbox). SMTP expects a regular MIME message.
---
 srht/email.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/srht/email.py b/srht/email.py
index 7a98a6135bf6..1f66e94c76d4 100644
--- a/srht/email.py
+++ b/srht/email.py
@@ -20,7 +20,6 @@ smtp_password = cfg("mail", "smtp-password", default=None)
smtp_from = cfg("mail", "smtp-from", default=None)
error_to = cfg("mail", "error-to", default=None)
error_from = cfg("mail", "error-from", default=None)
[message trimmed]