Path: csiph.com!weretis.net!feeder6.news.weretis.net!feeder9.news.weretis.net!feeder8.news.weretis.net!fu-berlin.de!news.servidellagleba.it!bofh.it!news.nic.it!robomod From: =?utf-8?Q?=C3=89tienne?= Mollier Newsgroups: linux.debian.maint.python Subject: Re: py7zr-related python packages was updated Date: Sun, 28 Apr 2024 17:20:01 +0200 Message-ID: References: X-Original-To: yokota X-Mailbox-Line: From debian-python-request@lists.debian.org Sun Apr 28 15:17:26 2024 Old-Return-Path: X-Amavis-Spam-Status: No, score=-116.864 tagged_above=-10000 required=5.3 tests=[BAYES_00=-2, DKIMWL_WL_HIGH=-2.355, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, LDO_WHITELIST=-5, PGPSIGNATURE=-5, RCVD_IN_DNSWL_MED=-2.3, UNPARSEABLE_RELAY=0.001, USER_IN_DKIM_WELCOMELIST=-0.01, USER_IN_DKIM_WHITELIST=-100] autolearn=ham autolearn_force=no MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="h5XrRCFe6KXj6i6h" Content-Disposition: inline X-Debian-User: emollier X-Mailing-List: archive/latest/21824 List-ID: List-URL: List-Archive: https://lists.debian.org/msgid-search/Zi5obTpfZgvahrv9@fusion.emlwks999.eu Approved: robomod@news.nic.it Lines: 63 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Cc: debian-python@lists.debian.org X-Original-Date: Sun, 28 Apr 2024 17:17:01 +0200 X-Original-Message-ID: X-Original-References: Xref: csiph.com linux.debian.maint.python:15795 --h5XrRCFe6KXj6i6h Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Yokota, yokota, on 2024-04-28: > > * https://salsa.debian.org/python-team/packages/python-pyppmd I reviewed through python-pyppmd and my remark are somewhat similar to packages I previously saw. 1. The debian/changelog misses the introduction of the patch to fix build failures on non-amd64 CPU architectures. 2. Introduction of autopkgtest-pkg-pybuild instead of -python would render the Testsuite non-superficial. This is not mandatory, but it is a low hanging fruit that would bring substancial quality to the package. I note the test is running at build time for this one, which is good. :) 3. I suggest again dh-sequence-python3 instead of dh-python. All that being said, the package looks to me in good shape, and I will have no objections to upload it once the changelog matches all the changes since 1.1.0+ds-1. I'll have a look at python-brotlicffi next, but I don't expect to have anything new to say. Don't hesitate to apply my existing feedback and suggestions on this package if I don't follow up explicitly on it, and to ping again once you think it is ready for upload. Have a nice day, :) --=20 .''`. =C3=89tienne Mollier : :' : pgp: 8f91 b227 c7d6 f2b1 948c 8236 793c f67e 8f0d 11da `. `' sent from /dev/pts/0, please excuse my verbosity `- on air: Future Crew - Second Reality (part II) --h5XrRCFe6KXj6i6h Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEj5GyJ8fW8rGUjII2eTz2fo8NEdoFAmYuaG0ACgkQeTz2fo8N EdpztA/6AnMe8EZGtXm7P857wabwH6Ni92bv8Tq9iGYtg+3D7YYwSbhyxeg0M2Z/ f1fDnj6HrK1rvXRm5I0u8sE+s9UB1J774B9KE6DHmSjf4bXj+O+fHF5Sfh7hT5iY ZW9jpRBhz1J7rBsDQxYlJdfxAZNw7xtsPUeCTRwc0TFSiuBEcgwdBoviQ9poU2yr o3pKDbaqjcdXKsTwvB9AJQHEZC2QZJGChMMggoWZB9+LgRYtt+pxzs8XoZv9ATrt cGiU9OwL9jRjR0T/y1oLuFiM/vNkm5+9NoPROdIZlVD95/W09LmA88t3rNMANEJO UeyfKT+5Y94sVeOFFUbzMAzri6PKwvjCQoWmUCux8ZTPs5ZrDe0Cs3Xx2hQYCt8j YBkii6+lDiLAZB/Z0q/KPWZ5+Fy76vta8myPDDuMAYMxozpkuBPEPngx0sdRqDZj 9/ZKx340RS+LExVUqtX00Gp/0X+6SwAGf9vZ7V1j4VefgC/6fxHtZWLHcuZOmswG umAQSuLBNrA/V6RHqsWwNRHCBd+tj1BnpCu/04nQAvncrLeTwgLJFkQHp/VwAinu WdHxlMm1FWgzgj5G+h64RcGubMudDYdWfuKH5u9dprqtzlRCatQIL4BmoP53FiBf tezlBd+0U6fDWD2OPDE2M+0CuQJFioz02h2/poBz79cOc7INswo= =89Rs -----END PGP SIGNATURE----- --h5XrRCFe6KXj6i6h--