Hi Jonathan,

Jonathan Kamens wrote:
> OK, this will be fixed in 4.1.

Yay, thanks!

> 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.

Ouch, that seemed rather non-trivial to figure out and reproduce.

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

Perfect!

> 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.

Ok, wasn't sure how relevant its content is. Just tried to help. :-)

> >    # EASY-INSTALL-ENTRY-SCRIPT: 
> > 'apt-listchanges==3.27','console_scripts','apt-listchanges'
> 
> Thanks, fixed this as well.

Great!

                Regards, Axel
-- 
 ,''`.  |  Axel Beckert <a...@debian.org>, https://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE

Reply via email to