~rjarry

Paris, France

~rjarry/aerc-devel

Last active 2 hours ago

~rjarry/dlrepo

Last active a day ago

~rjarry/aerc-discuss

Last active 2 days ago

~rjarry/aerc-announce

Last active 25 days ago
View more

Recent activity

Re: [PATCH aerc v3] aerc: use single event loop 3 hours ago

From Robin Jarry to ~rjarry/aerc-devel

Tim Culverhouse, Sep 25, 2022 at 21:55:
> Combine tcell events with WorkerMessages to better synchronize state
> with IO and UI. Remove Tick loop for rendering. Use events to trigger
> renders.
>
> Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
> ---
> v3: Rebase without a tcell-term version bump I had in my tree

By the way, this commit breaks ctrl-* key bindings. The ctrl-key does
not register immediately and must be followed by another key (or enter)
to actually register. It may be related to my config but it is worth
having a look.

Re: [PATCH aerc v3] aerc: use single event loop 16 hours ago

From Robin Jarry to ~rjarry/aerc-devel

Tim Culverhouse, Sep 25, 2022 at 21:55:
> Combine tcell events with WorkerMessages to better synchronize state
> with IO and UI. Remove Tick loop for rendering. Use events to trigger
> renders.
>
> Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>

That looks surprisingly simple :)

> diff --git a/aerc.go b/aerc.go
> index e1a1345b1d50..3c535d853809 100644
> --- a/aerc.go
> +++ b/aerc.go
> @@ -9,9 +9,9 @@ import (

Re: UI Refactor Proposal 17 hours ago

From Robin Jarry to ~rjarry/aerc-devel

Tim Culverhouse, Sep 25, 2022 at 21:01:
> Hi all -
>
> I would like to propose the following for a UI refactor.

Hi Tim,

this is awesome.

> 1. Create a single main event loop
>
> This loop will be responsible for all UI drawing, and all account
> state updates. To do this, all messages *from* the worker will be
> redirected to this loop. This keeps all input (ie tcell.Event events)

Re: [PATCH aerc] #64 opener overrides 20 hours ago

From Robin Jarry to ~rjarry/aerc-devel

Hey Jason,

thanks for your contribution. Please make sure to follow all guidelines
in the contributing guide:

https://git.sr.ht/~rjarry/aerc/tree/master/item/CONTRIBUTING.md?view-source#L34-52

Jason Stewart, Sep 25, 2022 at 18:01:
> opener overrides:
>
>   created lib.DoFileOpener() to handle:
>     - command selection/creation
>     - arg processing / var expansion
>

Re: [dlrepo/patches/.build.yml] build success a day ago

From Robin Jarry to ~rjarry/dlrepo

builds.sr.ht, Sep 23, 2022 at 17:02:
> dlrepo/patches/.build.yml: SUCCESS in 1m25s
>
> [tar: set members time][0] from [Julien Floret][1]
>
> [0]: https://lists.sr.ht/~rjarry/dlrepo/patches/35531
> [1]: julien.floret@6wind.com
>
> ✓ #849661 SUCCESS dlrepo/patches/.build.yml https://builds.sr.ht/~rjarry/job/849661

For some reason, I did not receive the patch email message.

Applied. Thanks!

Re: [PATCH dlrepo] cli: print target url a day ago

From Robin Jarry to ~rjarry/dlrepo

Julien Floret, Sep 15, 2022 at 11:59:
> Print the target URL before dumping the related info requested.
> It can help users navigate between the CLI and the web UI, and prevent
> login mistakes too.
>
> Signed-off-by: Julien Floret <julien.floret@6wind.com>
> Acked-by: Thomas Faivre <thomas.faivre@6wind.com>

Applied. Thanks!

Re: [PATCH dlrepo] views: use ACLs when resolving tag aliases a day ago

From Robin Jarry to ~rjarry/dlrepo

Julien Floret, Sep 16, 2022 at 10:34:
> Before this patch, when an HTTP request URL was using the
> "latest" or "stable" dynamic aliases for tag/versions, the
> redirected URL was determined without taking the access rights into
> account. Consequently, the request could be redirected to a version not
> accessible by the user, who could get an "Authentication Required" in
> response.
>
> Ensure the ACLs are checked when determining the candidate tags for
> these aliases.
>
> Signed-off-by: Julien Floret <julien.floret@6wind.com>
> Acked-by: Thomas Faivre <thomas.faivre@6wind.com>

Re: [PATCH dlrepo v2 0/3] rework the stable tag status a day ago

From Robin Jarry to ~rjarry/dlrepo

Julien Floret, Sep 23, 2022 at 17:10:
> This small series is a rework of the "stable" tag concept to decorrelate
> it from the "released" status, and add an "oldstable" alias.
>
> After the first cosmetic patch, patch 2 introduces a new "stable" status
> for tags, that can be changed via an HTTP PUT request. Request URLs
> using the "latest" alias are redirected to the latest tag declared
> "stable", whether it has already been released or not.
> Patch 3 adds the "oldstable" alias, which is redirected to the latest
> but one stable tag.
>
> Changes in v2: add stable info to "dlrepo-cli status" output.
>
> Julien Floret (3):

Re: [PATCH aerc] charset: handle unknown charsets more user-friendly a day ago

From Robin Jarry to ~rjarry/aerc-devel

Koni Marti, Sep 21, 2022 at 00:27:
> Do not throw an error when the charset is unknown; the message entity
> can still be read, but log the error instead.
>
> Reported-by: falsifian
> Signed-off-by: Koni Marti <koni.marti@gmail.com>

Acked-by: Robin-Jarry <robin@jarry.cc>

Applied. Thanks!