~emersion

France

https://emersion.fr

I'm Simon Ser, I write open-source software.

~emersion/soju-dev

Last active a day ago

~emersion/public-inbox

Last active 9 days ago

~emersion/mrsh-dev

Last active 5 months ago

~emersion/alps-dev

Last active 9 months ago

~emersion/drm-constraints

Last active 1 year, 1 day ago
View more

Recent activity

Re: [PATCH] Use NULL-tolerant comparison for DeliveryReceipts 2 days ago

From Simon Ser to ~emersion/soju-dev

Good catch, thanks! Pushed.

> This *might* be a related to #115

Unlikely, I think #115 was already happening on 0.1.0.

[PATCH] seatd-launch: print unlink/kill errors 3 days ago

From Simon Ser to ~kennylevinsen/seatd-devel

Makes it easier to find out that something went wrong.
---
 seatd-launch/seatd-launch.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/seatd-launch/seatd-launch.c b/seatd-launch/seatd-launch.c
index ba61f5d5c185..b9d33d761313 100644
--- a/seatd-launch/seatd-launch.c
+++ b/seatd-launch/seatd-launch.c
@@ -161,8 +161,12 @@ int main(int argc, char *argv[]) {
		}
	}

	unlink(sockpath);
[message trimmed]

[PATCH] seatd-launch: exit with status >128 if child is signalled 3 days ago

From Simon Ser to ~kennylevinsen/seatd-devel

Mimick shells and exit with a status >128 if our child has been
signalled. Exiting with 128 + signal number is what most shells do
(POSIX only requires them to exit with >128).
---
 man/seatd-launch.1.scd      | 8 ++++++++
 seatd-launch/seatd-launch.c | 4 +++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/man/seatd-launch.1.scd b/man/seatd-launch.1.scd
index d882ba5e8f5a..9234203f5a0c 100644
--- a/man/seatd-launch.1.scd
+++ b/man/seatd-launch.1.scd
@@ -38,6 +38,14 @@ superior to it for two reasons:
. The specified command never runs as root
[message trimmed]

Re: [PATCH] add flag to specify base path to themes 6 days ago

From Simon Ser to ~migadu/alps-devel

Thanks for the patch! Can you tell more about your use-case?

If this is about installing alps properly on the system (with themes in
e.g. /usr/share/alps/), maybe a build flag would be better? This is
possible with something like:

    -ldflags "-X main.themesPath=asdf"

Re: [PATCH] add flag to specify base path to themes 7 days ago

From Simon Ser to ~migadu/alps-devel

On Thursday, September 9th, 2021 at 20:50, Peter Sanchez <pjs@petersanchez.com> wrote:

> >+func loadTheme(themesPath, name string, base *template.Template) (*template.Template, error) {
>
> I believe this should be `themesPath string` - it's missing it's type
> cast.

You mean that the argument type is missing? That's valid Go, if the
type is omitted, it takes the one from the next argument.

Re: [PATCH] Check for TLS config in wss listeners 7 days ago

From Simon Ser to ~emersion/soju-dev

Pushed, thanks!

Re: [getopt] stable release? 7 days ago

From Simon Ser to ~sircmpwn/public-inbox

On Thursday, September 9th, 2021 at 09:09, Drew DeVault <sir@cmpwn.com> wrote:

> I don't really know what all is involved in making stable Go libraries.
>
> Is it as simple as tagging it?

Yes, just tag it, and everything should pick up the new version. Name
the tag "v1.0.0" (the "v" prefix is important) to make it clear it's a
version tag.

Re: [gamja] Focus composer on visibilitychange 9 days ago

From Simon Ser to ~emersion/public-inbox

On Tuesday, September 7th, 2021 at 09:37, Hubert HIRTZ <hubert.hirtz.ocre@cea.fr> wrote:

> When clicking a link in gamja, it sends the user to a new tab.  When the
> user comes back, it expects the composer to be focused, but the clicked
> link is focused.
>
> Solution:
> https://developer.mozilla.org/en-US/docs/Web/API/Page_Visibility_API
>
>      document.addEventListener('visibilitychange', () => {
>          if (!document.hidden) {
>              document.getElementById('composer').text.focus()
>          }
>      })