Hi,

Guillem Jover (2019-08-09):
> After some thought, I think a local aptitude-specific wrapper might be
> even better and obviates the question of whether dpkg-parsechangelog
> should be moved or not. :)

FWIW, this makes sense to me.

> See the attached patch, which has seen no testing, but can do that
> once and if it is deemed a good path forward.

Thanks a lot!

Dear Aptitude maintainers, FYI this is now the last remaining blocker
for the removal of libparse-debian-changelog-perl, so I'm trying
to facilitate the integration of Guillem's patch:

I have applied Guillem's patch and fixed the typo mentioned below, on
top of current sid's 0.8.12-3 (after disabling the build of doc
translations as this makes the package FTBFS for me with current po4a;
same on the Reproducible Builds infra¹), and I did these tests:

 - `aptitude changelog apt` works
 - In the Aptitude UI I could view the changelog of both
   installed and non-installed packages.

I'm not sure if any of those exercises the affected code path.

I'm happy to do more tests if it helps fix this bug, but I would
need some guidance regarding what/how to test :)

> -dist_bin_SCRIPTS = aptitude-create-state-bundle aptitude-run-state-bundle
> +dist_bin_SCRIPTS = apitude-changelog-parser \
> +     aptitude-create-state-bundle aptitude-run-state-bundle

Typo: apitute-changelog-parser → aptitude-changelog-parser

[1] 
https://tests.reproducible-builds.org/debian/rb-pkg/unstable/armhf/aptitude.html

Reply via email to