<div dir="ltr">I’m currently travelling. Might have time for some patches mid/end of next week. If someone wants to race me to it (should be simple?), please go ahead.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 7, 2017 at 9:02 AM, Holger Levsen <span dir="ltr"><<a href="mailto:holger@layer-acht.org" target="_blank">holger@layer-acht.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<span class=""><br>
On Tue, Mar 07, 2017 at 01:40:30PM +0000, Simon McVittie wrote:<br>
> This seems to have caused piuparts regressions for what I believe are<br>
> otherwise correct packages:<br>
> <a href="https://piuparts.debian.org/sid/fail/libgtk-3-doc_3.22.9-3.log" rel="noreferrer" target="_blank">https://piuparts.debian.org/<wbr>sid/fail/libgtk-3-doc_3.22.9-<wbr>3.log</a><br>
<br>
</span>thanks for notifying us about this, Simon!<br>
<span class=""><br>
> Since the release team use piuparts regressions as a way to gate packages<br>
> entering testing, this is likely to be creating extra work for the<br>
> release team and for maintainers/NMUers who need changes unblocked.<br>
<br>
</span>I guess that's somewhat fair game, as since this is the case, the workload for<br>
piuparts maintenance and maintainers has increased quite a bit… :-)<br>
<span class=""><br>
> I can't see anything in that log that looks like a failure, other than:<br>
><br>
> 0m13.6s DUMP:<br>
>   Removing 'local diversion of /usr/bin/update-alternatives to /usr/bin/update-alternatives.<wbr>orig'<br>
>   cat: /tmp/log-alternatives.log: No such file or directory<br>
>   rm: cannot remove '/tmp/log-alternatives.log': No such file or directory<br>
> 0m13.6s ERROR: Command failed (status=1): ['chroot', '/srv/<a href="http://piuparts.debian.org/tmp/tmpMU3r5N" rel="noreferrer" target="_blank">piuparts.debian.org/tmp/<wbr>tmpMU3r5N</a>', 'tmp/scripts/post_install_log_<wbr>alternatives']<br>
>   Removing 'local diversion of /usr/bin/update-alternatives to /usr/bin/update-alternatives.<wbr>orig'<br>
>   cat: /tmp/log-alternatives.log: No such file or directory<br>
>   rm: cannot remove '/tmp/log-alternatives.log': No such file or directory<br>
><br>
> I assume this is because libgtk-3-doc is such a simple package that it<br>
> doesn't depend on anything that would cause any alternatives to be<br>
> updated, so the log was never created.<br>
<br>
</span>I guess so too, but I'll leave the detailed analysis and followups to Michael…<br>
<span class=""><br>
> To be robust against the log either never being created, or being cleaned up<br>
> by tmpreaper or similar, it would seem better to do:<br>
><br>
> -cat /tmp/log-alternatives.log<br>
> -rm /tmp/log-alternatives.log<br>
> +cat /tmp/log-alternatives.log || :<br>
> +rm -f /tmp/log-alternatives.log<br>
<br>
</span>I think I prefer "|| true" over "|| :" for readability…<br>
<span class=""><br>
> The scripts added by<br>
> <a href="https://github.com/stapelberg/piuparts/commit/4867f7251189bd9f48fc229804a6ae44623546e4" rel="noreferrer" target="_blank">https://github.com/stapelberg/<wbr>piuparts/commit/<wbr>4867f7251189bd9f48fc229804a6ae<wbr>44623546e4</a><br>
> also don't use set -e, unless that was added later. They probably should.<br>
<br>
</span>agreed.<br>
<br>
Michael, can you please send some patches along?! ;-)<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
cheers,<br>
        Holger<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Best regards,<br>Michael</div>
</div>