<div dir="ltr">Hi Mike,<br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 28, 2018 at 6:15 PM Mike Brodbelt <<a href="mailto:mike@coruscant.org.uk">mike@coruscant.org.uk</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">On 21/06/18 12:34, Felipe Sateler wrote:<br>
<br>
> Patches welcome. If you decide to implement this, make sure that the <br>
> code is only active with a running sysvinit (ie, don't affect chroots).  <br>
> It is not trivial, which is why I didn't implement it myself. You <br>
> probably need to integrate ucf to handle all the corner cases.<br>
<br>
Patch attached.</blockquote><div><br></div><div>Thanks. However, it has some flaws that need to be reworked for this to work at all:</div><div><br></div><div>1. We can't rely on gcc being installed</div><div>2. It doesn't check /proc is actually mounted</div><div>3. dpkg status check is racy: sysvinit might be upgraded at the same time, and thus not be "installed"</div><div>4. dpkg status check is for the wrong package: sysvinit no longer exists (I think you want sysvinit-core).</div><div>5. The postinst should be the one for libpulse0, not pulseaudio</div><div><br></div><div>I would suggest poking around in /proc/1/ instead to check for sysvinit.</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"> I haven't gone through all the steps of integrating ucf <br>
- the disable autospawn file isn't currently marked in the package as a <br>
conffile, so this patch, while it works in my limited testing, could <br>
still be improved on there.<br></blockquote><div><br></div><div>You missed the conffile mark because it is shipped in libpulse0 :)</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>
LMK what you think - if you're happy with this approach and will <br>
integrate it, I can work on ucf integration as well.<br></blockquote><div><br></div><div>I don't have objections in principle, but I want a good and relatively not-risky implementation first.</div></div><div><br></div><div>BTW, it might be easier to do the review/fixup dance in a merge request on salsa: <a href="https://salsa.debian.org/pulseaudio-team/pulseaudio">https://salsa.debian.org/pulseaudio-team/pulseaudio</a></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><br>Saludos,<br>Felipe Sateler</div></div>