Path: csiph.com!weretis.net!feeder8.news.weretis.net!news.samoylyk.net!gothmog.csi.it!bofh.it!news.nic.it!robomod From: Emmanuel Arias Newsgroups: linux.debian.maint.python Subject: Re: Package review python-ecs-logging Date: Wed, 05 Mar 2025 20:50:01 +0100 Message-ID: References: X-Original-To: =?iso-8859-1?Q?Sch=F6ke=2C?= Karsten X-Mailbox-Line: From debian-python-request@lists.debian.org Wed Mar 5 19:49:48 2025 Old-Return-Path: X-Amavis-Spam-Status: No, score=-113.447 tagged_above=-10000 required=5.3 tests=[BAYES_00=-2, BODY_8BITS=1.5, DKIMWL_WL_HIGH=-0.438, 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="8ErnyoPd4Uf8+8yy" Content-Disposition: inline X-Debian-User: eamanu X-Mailing-List: archive/latest/22875 List-ID: List-URL: List-Archive: https://lists.debian.org/msgid-search/Z8iqxscx5T3h_GLR@debian Approved: robomod@news.nic.it Lines: 72 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Cc: "debian-python@lists.debian.org" X-Original-Date: Wed, 5 Mar 2025 16:49:26 -0300 X-Original-Message-ID: X-Original-References: <9547ff7d8d47444ebde30ba2c3fd415f@geobasis-bb.de> Xref: csiph.com linux.debian.maint.python:16761 --8ErnyoPd4Uf8+8yy Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hello! Thank you for your work! I took a look to your package. I leave you some co= mments - d/control: wrap-and-sort can be excecuted. - d/control: I guess Build-Depends-Indep is not necessary - d/control: You can use autopkgtest-pkg-pybuild as Testsuite, instead of a= utopkgtest-pkg-python. And unittests will run in autopkgtests. - d/control: I gues you don't have permissions to create projects under pyt= hon-team? That's why Vcs-* does not exists. - d/rules: You can remove the template comments - d/patches/001_disable_elasticapm_tests.patch: The patch descriptions has = part of d/changelog, I think is a typo?. Let me know, if you made some changes to review it again :-)=20 Cheers! On Wed, Mar 05, 2025 at 11:59:10AM +0000, Sch=C3=B6ke, Karsten wrote: > Hello, >=20 > we continue to work on the dependencies in the elasticsearch environment. > I have finished packaging ecs-logging, could someone review my work and i= f OK, sponsor it? > https://salsa.debian.org/schoekek-guest/python-ecs-logging > This is a new dependency on es_client, this in turn on elasticsearch-cura= tor... >=20 > If it's OK, I would push it to the DPT repo. >=20 > Many thanks and best regards > Karsten --=20 cheers, Emmanuel Arias =E2=A2=80=E2=A3=B4=E2=A0=BE=E2=A0=BB=E2=A2=B6=E2=A3=A6=E2=A0=80 =E2=A3=BE=E2=A0=81=E2=A2=A0=E2=A0=92=E2=A0=80=E2=A3=BF=E2=A1=81 eamanu@de= bian.org =E2=A2=BF=E2=A1=84=E2=A0=98=E2=A0=B7=E2=A0=9A=E2=A0=8B=E2=A0=80 OpenPGP: = 13796755BBC72BB8ABE2AEB5 FA9DEC5DE11C63F1 =20 =E2=A0=88=E2=A0=B3=E2=A3=84 --8ErnyoPd4Uf8+8yy Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEE3lnVbvHK7ir4q61+p3sXeEcY/EFAmfIqsUACgkQ+p3sXeEc Y/G4vA/+JWnRtajClsJx/iwVIymMVIGtedAPptWsmFl126XTgN6rxqaPgLWN8n/G Ehxs0QPKUWLitOOk8u3imYfRJgmYJbeYHoDwnMZILMgQhDtxKSur+d3Ajkkozd5M KrM0cmQXlP/bzlLXORvkgW4ocdsfXpBmm8JdtDCT0nLBj0OSu+wQa1vsmhUc8oCN 8piS5WOk9pBpkC4kemyEzy0J5rOIIvCmrouJMpxJ3L3EUrIGrKFtpzPjuESOM3a1 9zxDTfurrtnvaYKwVMYRp6j9NFRfnQhtjqs8VhTytHybQBgdXr7lVQGg9WZGL9au IMMOwd5KiZqEduK1jFGSCVgelsdDgq4mLwhpX0ARpKiJl7M0Xkw9LSaCH0TO4zyj epCr9RoZNgjgM4t09N/9JMrUje601yK1rtLCqX3dFt649wzR5roWa6TWazWX92iy JIdCRjQAJ9JYFno+0MRGUHmU1ipy+2Pkv1vCfxmn3vkP1BZBVbCGZeCmzi3Dv8wf tyFE8ikatVCcj0oj13pA+3vLCJBWAmlZkz5h82GOrzonrUZ+PX/dwvQKKVUZk2GD 6xVHrtIr04LEBj69mLPaAYuYsmiCBNwqcmMVL0ykpEx71FPVE1mGNQ4D2/SZWnOl 0qYFybmgf9rSLjZQ9rEmvbawGiJYYKzJSQ9AZH9FxbeTt65dfYA= =IMu5 -----END PGP SIGNATURE----- --8ErnyoPd4Uf8+8yy--