From: Noam Postavsky <npostavs@gmail.com>
To: Lucas Werkmeister <mail@lucaswerkmeister.de>
Cc: 31498@debbugs.gnu.org
Subject: bug#31498: [PATCH] Notify systemd in daemon-initialized and kill-emacs
Date: Sat, 09 Jun 2018 08:29:22 -0400 [thread overview]
Message-ID: <87k1r816u5.fsf@gmail.com> (raw)
In-Reply-To: <20180609105616.12927-1-mail@lucaswerkmeister.de> (Lucas Werkmeister's message of "Sat, 9 Jun 2018 12:56:16 +0200")
> >> I think it should have a NEWS entry mentioning that Emacs now supports
> >> this systemd notification feature. I would probably go ahead and
> >> combine with #31521, since it's using the same feature.
> >>
> >
> > Do you mean, announce both in this patch, or completely merge that patch
> > into this one?
> The latter (maybe it would have been clearer to put those two sentences
> in separate paragraphs).
I think I still managed to be unclear about this; I meant you should
both announce the change in etc/NEWS, and merge the patches.
Lucas Werkmeister <mail@lucaswerkmeister.de> writes:
> With --[bg-]daemon and Type=forking, systemd will only consider the
> daemon to have fully started up once the original process exits, and
> will wait until then to start units depending on the Emacs service. To
> get the same functionality with --fg-daemon, use Type=notify instead of
> Type=simple and explicitly send a readiness notification to systemd at
> the point where the forked process would in --bg-daemon mode notify its
> parent process and cause it to exit. Similarly, notify systemd at the
> beginning of the shutdown process as well. (Both of these calls are
> successful no-ops if emacs was not started by systemd.)
> * etc/emacs.service: Update Type.
> * src/emacs.c (daemon-initialized): Call sd_notify().
> * src/emacs.c (kill-emacs): Call sd_notify().
Thanks, I think I still managed to be unclear about the
next prev parent reply other threads:[~2018-06-09 12:29 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-18 17:40 bug#31498: [PATCH] Notify systemd in daemon-initialized Lucas Werkmeister
2018-06-07 13:08 ` Noam Postavsky
2018-06-07 15:38 ` Eli Zaretskii
2018-06-07 21:40 ` Lucas Werkmeister
2018-06-08 0:41 ` Noam Postavsky
2018-06-09 10:27 ` bug#31498: [PATCH] Notify systemd in daemon-initialized and kill-emacs Lucas Werkmeister
2018-06-09 10:55 ` bug#31498: (no subject) Lucas Werkmeister
2018-06-09 10:56 ` bug#31498: [PATCH] Notify systemd in daemon-initialized and kill-emacs Lucas Werkmeister
2018-06-09 12:29 ` Noam Postavsky [this message]
2018-06-09 12:31 ` Lucas Werkmeister
2018-06-09 12:36 ` Noam Postavsky
2018-06-09 13:00 ` Lucas Werkmeister
2018-06-09 14:05 ` Noam Postavsky
2018-06-13 13:42 ` Lucas Werkmeister
2018-06-13 16:33 ` Eli Zaretskii
2018-06-09 13:01 ` Lucas Werkmeister
2018-07-12 22:20 ` bug#31498: [PATCH] Notify systemd in daemon-initialized and, kill-emacs Lucas Werkmeister
2018-07-13 6:58 ` Eli Zaretskii
2018-07-25 12:03 ` bug#31498: [PATCH] Notify systemd in daemon-initialized and, , kill-emacs Lucas Werkmeister
2018-07-25 14:48 ` Eli Zaretskii
2018-07-26 1:13 ` Noam Postavsky
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k1r816u5.fsf@gmail.com \
--to=npostavs@gmail.com \
--cc=31498@debbugs.gnu.org \
--cc=mail@lucaswerkmeister.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.