Bug#1005778: pypandoc: diff for NMU version 1.7.2+ds0-1.1
On 2022-02-28 at 15:31:58 -0500, nick black wrote: > agreed, so long as it's not required by most outputs (i.e. you > can run pypandoc without it and it works). Latex is only used for pdf output, and even there it's just the default option, but there is support for other engines as well. If that wasn't the case, I agree that it should be a dependency, but it should be a dependency of pandoc rather than pypandoc. The issue here is that the tests of pypandoc do try to generate a pdf (as it is a pretty common usecase), and thus they do need latex installed, but that's only a requirement for the tests. > alrightie. this was my NM application bug; i guess i will find > another one. have a good day! I'm even more sorry about having wasted your time then :( -- Elena ``of Valhalla'' signature.asc Description: PGP signature
Bug#1005778: pypandoc: diff for NMU version 1.7.2+ds0-1.1
> However, I don't think that adding lmodern to the package Depends is the > right solution, as that would lead to parts of tex (admittely, a small > part, but still) having to be installed on the system, which is not > required by pandoc itself. > The right solution, I believe, is to add lmodern to the tests/control > Depends, and I've just realized that I prepared that in git, but I never > actually did the upload (ups), and was wondering why it wasn't > working. agreed, so long as it's not required by most outputs (i.e. you can run pypandoc without it and it works). > I will do the upload myself in the next few days, since I have an > up-to-date copy of the repo locally (and salsa is down). alrightie. this was my NM application bug; i guess i will find another one. have a good day! signature.asc Description: PGP signature
Bug#1005778: pypandoc: diff for NMU version 1.7.2+ds0-1.1
On 2022-02-28 at 10:20:10 -0500, nick black wrote: > lmodern needed be listed as a regular Depends, not just a > Build-Depends, in order for the autopkgtests to successfully > run. I have added it, and verified that this fixes the > autopkgtests on the testing distribution. Thanks for working on it! However, I don't think that adding lmodern to the package Depends is the right solution, as that would lead to parts of tex (admittely, a small part, but still) having to be installed on the system, which is not required by pandoc itself. The right solution, I believe, is to add lmodern to the tests/control Depends, and I've just realized that I prepared that in git, but I never actually did the upload (ups), and was wondering why it wasn't working. I will do the upload myself in the next few days, since I have an up-to-date copy of the repo locally (and salsa is down). sorry. -- Elena ``of Valhalla'' signature.asc Description: PGP signature
Bug#1005778: pypandoc: diff for NMU version 1.7.2+ds0-1.1
Control: tags 1005778 + patch Control: tags 1005778 + pending Dear maintainer, I've prepared an NMU for pypandoc (versioned as 1.7.2+ds0-1.1) and it will be uploaded by my Account Manager, Sebastian Ramacher. Please feel free to tell me if he should delay it. lmodern needed be listed as a regular Depends, not just a Build-Depends, in order for the autopkgtests to successfully run. I have added it, and verified that this fixes the autopkgtests on the testing distribution. hack on, nick diff -Nru pypandoc-1.7.2+ds0/debian/changelog pypandoc-1.7.2+ds0/debian/changelog --- pypandoc-1.7.2+ds0/debian/changelog 2021-12-29 05:19:10.0 -0500 +++ pypandoc-1.7.2+ds0/debian/changelog 2022-02-28 10:12:41.0 -0500 @@ -1,3 +1,11 @@ +pypandoc (1.7.2+ds0-1.1) unstable; urgency=medium + + * Non-maintainer upload. + * Add dependency on lmodern for runtime to fix autopkgtests. +Closes: #1005778 + + -- Nick Black Mon, 28 Feb 2022 10:12:41 -0500 + pypandoc (1.7.2+ds0-1) unstable; urgency=medium * New upstream release. diff -Nru pypandoc-1.7.2+ds0/debian/control pypandoc-1.7.2+ds0/debian/control --- pypandoc-1.7.2+ds0/debian/control 2021-12-29 05:19:10.0 -0500 +++ pypandoc-1.7.2+ds0/debian/control 2022-02-28 10:10:24.0 -0500 @@ -27,6 +27,7 @@ Package: python3-pypandoc Architecture: all Depends: + lmodern, pandoc, ${misc:Depends}, ${python3:Depends}