[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#726156: marked as done (latest APT breaks InstallProgress)



Your message dated Tue, 15 Oct 2013 19:48:26 +0000
with message-id <E1VWAbS-0006mR-IK@franck.debian.org>
and subject line Bug#726156: fixed in apt 0.9.12.1
has caused the Debian Bug report #726156,
regarding latest APT breaks InstallProgress
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
726156: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=726156
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: python-apt
Version: 0.8.9.1
Tags: patch

Hello,


It seems that the latest APT update breaks the base python-apt
InstallProgress, as some package names in the status stream also feature
the package architecture (packagename:arch).

This is the produced traceback:


Traceback (most recent call last):
  File "/usr/bin/bricks", line 181, in run
    engine.cache.commit(AcquireProgress(self.parent),
InstallProgress(self.parent))
  File "/usr/lib/python2.7/dist-packages/apt/cache.py", line 504, in commit
    res = self.install_archives(pm, install_progress)
  File "/usr/lib/python2.7/dist-packages/apt/cache.py", line 467, in
install_archives
    res = install_progress.run(pm)
  File "/usr/lib/python2.7/dist-packages/apt/progress/base.py", line
206, in run
    res = self.wait_child()
  File "/usr/lib/python2.7/dist-packages/apt/progress/base.py", line
276, in wait_child
    self.update_interface()
  File "/usr/lib/python2.7/dist-packages/apt/progress/base.py", line
253, in update_interface
    if float(percent) != self.percent or status_str != self.status:
ValueError: could not convert string to float: amd64


I'm attaching a quick and dirty patch that fixes the issue.


Regards,
Eugenio

-- 
.: Eugenio Paolantonio - Semplice Linux developer :.

http://semplice-linux.org
http://medesimo.eu

Twitter: @eugenio_g7 - Homepage: http://me.medesimo.eu
--- orig/apt/progress/base.py	2013-06-06 18:32:36.000000000 +0200
+++ mod/apt/progress/base.py	2013-10-12 22:27:54.050668349 +0200
@@ -221,10 +221,17 @@
             return
 
         pkgname = status = status_str = percent = base = ""
+        architecture = "all"
 
         if line.startswith('pm'):
             try:
                 (status, pkgname, percent, status_str) = line.split(":", 3)
+                # Handle pkgnames with :architecture, otherwise we'll later
+                # try to make the package architecture a float.
+                try:
+                    float(percent)
+                except ValueError:
+                    (status, pkgname, architecture, percent, status_str) = line.split(":", 4)
             except ValueError:
                 # silently ignore lines that can't be parsed
                 return

--- End Message ---
--- Begin Message ---
Source: apt
Source-Version: 0.9.12.1

We believe that the bug you reported is fixed in the latest version of
apt, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 726156@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Michael Vogt <mvo@debian.org> (supplier of updated apt package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Tue, 15 Oct 2013 18:25:51 +0200
Source: apt
Binary: apt libapt-pkg4.12 libapt-inst1.5 apt-doc libapt-pkg-dev libapt-pkg-doc apt-utils apt-transport-https
Architecture: source all amd64
Version: 0.9.12.1
Distribution: unstable
Urgency: low
Maintainer: APT Development Team <deity@lists.debian.org>
Changed-By: Michael Vogt <mvo@debian.org>
Description: 
 apt        - commandline package manager
 apt-doc    - documentation for APT
 apt-transport-https - https download transport for APT
 apt-utils  - package management related utility programs
 libapt-inst1.5 - deb package format runtime library
 libapt-pkg-dev - development files for APT's libapt-pkg and libapt-inst
 libapt-pkg-doc - documentation for APT development
 libapt-pkg4.12 - package management runtime library
Closes: 726047 726156
Changes: 
 apt (0.9.12.1) unstable; urgency=low
 .
   [ Michael Vogt ]
   * do not send pkgname:arch over the APT::Status-Fd to not break
     clients (closes: 726156). A new APT::Status-deb822-Fd will be
     used to fix this.
   * add integration tests for APT::Status-Fd
   * add missing _() around the new "Progress" string
 .
   [ David Kalnischkies ]
   * fix progress-segfault in case of dpkg errors/prompts (Closes: 726047)
Checksums-Sha1: 
 8923824ae1deb775e004e368803dc538ba0cc22b 1617 apt_0.9.12.1.dsc
 fae193a24248b4703ced9f78df49f8102b4016e2 3420713 apt_0.9.12.1.tar.gz
 faa693bbf6c8cce0ffe88543606584894a1c8954 266658 apt-doc_0.9.12.1_all.deb
 1a83d61e92d464bc3c18939183c2d255e6ad5a98 555648 libapt-pkg-doc_0.9.12.1_all.deb
 0ff24a51a4f66a3b998eef66b09b77a55f468e0f 717776 libapt-pkg4.12_0.9.12.1_amd64.deb
 84ae3b235075555124591358efc8412aa3939a61 157870 libapt-inst1.5_0.9.12.1_amd64.deb
 f1cb4c8448b9d47ebd3ab0dbff6e446b74b71821 1046782 apt_0.9.12.1_amd64.deb
 dbadead6fb20fb02d889decb082fb077d0c51335 179576 libapt-pkg-dev_0.9.12.1_amd64.deb
 e6e7040722f3fa4860d1441bacb22ccfc240bc10 345514 apt-utils_0.9.12.1_amd64.deb
 4d40c9d6d9fda17ad06266c0ea37c18b309c1467 124666 apt-transport-https_0.9.12.1_amd64.deb
Checksums-Sha256: 
 74c5f554a8878d01874db6534f827e5b2b89f931eae1cb6fb5dd327023c162fd 1617 apt_0.9.12.1.dsc
 9cec76243557d1c597017eb33ebec8090c34f62fa771109f8994f7fe0f517e23 3420713 apt_0.9.12.1.tar.gz
 5b9691da8acc280ccf1410933bc425797a8c0d4e2490a51e5b676a01585c7023 266658 apt-doc_0.9.12.1_all.deb
 335b94a776fd04f8496e8f19a8c8dc2a8dc26b493a85cbdee0d9c42dadd5986e 555648 libapt-pkg-doc_0.9.12.1_all.deb
 d23df99fc20eaa0e8d02ac38e0c8d823cc34c1fdc38a009cf85e605a105608ec 717776 libapt-pkg4.12_0.9.12.1_amd64.deb
 0670d6920e98404277baf8ada8bc0f86f79886dd89b24365118de3487a0fd19e 157870 libapt-inst1.5_0.9.12.1_amd64.deb
 ecfc3e46a9d1859ea84000a73225cf648d8a1b5b8412102c6cd9fe8e4fc86532 1046782 apt_0.9.12.1_amd64.deb
 43f22c65f99bd560da8c67e5e9cac7e051f87971b804b924b1ca5c4587f5c7f1 179576 libapt-pkg-dev_0.9.12.1_amd64.deb
 f3094c83e0a374c086826dbc7318f343d671431bfe92bf54f14866dc6eab43ec 345514 apt-utils_0.9.12.1_amd64.deb
 25c0ee11e9500f13d8574f346d13f68dc9bcd9545f0db01b5a432088c6a7f17f 124666 apt-transport-https_0.9.12.1_amd64.deb
Files: 
 0b42dbc22db642f9e64beadbe6f1c582 1617 admin important apt_0.9.12.1.dsc
 d0cf3377f6a99ca7a9ce9df0977cf3a5 3420713 admin important apt_0.9.12.1.tar.gz
 4f0ad16d141dfe2d858141130672426e 266658 doc optional apt-doc_0.9.12.1_all.deb
 189313279c0040c3dc087a15ef297736 555648 doc optional libapt-pkg-doc_0.9.12.1_all.deb
 383a2cf6d61a51ec7da6f4510881265b 717776 libs important libapt-pkg4.12_0.9.12.1_amd64.deb
 30a2d923a96922a8c5292a120739db10 157870 libs important libapt-inst1.5_0.9.12.1_amd64.deb
 43d47bea071f4036dc72750914fca256 1046782 admin important apt_0.9.12.1_amd64.deb
 7465b327d37331dc1b4ba1b41a23875f 179576 libdevel optional libapt-pkg-dev_0.9.12.1_amd64.deb
 4fae40ccdf35be880260d50caac56423 345514 admin important apt-utils_0.9.12.1_amd64.deb
 64e3833453eca32789ec8dad5c70911a 124666 admin optional apt-transport-https_0.9.12.1_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.15 (GNU/Linux)

iEYEARECAAYFAlJdmGQACgkQliSD4VZixzSqkQCfb3pg8FTLAnMVYfRQO4A7TV+B
+zwAn0ql88PaoE4DLI7NPFxkSe+IV0Cb
=cR5u
-----END PGP SIGNATURE-----

--- End Message ---

Reply to: