[Bug 1640892] Re: pbuilder: remove_packages() checks for existence in host, not in chroot

[Brian Murray]

Hello Mattia, or anyone else affected,

Accepted pbuilder into yakkety-proposed. The package will build now and
be available at
https://launchpad.net/ubuntu/+source/pbuilder/0.226.1ubuntu0.1 in a few
hours, and then in the -proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to
enable and use -proposed.  Your feedback will aid us getting this update
out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested, and change the tag
from verification-needed to verification-done. If it does not fix the
bug for you, please add a comment stating that, and change the tag to
verification-failed.  In either case, details of your testing will help
us make a better decision.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance!

** Changed in: pbuilder (Ubuntu Yakkety)
       Status: In Progress => Fix Committed

** Tags added: verification-needed

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to pbuilder in Ubuntu.
https://bugs.launchpad.net/bugs/1640892

Title:
  pbuilder: remove_packages() checks for existence in host, not in
  chroot

Status in pbuilder package in Ubuntu:
  Fix Released
Status in pbuilder source package in Yakkety:
  Fix Committed
Status in pbuilder package in Debian:
  Fix Released

Bug description:
  The fix:
  https://anonscm.debian.org/git/pbuilder/pbuilder.git/commit/?id=bf1acdc

  [Impact]

  The remove_packages() function in pbuilder-modules contains:
      if (dpkg -s "$pkg" 2>&1)>/dev/null ; then
  This obviously must
      if ($CHROOTEXEC dpkg -s "$pkg" 2>&1)>/dev/null ; then

  This causes remove_packages() to try to remove packages that are not
  present, actually making the function not working at all in some
  circumstances, leading to crashes; or anyway, it causes it to try to
  remove packages from the chroot even when they are not there.

  [Test Case]

  2 ways:

  1)
   * get a sarge chroot, or anyway around that time
   * assuming you have pbuilder installed on the host and not in the chroot,
     run:
         pbuilder u --removepackages pbuilder
   * pbuilder crashes
  2)
   * get a regular chroot of something common these days
   * run `pbuilder u --removepackages pbuilder`
   * notice how it tries to remove pbuilder from inside the chroot (it
     shouldn't even try)

  [Regression Potential]

   * for yakkety, no potential regression, this is a targeted fix.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/pbuilder/+bug/1640892/+subscriptions