Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.bugs.dist > #1171284

Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008

From Jonathan Kamens <jik@kamens.us>
Newsgroups linux.debian.bugs.dist
Subject Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008
Date 2023-10-09 21:40 +0200
Message-ID <Hn4gF-eHW4-3@gated-at.bofh.it> (permalink)
References (2 earlier) <Hn197-eGaP-3@gated-at.bofh.it> <Hn1Vv-eGFY-7@gated-at.bofh.it> <Hn2Rz-eHeS-3@gated-at.bofh.it> <Hn0Zr-eG7O-3@gated-at.bofh.it> <Hn2Rz-eHeS-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

OK, this will be fixed in 4.1. Description of the bug and fix, copied 
from the commit message:

     Bug:
     
     * Main package a has both changelog and NEWS.
     * Subpackage a-sub has identical changelog but no NEWS.
     * Both a and a-sub version 1 are installed but not in database.
     * apt goes to upgrade a and a-sub to version 2.
     * apt-listchanges parses a-sub first, records installed entries under
       package a instead of a-sub, since we were using the package name in
       the changelog entry to determine where in the seen DB to record
       entries.
     * apt-listchanges parses a, sees that there are entries for it in
       database, therefore does not parse installed entries so does not
       notice NEWS entry in installed package.
     * As a result, apt-listchanges displays NEWS entry for a when it
       shouldn't have.
     
     Fix:
     
     * Associate entries with binary package names, not the package name in
       the changelog entry itself.
     * When checking if an entry has already been seen, check across all
       packages, not just the binary package being parsed. To facilitate
       this, the seen database maintains a merged dict of all checksums.

I added a unit test for this case which now passes, and all other unit 
tests continue to pass with the change described above.

Additional FYI comments below.

On 10/9/23 14:05, Axel Beckert wrote:
> Anything I can help? This is a Sid installation running more or less
> permanently (besides reboots :-) since May 2016. So the
> apt-listchanges database might have seen a few packages. Then again,
> it seems rather short for > 14'000 installed packages.
Note that the database is entirely replaced when upgraded from pre-4.0 
to 4.x because its format and what we're storing in it are completely 
different.
> BTW, while trying to figure out where that db could be I noticed that
> despite 4.0 is installed according to "dpkg -l apt-listchanges", the
> tool itself contains a different version number:
>
>    ~ → head -2 /usr/bin/apt-listchanges
>    #!/usr/bin/python3
>    # EASY-INSTALL-ENTRY-SCRIPT: 'apt-listchanges==3.27','console_scripts','apt-listchanges'

Thanks, fixed this as well.

   jik


Back to linux.debian.bugs.dist | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008 Axel Beckert <abe@debian.org> - 2023-10-09 18:10 +0200
  Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008 Russ Allbery <rra@debian.org> - 2023-10-09 18:20 +0200
    Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008 Jonathan Kamens <jik@kamens.us> - 2023-10-09 19:10 +0200
      Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008 Axel Beckert <abe@debian.org> - 2023-10-09 20:10 +0200
        Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008 Jonathan Kamens <jik@kamens.us> - 2023-10-09 21:40 +0200
          Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008 Axel Beckert <abe@debian.org> - 2023-10-09 21:50 +0200
      Bug#1053725: apt-listchanges: Shows NEWS for package tor from 2008 Russ Allbery <rra@debian.org> - 2023-10-09 20:10 +0200

csiph-web