Bug#840146: pbuilder: remove_packages checks for existence in host, not in chroot

[James Clarke]

Control: found -1 0.224
Control: severity -1 important

Hi Thorsten,
> On 8 Oct 2016, at 21:20, Thorsten Glaser <tg at mirbsd.de> wrote:
> 
> Package: pbuilder
> Version: 0.226.1
> Severity: serious
> Justification: breaks basic use
> 
> The remove_packages function in pbuilder-modules contains:
> 
>        if (dpkg -s "$pkg" 2>&1)>/dev/null ; then
> 
> This obviously must be:
> 
>        if ($CHROOTEXEC dpkg -s "$pkg" 2>&1)>/dev/null ; then
> 
> Because of this, I can’t “upgrade” (reads: switch between apt- and
> aptitude-based resolvers) my sarge chroot any more.

I agree, this is bad. However, if this is only affecting sarge chroots,
I fail to see how this is release-critical. I understand it's a pain for
your use, but it certainly does not break "basic use". If it's
problematic for wheezy+ chroots, please feel free to change the severity
back to serious.

Regards,
James