Prague, Czech Republic
From Robin Opletal to ~sircmpwn/public-inbox
I can really agree that such tool would be great. The issues and limitations of physical tools are exactly the ones I experience. Thanks for the post. Side note: will your talk in Italy be recorded and available/streamed on any platform, or is it just an in-person event? Thanks, Robin
From Robin Opletal to ~sircmpwn/sr.ht-discuss
Thanks for letting me know and for the latest financial report.
From Robin Opletal to ~sircmpwn/sr.ht-discuss
Hi there, What’s the reason for financial reports no longer being posted on the blog? Thanks and best regards, Robin
From Robin Opletal to ~sircmpwn/aerc
On Thu Jan 14, 2021 at 7:19 AM CET, Reto wrote: > Time appropriate greetings, > Thanks for the patch. > > On Tue, Jan 12, 2021 at 09:58:54PM +0100, Robin Opletal wrote: > > + // Used to handle "/", "." and ".." in filenames correctly > > When I said add a comment explaining why I meant something a bit more > descriptive ;) > Anyhow, fixed it up locally and merged it. > I also took the liberty to change the if chain to a switch statement, > which > hopefully helps the reader a bit. >
From Robin Opletal to ~sircmpwn/aerc
On Tue Jan 12, 2021 at 9:55 PM CET, Reto wrote: > Maybe https://git-send-email.io/ helps? Thanks - went through it once, should probably rinse and repeat. > If you mean some other feature, you need to send that as a separate > patch. > The way patch sets work is that they should bundle commits of a single > "feature" It's the same patch, with a fix in a second commit which, instead of ammending to the previous commit I have made another commit.
From Robin Opletal to ~sircmpwn/aerc
--- commands/msgview/save.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/commands/msgview/save.go b/commands/msgview/save.go index ef6bba8..375be96 100644 --- a/commands/msgview/save.go +++ b/commands/msgview/save.go @@ -182,7 +182,10 @@ func generateFilename(part *models.BodyStructure) string { filename = fn } else if fn, ok := part.Params["name"]; ok { filename = fn } else { }[message trimmed]
From Robin Opletal to ~sircmpwn/aerc
On Tue Jan 12, 2021 at 9:31 PM CET, Reto wrote: > Hi Robin, > Thanks for the patch. > However it does not seem to apply for me... is this the latest patch? > There were some hiccups along the way I think, care to resend? Hi Reto, thank you for reviewing the patch. Yeah, I am not quite used to using git-send-email yet. Have to spend some more time with it to get comfortable. I am thinking what the best way would be to send both of the commits that I have made as a single patch? Probably send it as a "new" patch without any annotate?
From Robin Opletal to ~sircmpwn/aerc
On Thu Jan 7, 2021 at 11:58 AM CET, Robin Opletal wrote: > On Thu Jan 7, 2021 at 11:38 AM CET, Simon Ser wrote: > > This fixes up the wrong line -- now there are two lines with spaces > > instead of tabs. > > > > You can use `go fmt ./...` to fix everything. > > Thanks for letting me know. Fixed in another patch. It would also be good if I started using git commit --ammend, huh....
From Robin Opletal to ~sircmpwn/aerc
On Thu Jan 7, 2021 at 11:38 AM CET, Simon Ser wrote: > This fixes up the wrong line -- now there are two lines with spaces > instead of tabs. > > You can use `go fmt ./...` to fix everything. Thanks for letting me know. Fixed in another patch.
From Robin Opletal to ~sircmpwn/aerc
--- commands/msgview/save.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/commands/msgview/save.go b/commands/msgview/save.go index 7649a5f..375be96 100644 --- a/commands/msgview/save.go +++ b/commands/msgview/save.go @@ -181,10 +181,10 @@ func generateFilename(part *models.BodyStructure) string { if fn, ok := part.DispositionParams["filename"]; ok { filename = fn } else if fn, ok := part.Params["name"]; ok { filename = fn }[message trimmed]