~benburwell

https://www.benburwell.com

~benburwell/patchall

Last active 5 months ago

~benburwell/test

Last active 6 months ago

~benburwell/howtochooseapassword

Last active 7 months ago
View more

Recent activity

Re: [PATCH v2] Contextual UI Configuration 5 days ago

From Ben Burwell to ~sircmpwn/aerc

On Thu Jan 23, 2020 at 13:56, Srivathsan Murali wrote:
> + Adds parsing of contextual ui sections to aerc config.
> + Add GetUiConfig method for AercConfig that is used to get the
>   specialized UI config.
> + Add UiConfig method to AccountView to get specialized UI Config.
> + Modifies Aerc codebase to use specialized UIConfig instead.
> + Adds documentation for Contextual UI Configuration
> ---
> Finaly got time to work on v2 of this patch, made changes according
> to Drew's feedback from the last version.
> Also added subject context to have specific index-format for patches.
>
> Happy new year folks :D

[PATCH] Fix handling of multiple template-dirs 5 days ago

From Ben Burwell to ~sircmpwn/aerc

Before, while the docs stated that template-dirs was a colon-separated
list, a delimiter was not specified in the struct tag, so it was falling
back to the default for the ini library (a comma). Also added a note to
the docs to clarify that templates are configured in the [templates]
section.
---
 config/config.go      | 6 +++---
 doc/aerc-config.5.scd | 4 +++-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/config/config.go b/config/config.go
index e5f7395..fe548ff 100644
--- a/config/config.go
+++ b/config/config.go
[message trimmed]

Re: [PATCH] imap: emit messageinfo when changing read state. 5 days ago

From Ben Burwell to ~sircmpwn/aerc

On Thu Jan 23, 2020 at 08:17, Reto Brunner wrote:
> -	imapw.worker.PostMessage(&types.Done{types.RespondTo(msg)}, nil)
> +	imapw.worker.PostAction(&types.FetchMessageHeaders{
> +		Uids: msg.Uids,
> +	}, func(msg types.WorkerMessage) {
> +		switch m := msg.(type) {
> +		case *types.Error:
> +			err := fmt.Errorf("handleReadMessages: %v", m.Error)
> +			imapw.worker.Logger.Printf("could not fetch headers: %s", err)
> +			emitErr(err)
> +		case *types.Done:
> +			imapw.worker.PostMessage(&types.Done{types.RespondTo(msg)}, nil)

Note that the msg parameter from the callback is shadowing here, so the

[PATCH] Add docs for reply -T 19 days ago

From Ben Burwell to ~sircmpwn/aerc

---
A minor omission I noticed while preparing the list-reply patch. Since
that series has been nack'd, I'm mailing this one separately. Thanks!

 doc/aerc.1.scd | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/doc/aerc.1.scd b/doc/aerc.1.scd
index 73a4d83..c0e8ad4 100644
--- a/doc/aerc.1.scd
+++ b/doc/aerc.1.scd
@@ -116,13 +116,15 @@ message list, the message in the message viewer, etc).

	*-p*: Pipe just the selected message part, if applicable
[message trimmed]

[PATCH] Strip trailing newline from address book entries without names 19 days ago

From Ben Burwell to ~sircmpwn/aerc

When the list of completions from the external command doesn't have
associated contact names, the email address we attempt to parse was
being terminated with a newline. Now, we strip the trailing newline if
present.
---
 completer/completer.go | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/completer/completer.go b/completer/completer.go
index f6900ee..bc6c96f 100644
--- a/completer/completer.go
+++ b/completer/completer.go
@@ -138,16 +138,18 @@ func readCompletions(r io.Reader) ([]string, error) {
			return nil, err
[message trimmed]

Re: [PATCH 1/2] Teach the reply command about mailing lists 22 days ago

From Ben Burwell to ~sircmpwn/aerc

On Mon Jan 6, 2020 at 8:47 AM, Reto wrote:
> Would it make sense to fallback to List-ID and try to parse it as a
> mail address?  Probably not RFC approved, but may be useful for some
> random list that doesn't set the List-Post header?

I would rather not add features that deviate from RFCs until we need
them for a sort of "quirks mode."

> I don't think that we need to (nor should) set all possible
> combinations as keyboard shortcuts. [...] People should set up those
> more "special" ones themselves.

Fair enough -- I was sort of on the fence about including the shortcuts;
I've removed them from v2.

[PATCH v2 2/2] Add docs for reply -T 22 days ago

From Ben Burwell to ~sircmpwn/aerc

---
 doc/aerc.1.scd | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/doc/aerc.1.scd b/doc/aerc.1.scd
index 70d1bdb..6dbb7b5 100644
--- a/doc/aerc.1.scd
+++ b/doc/aerc.1.scd
@@ -116,7 +116,7 @@ message list, the message in the message viewer, etc).

	*-p*: Pipe just the selected message part, if applicable

*reply* [-aql]
*reply* [-aql] [-T <template-file>]
[message trimmed]

[PATCH v2 1/2] Teach the reply command about mailing lists 22 days ago

From Ben Burwell to ~sircmpwn/aerc

Mailing lists can add a "List-Post" header containing a mailto URL
specifying where mail should be sent to post to the list. Add a -l
option to the reply command which will send the reply to the mailing
list found in the List-Post header, if present.

This header is specified in RFC 2369, the same RFC which specifies the
use of the List-Unsubscribe header with the same format. Thus, extract
and re-use the parsing logic from unsubscribe.go.

Additionally, perform cleanup of lingering references to when reply and
forward were implemented in the same file.
---
Since v1:
[message trimmed]

[PATCH 2/2] Add docs for reply -T 23 days ago

From Ben Burwell to ~sircmpwn/aerc

---
 doc/aerc.1.scd | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/doc/aerc.1.scd b/doc/aerc.1.scd
index 70d1bdb..6dbb7b5 100644
--- a/doc/aerc.1.scd
+++ b/doc/aerc.1.scd
@@ -116,7 +116,7 @@ message list, the message in the message viewer, etc).

	*-p*: Pipe just the selected message part, if applicable

*reply* [-aql]
*reply* [-aql] [-T <template-file>]
[message trimmed]

[PATCH 1/2] Teach the reply command about mailing lists 23 days ago

From Ben Burwell to ~sircmpwn/aerc

Mailing lists can add a "List-Post" header containing a mailto URL
specifying where mail should be sent to post to the list. Add a -l
option to the reply command which will send the reply to the mailing
list found in the List-Post header, if present.

This header is specified in RFC 2369, the same RFC which specifies the
use of the List-Unsubscribe header with the same format. Thus, extract
and re-use the parsing logic from unsubscribe.go.

Additionally, perform cleanup of lingering references to when reply and
forward were implemented in the same file.
---
 commands/msg/reply.go                         | 145 +++++++++++++-----
 commands/msg/unsubscribe.go                   |  28 +---
[message trimmed]