[Buildroot] [PATCH 1/1] package/syrepo: fix SysV init script

Thomas Petazzoni
2020-04-25T19:37:17Z

On Sat, 25 Apr 2020 17:26:49 +0200
Heiko Thiery <heiko.thiery@gmail.com> wrote:

> > > The sysrepo-plugind does not support to be controlled via the PID by the
> > > start-stop-daemon. This is because sysrepo-plugind does a fork.
> > > Thus the daemon is running with another PID known by start-stop-daemon.
> >
> > "known" by start-stop-daemon, or "not known" ?
>
> Ok, I was unclear. The PID used by the daemon is another one that the
> start-stop-daemon knows.
>
> >
> > Also, it is not very clear. Indeed, in the "start" step, you're not
> > asking start-stop-daemon to create a PID file using the -m option. So
> > you're relying on the daemon to create one. Is this PID file incorrect
> > ? Normally, when it is created by the daemon itself, even if the daemon
> > forks, it should be fine.
>
> I recognized that and tried to do it that way (with the -m option).
> But in doing so I figured out that the "stop" step does not work. The
> daemon is running by a PID that is +1 compared to the one the
> start-stop-daemon knows from the PIDFILE.

But is sysrepo-plugind creating its own pidfile ?

Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering