<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Bringing in the debhelper maintainers into the loop again.</div><br><div class="gmail_quote"><div dir="ltr">On Sun, Jan 13, 2019 at 10:31 AM Dmitry Bogatov <<a href="mailto:KAction@debian.org">KAction@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
[ Do you want me to re-submit this patch as merge request? ]<br></blockquote><div><br></div><div>Not necessary (yet)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
[2019-01-11 14:58] Felipe Sateler <<a href="mailto:fsateler@debian.org" target="_blank">fsateler@debian.org</a>><br>
> > [2011-06-14 11:39] Joey Hess <<a href="mailto:joeyh@debian.org" target="_blank">joeyh@debian.org</a>><br>
> > ><br>
> > > part 1     text/plain                 618<br>
> > > Helmut Grohne wrote:<br>
> > > > A failure from the update-rc.d cannot make postinst fail, because the<br>
> > > > exit code is not checked.<br>
> > ><br>
> > > Yes it is (set -x), but I meant invoke-rc.d anyway, obviously</blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> > ><br>
> > > > So in my view the reason for a postinst failure is unrelated to<br>
> > > > update-rc.d. Can you explain your reasoning?<br>
> > ><br>
> > > Simple separation of concerns, invoke-rc.d is responsible for running<br>
> > > the init script and determining if it fails and propigating a failing<br>
> > > exit status. debhelper allows it to do do. The right way to make 6 be<br>
> > > ignored is to get invoke-rc.d to ignore it, not add cruft to every<br>
> > > postinst that calls it.</blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> ><br>
> > Here I propose patch to implement just that:<br>
> > [...]<br>
> I'm wary of unintended consequences here. Do we have services that return<br>
> exit code 6 but as a failure code?<br>
<br>
Seems there is nobody, who use exit code 6 for another purposes. You can<br>
make sure by checking is output of 'grep -R "exit 6" -C6' in directory<br>
with all init scripts unpacked:<br></blockquote><div><br></div><div><snip></div><div><br></div><div>Rereading invoke-rc.d manpage, it says quite explicitly:</div><div><br></div><div>> Should  an init script be executed, invoke-rc.d always returns the status code returned by the init script.<br></div></div><div dir="ltr"><br></div>So it looks like this would be quite the change in behavior. </div><div dir="ltr"><br>I wonder what the best approach it. OTOneH, I sympathize with Joey's argument that these checks should be centralized. OTOtherH, I'm not sure we can consider invoke-rc.d a debhelper implementation detail and change behaviour like this.</div><div dir="ltr"><br></div><div dir="ltr">Maybe a new flag to invoke-rc.d could be used for this? I'm not sure.<br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><br>Saludos,<br>Felipe Sateler</div></div></div></div>