<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Jun 29, 2018 at 3:26 AM Mike Brodbelt <<a href="mailto:mike@coruscant.org.uk">mike@coruscant.org.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 29/06/18 04:26, Felipe Sateler wrote:<br>
<br>
> Thanks. However, it has some flaws that need to be reworked for this to <br>
> work at all:<br>
> <br>
> 1. We can't rely on gcc being installed<br>
<br>
Would "dpkg-architecture -qDEB_BUILD_MULTIARCH" be a reasonable replacement?<br></blockquote><div><br></div><div>Unfortunately not. It is part of dpkg-dev, which is also not installed by default (and requiring it via pulseaudio doesn't make much sense). I think using /proc/1/comm or /proc/1/exe are better alternatives.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> 2. It doesn't check /proc is actually mounted<br>
> 3. dpkg status check is racy: sysvinit might be upgraded at the same <br>
> time, and thus not be "installed"<br>
> 4. dpkg status check is for the wrong package: sysvinit no longer exists <br>
> (I think you want sysvinit-core).<br>
> 5. The postinst should be the one for libpulse0, not pulseaudio<br>
<br>
Will have a poke at these when I get a minute.<br>
<br>
> You missed the conffile mark because it is shipped in libpulse0 :)<br>
<br>
That would explain that :-)<br>
<br>
> I don't have objections in principle, but I want a good and relatively <br>
> not-risky implementation first.<br>
<br>
Ack.<br>
<br>
> BTW, it might be easier to do the review/fixup dance in a merge request <br>
> on salsa: <a href="https://salsa.debian.org/pulseaudio-team/pulseaudio" rel="noreferrer" target="_blank">https://salsa.debian.org/pulseaudio-team/pulseaudio</a><br>
<br>
OK. Haven't played with that before, but will have a look.<br></blockquote><div><br></div><div>It's not strictly required, but I would prefer that, thanks.</div><div> </div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><br>Saludos,<br>Felipe Sateler</div></div>