Your message dated Mon, 17 Oct 2011 12:47:08 +0000
with message-id <e1rfmay-0001lc...@franck.debian.org>
and subject line Bug#645309: fixed in debpartial-mirror 0.3.0
has caused the Debian Bug report #645309,
regarding debpartial-mirror: cannot handle duplicate entries in Packages
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 ow...@bugs.debian.org
immediately.)


-- 
645309: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=645309
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: debpartial-mirror
Version: 0.2.99
Severity: grave

(Normally important, but bumped to RC because it's breaking for
security updates on squeeze and that needs to not happen)

If a Packages file contains two entries for the same package name,
debpartial-mirror will die with this exception:

Traceback (most recent call last):
 File "/usr/bin/debpartial-mirror", line 177, in <module>
   main()
 File "/usr/bin/debpartial-mirror", line 169, in main
   controller.executeCommand(cmnd)
 File "/usr/lib/pymodules/python2.6/debpartial_mirror/Controller.py", line 34, 
in executeCommand
   self._commands.get(commandName)()
 File "/usr/lib/pymodules/python2.6/debpartial_mirror/Controller.py", line 38, 
in doAll
   if self._load():
 File "/usr/lib/pymodules/python2.6/debpartial_mirror/Controller.py", line 72, 
in _load
   if not b.load():
 File "/usr/lib/pymodules/python2.6/debpartial_mirror/Backend.py", line 111, in 
load
   return self._dists.load()
 File "/usr/lib/pymodules/python2.6/debpartial_mirror/Dists.py", line 186, in 
load
   processTagFile(index_filename, addPackage)
 File "/usr/lib/pymodules/python2.6/debpartial_mirror/Dists.py", line 487, in 
processTagFile
   sectionHandler(section)
 File "/usr/lib/pymodules/python2.6/debpartial_mirror/Dists.py", line 182, in 
addPackage
   pkglist.add(package)
 File "/usr/lib/pymodules/python2.6/cdd/PackageList.py", line 167, in add
   raise PackageAlreadyExists, package['Package']
cdd.PackageList.PackageAlreadyExists

Trivial patch follows which only mirrors the most recent version
(which is probably what you wanted, and anyway much better than
failing).

diff -b -x debian -x DEBIAN -x build -ruN 
debpartial-mirror-0.2.99//debpartial_mirror/Dists.py 
/home/asuffield/src/debpartial-mirror-0.2.99//debpartial_mirror/Dists.py
--- debpartial-mirror-0.2.99//debpartial_mirror/Dists.py        2010-10-22 
21:58:37.000000000 +0100
+++ /home/asuffield/src/debpartial-mirror-0.2.99//debpartial_mirror/Dists.py    
2011-10-14 10:37:56.207266435 +0100
@@ -179,6 +179,14 @@
                        def addPackage(section):
                                package = pkg(section)
                                package.releaseInfo = releaseInfo
+                                name = package['Package']
+                                if name in pkglist:
+                                        oldver = pkglist[name]['Version']
+                                        newver = package['Version']
+                                        if apt_pkg.version_compare(oldver, 
newver) < 1:
+                                                pkglist.remove(name)
+                                                pkglist.add(package)
+                                else:
                                pkglist.add(package)
 
                        index_filename = os.path.join(self._filesystem.base(), 
file)


Right now, security.debian.org has duplicates on amd64 for these packages:
Package: openjdk-6-doc
Package: openjdk-6-jre-lib
Package: openjdk-6-source

Thijs Kinkhorst observed that this is because openjdk is hard to build
on all architectures and they still don't have the latest update on
powerpc and sparc, and suggested that this is just something that will
happen from time to time.

Also it's probably going to happen all the time for sid. I'm surprised
this wasn't noticed and fixed before now.



--- End Message ---
--- Begin Message ---
Source: debpartial-mirror
Source-Version: 0.3.0

We believe that the bug you reported is fixed in the latest version of
debpartial-mirror, which is due to be installed in the Debian FTP archive:

debpartial-mirror_0.3.0.dsc
  to main/d/debpartial-mirror/debpartial-mirror_0.3.0.dsc
debpartial-mirror_0.3.0.tar.gz
  to main/d/debpartial-mirror/debpartial-mirror_0.3.0.tar.gz
debpartial-mirror_0.3.0_all.deb
  to main/d/debpartial-mirror/debpartial-mirror_0.3.0_all.deb



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 645...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Otavio Salvador <ota...@debian.org> (supplier of updated debpartial-mirror 
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 ftpmas...@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Mon, 17 Oct 2011 10:42:20 -0200
Source: debpartial-mirror
Binary: debpartial-mirror
Architecture: source all
Version: 0.3.0
Distribution: unstable
Urgency: low
Maintainer: Otavio Salvador <ota...@debian.org>
Changed-By: Otavio Salvador <ota...@debian.org>
Description: 
 debpartial-mirror - tools to create partial Debian mirrors
Closes: 640628 645309
Changes: 
 debpartial-mirror (0.3.0) unstable; urgency=low
 .
   [ Martin Fuzzey ]
   * Apply patch from Pino Toscano to user errno module rather than hardcoded 
constants (Closes: #640628)
   * Apply patch from Andrew Suffield to use most recent of multiple versions 
(Closes: #645309)
Checksums-Sha1: 
 f6b5c88de67d5d971b008fd305a04af056dcad41 1713 debpartial-mirror_0.3.0.dsc
 45bec7c3da77303533d8af7ede1a866071489bd8 104512 debpartial-mirror_0.3.0.tar.gz
 82beba4b12a870e45b0b60dcdc914b980a35d8be 30698 debpartial-mirror_0.3.0_all.deb
Checksums-Sha256: 
 8b33999d93248b118dd4ecb969d3062fb57cb024a3389e707aebfebb2811794b 1713 
debpartial-mirror_0.3.0.dsc
 e9106803ff95d8b58c2024d2f55551b332b52ac316629ef1569a94b006125407 104512 
debpartial-mirror_0.3.0.tar.gz
 c831b7cc6a55ad397e3154b63d7748825c16b6e0a513be298e36cc4c89c6dcab 30698 
debpartial-mirror_0.3.0_all.deb
Files: 
 d1027aa31e0eabae0bff41abc0d014b1 1713 net extra debpartial-mirror_0.3.0.dsc
 74c2ab2608bebd4f49f0fceeb83490d2 104512 net extra 
debpartial-mirror_0.3.0.tar.gz
 5ff5ad48dbeaa08df75e7f804c32308a 30698 net extra 
debpartial-mirror_0.3.0_all.deb

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

iQIcBAEBCAAGBQJOnCMKAAoJEOtw/vPN/G5PR1YQAL82cc8ABxNzVdeqotDoZsyi
0/QyPoi3uzkVoWCy+LNAjC5qWabzLGKDbyYwoyeRy9Nd4qGDN0ua5yvGC26wDPc7
Gg6sM9OoCPQNeeLjBIM893QGE0WdEOwBuMTZ89NvY2OEP5O5SRHnNAxE6B9IIC74
ERjPUwwCG1UCrNNbMDSuuhPZnNDURXgBktvtDbZsyJYW/gsWlY9yq8gENje2m+e3
pIl2+YuIz92Rm+DNM+zuVsfH4fF3jsLHSBYJNhZFRumxIlPFsn8gVbPSmS59kl7F
Hy9/+5D7NLxGvdAxdDVqo++6nGupRgCFVkS5ZK6ZDBVaxyP2iEai44zpWVhkHSMr
n1fPx/3zgY0pVsk8U5XTIk7DE48S47YMrl4YHyelEwc3atD5dYeEly/Cu/jFKSp1
mP5KteKk0Phed0dC6DL0Zw+Vy19yGWKHfwlP0a7CoRgsnm+g/Nl030zWne+eeokB
YC/XDN5mp826PRqPoaE6gUoMUYTZtcLQldbQmV4NQ3ztJrkktxgeMpsGJ5sw8uh6
G0mWH+kCzIQ0dzr7JoSvFkPKiK4cvzV+PxhTPUlPNi7onkhqFd2S8CcyX/V1fFv6
ur/jiQKIiDRBBiqsF3M2lt85lbg6XC2Xr6G1Mf2EXxWsSJKcp0TgC9lzdcRnVttQ
QcVBzGyxEf0BQVOpDqYr
=cSnK
-----END PGP SIGNATURE-----



--- End Message ---

Reply via email to