<div dir="ltr">Dear Michael,<div><br></div><div>Thank you for the quick response and patch!</div><div><br></div><div>It looks good to me, as a systemd novice. My only questions would be:</div><div><br></div><div>* Whether it's possible for the services generated by <span style="font-size:12.8px">systemd-sysv-generator to be masked in their initial state? If so, we might want to check for not-found before masked, to give systemd a chance to parse the file and see if it should be masked or not.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">* Is systemctl daemon-reload synchronous? When it returns control to the OS, has systemd run </span><span style="font-size:12.8px">systemd-sysv-generator, has it finished, and has systemd parsed any new .service files that were generated? If not, then we might have to wait for that before asking systemd to start the service.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thanks, Chris.</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 31 May 2016 at 12:41, Michael Biebl <span dir="ltr"><<a href="mailto:biebl@debian.org" target="_blank">biebl@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Am 31.05.2016 um 12:16 schrieb Chris Wilson:<br>
> Jessie's 40-systemd script automatically invokes "systemctl daemon-reload" but<br>
> only when run from a dpkg install. Even this backwards-compatibility measure<br>
> seems to have been removed in unstable.<br>
<br>
</span>This is correct. Nowadays invoke-rc.d calls daemon-reload, so doing it<br>
twice is a bit excessive.<br>
<span class=""><br>
> Please consider invoking "systemctl -p LoadState show $service" from 40-systemd,<br>
> and if it says "not-found" then run "systemctl daemon-reload" to generate a<br>
> service file for it.<br>
<br>
</span>I think in a case like yours, where the package manager is not involved,<br>
doing an implicit reload for not-found in the lsb hook makes sense.<br>
After all, we are pretty sure the sysv init script actually exists. So<br>
the missing generated service file is indeed a pretty good indicator for<br>
a missing daemon-reload.<br>
<br>
So something like the attached patch?<br>
<span class="HOEnZb"><font color="#888888"><br>
Michael<br>
<br>
--<br>
Why is it that all of the instruments seeking intelligent life in the<br>
universe are pointed away from Earth?<br>
</font></span></blockquote></div><br></div>