Re: [RFS] python-pweave
Hi, uploaded! Regards Anton Am So., 21. Juni 2020 um 19:56 Uhr schrieb Nilesh Patra : > > Hi, > python-pweave has a RC bug: #962104 filed against it. I've fixed this > My changes are pushed here[1]. > Needs review and sponsorship. > > [1]: https://salsa.debian.org/science-team/python-pweave > > Thanks and regards > Nilesh
[RFS] python-pweave
Hi, python-pweave has a RC bug: #962104 filed against it. I've fixed this My changes are pushed here[1]. Needs review and sponsorship. [1]: https://salsa.debian.org/science-team/python-pweave Thanks and regards Nilesh
Re: [RFS] python-pweave
On 27.02.20 10:35, Nilesh Patra wrote: > Done. Can you please review+upload if everything looks OK? Uploaded, you can go ahead and tag it.
Re: [RFS] python-pweave
On Thu, 27 Feb 2020, 14:36 Christian Kastner, wrote: > On 27.02.20 06:17, Nilesh Patra wrote: > > I fixed python-pweave's currently failing autopkgtests. > > Build+autopkgtests pass, and also made it lintian clean. > > I have pushed my changes to the team repo here[1]. > > Needs review and sponsorship. > > Looks good! Please add/change the following minor issues: > > d/changelog: > * In addition to switching from debhelper to -compat, please > mention the switch from level 10 to 12 > * With "Fix pandoc flags", please mention that this was done > by adding a patch to upstream. My first read was that you > changed something in debian/rules. > > d/copyright: > * The first stanza should be extended from 2013 to 2013-2016 > > d/test/control: mixes indentation spaces/tabs (last line) > > Please fix these; I will upload right after. > Done. Can you please review+upload if everything looks OK? > > Other general advice: the package is maintained with git-buildpackage. > With such packages, maintaining patches to upstream with ``gbp pq`` has > the added advantage that it automatically adds a number of DEP-3 [1] > relevant fields to the header. > > Speaking of DEP-3, I would have added a 'Bug-Debian' line linking the > patch to #897103; but that's not required, just good practice. > > [1] https://dep-team.pages.debian.net/deps/dep3/ Noted thanks! Shall incorporate in future. > >
Re: [RFS] python-pweave
On 27.02.20 06:17, Nilesh Patra wrote: > I fixed python-pweave's currently failing autopkgtests. > Build+autopkgtests pass, and also made it lintian clean. > I have pushed my changes to the team repo here[1]. > Needs review and sponsorship. Looks good! Please add/change the following minor issues: d/changelog: * In addition to switching from debhelper to -compat, please mention the switch from level 10 to 12 * With "Fix pandoc flags", please mention that this was done by adding a patch to upstream. My first read was that you changed something in debian/rules. d/copyright: * The first stanza should be extended from 2013 to 2013-2016 d/test/control: mixes indentation spaces/tabs (last line) Please fix these; I will upload right after. Other general advice: the package is maintained with git-buildpackage. With such packages, maintaining patches to upstream with ``gbp pq`` has the added advantage that it automatically adds a number of DEP-3 [1] relevant fields to the header. Speaking of DEP-3, I would have added a 'Bug-Debian' line linking the patch to #897103; but that's not required, just good practice. [1] https://dep-team.pages.debian.net/deps/dep3/
[RFS] python-pweave
Hi, I fixed python-pweave's currently failing autopkgtests. Build+autopkgtests pass, and also made it lintian clean. I have pushed my changes to the team repo here[1]. Needs review and sponsorship. [1]: https://salsa.debian.org/science-team/python-pweave Thanks and regards Nilesh