[Aptitude-devel] Bug#967911: aptitude: aptitude-changelog-parser leaves garbage on the screen

[Guillem Jover]

Hi!

On Tue, 2020-08-04 at 20:56:48 +0200, Guillem Jover wrote:
> Control: tags -1 patch

> On Tue, 2020-08-04 at 20:09:05 +0200, Sven Joachim wrote:
> > Package: aptitude
> > Version: 0.8.13-1+b1
> > Severity: normal
> > X-Debbugs-Cc: Sven Joachim <svenjoac at gmx.de>, Guillem Jover <guillem at debian.org>
> 
> > When viewing the Debian changelog of bash in aptitude's TUI, I saw the
> > screen flash and then error messages from aptitude-changelog-parser
> > appeared scattered across the screen, redrawing the screen with Ctrl-l
> > got rid of these and displayed the changelog properly.
> > 
> > The bash changelog has at least two incorrectly formatted entries for
> > versions 2.01.1-3.1 and 2.01-0 which aptitude-changelog-parser
> > apparently complains about.
 
> > Viewing the changelog from the commandline with "aptitude changelog
> > bash" does not show such a problem, though.
> 
> I guess the TUI is not ignoring stderr, while the CLI invocation is?
> Although that means that if the libdpkg-perl package is not installed,
> there will be no warning helping the user along, so it would be nice
> to get the CLI to not ignore the warnings for consistency?
> 
> In any case, Dpkg::Changelog defaults to printing these parsing
> problems as warnings, but there is no way to pass a «verbose => 0» via
> changelog_parse(). I'll add support for that too.

This is now supported since dpkg 1.20.6, so I'm attaching the updated
patch.

> For the aptitude-changelog-parser, it could be modified for now to
> quiesce late warnings like in the attached patch.

And the old can should then be ignored.

Thanks,
Guillem
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-aptitude-changelog-parser-Quiesce-Dpkg-Changelog-par.patch
Type: text/x-diff
Size: 1020 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/aptitude-devel/attachments/20211109/0521bcba/attachment.patch>