Bug #1177720 “Problem with custom InstallProgress” : Bugs : python-apt package : Ubuntu

Problem with custom InstallProgress

Bug #1177720 reported by Alexey Palazhchenko
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
python-apt (Debian)
Fix Released
Undecided
Unassigned
python-apt (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Hello,

I'm using Ubuntu 12.04.2 LTS and python-apt 0.8.3ubuntu7. I wrote a small Python program (attached) to install a package. It doesn't have custom fork(), but program behaves like it does.

aleksi@ubuntu-virtual-server:~$ sudo ./test.py
Reading package lists... Done
Building dependency tree
Reading state information... Done
Fetched 0 B in 0s (0 B/s)
custom fork found
got pid: 9610
start_update
got pid: 0
got fd: 4
finish_update
aleksi@ubuntu-virtual-server:~$ Selecting previously unselected package goo.
(Reading database ... 88751 files and directories currently installed.)
Unpacking goo (from .../goo_0.155-10_amd64.deb) ...
Processing triggers for menu ...
Processing triggers for man-db ...
Setting up goo (0.155-10) ...
                             Processing triggers for menu ...

                                                             aleksi@ubuntu-virtual-server:~$

Tags: precise
Revision history for this message
Alexey Palazhchenko (aleksi775) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in python-apt (Ubuntu):
status: New → Confirmed
tags: added: precise
Changed in python-apt (Ubuntu):
importance: Undecided → Low
Revision history for this message
Julian Andres Klode (juliank) wrote :

This is a problem with base.InstallProgress (just checked on 0.9.3.3) as well, I haven't looked any closer at it yet, though.

Revision history for this message
Julian Andres Klode (juliank) wrote :

Nobody except you seems to have used this DepCache.commit() The problem is that we do not set the child_pid attribute of the progress handler, thus failing. If we do this, we hit some more bugs, which I fixed in the attached patch.

Fixed in Debian git, commits:

http://anonscm.debian.org/gitweb/?p=apt/python-apt.git;a=commit;h=47dfb3fac23721f76fc3084835f2c3c6382777ce
http://anonscm.debian.org/gitweb/?p=apt/python-apt.git;a=commit;h=cd47e3e7e0a87ead87724d0120d408b7993e94c7

Changed in python-apt (Debian):
status: New → Fix Committed
Revision history for this message
Julian Andres Klode (juliank) wrote :

I'll upload 0.9.3.4 to Debian soon then, and we can sync to Ubuntu afterwards.

Changed in python-apt (Ubuntu):
status: Confirmed → In Progress
Changed in python-apt (Debian):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package python-apt - 0.9.3.4

---------------
python-apt (0.9.3.4) unstable; urgency=medium

  * python/progress.cc: Fix InstallProgress to set child_pid (LP: #1177720)
  * python/progress.cc: Remove invalid checks for mixedCase methods
  * utils/get_*_mirrors*.py: Ensure failure if no mirrors were read
  * utils/get_*_mirrors*.py: Use sys.stderr.write instead of print
  * Support all compressors for data.tar/control.tar (Closes: #718330)
  * Revert "aptsources/sourceslist.py: ensure that our sources are opened with UTF-8"
    (Closes: #742118)

 -- Julian Andres Klode <email address hidden> Thu, 20 Mar 2014 16:48:12 +0100

Changed in python-apt (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Loading subscribers...

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.