Re: New packages for review

gregor herrmann <[email protected]> Sun, 12 Oct 2025 15:41:05 +0200
Newsgroups gmane.linux.debian.devel.perl
Message-ID <[email protected]>
------------=_1760276468-10014-40
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, 07 Oct 2025 00:21:51 +1300, Andrew Ruthven wrote:

Some rather quick review:

>libwebservice-dropbox-perl
>https://salsa.debian.org/perl-team/modules/packages/libwebservice-dropbox-=
perl

* debian/libwebservice-dropbox-perl.docs can probably be removed (the
   files are in .examples anyway)
* d/control: one of libmodule-build-perl and libmodule-build-tiny-perl
   should be enough (I suppose the -tiny variant)
* I: libwebservice-dropbox-perl: wrong-section-according-to-package-name mi=
sc =3D> perl

>libalgorithm-loops-perl
>https://salsa.debian.org/perl-team/modules/packages/libalgorithm-loops-perl

* Maybe install ex/ as examples?
* d/control: short description should be a noun phrase
* I: libalgorithm-loops-perl source: patch-not-forwarded-upstream [debian/p=
atches/fix-pod.patch]

>libfuture-http-perl
>https://salsa.debian.org/perl-team/modules/packages/libfuture-http-perl

* d/control: short description should be a noun phrase
* version contraints (libexperimental-perl libfuture-perl) seem to be
   satisfied in oldstable
* Maybe add debian/tests/pkg-perl/syntax-skip

>libmoox-role-eventemitter-perl
>https://salsa.debian.org/perl-team/modules/packages/libmoox-role-eventemit=
ter-perl

* Hrm, this wohl copyright/license situation is a bit weird.
   In any case, LICENSE contains Artistic-2.0
* Short and long description are quite terse :)
* ITP bug in d/changelog
* autopkgtest smoke.t fails

>libobject-import-perl
>https://salsa.debian.org/perl-team/modules/packages/libobject-import-perl

* d/copyright: that's unusual:
   "??, Max Maischein <[email protected]>"
   :)
   and in general I only see Zsban Ambrus as a copyright holder
* d/control: no "<!nocheck>" for libmath-bigint-perl?

>libnet-async-websocket-perl
>https://salsa.debian.org/perl-team/modules/packages/libnet-async-websocket=
-perl

* libtest-simple-perl | libtest2-suite-perl -->
   2x "<!nocheck>"
   and probably
   "libtest-simple-perl (>=3D 1.302200) <!nocheck> | libtest2-suite-perl <!=
nocheck>"
* d/control: short description should be a noun phrase

>libwww-mechanize-chrome-perl
>https://salsa.debian.org/perl-team/modules/packages/libwww-mechanize-chrom=
e-perl
>This one isn't ready for upload yet, there are two failing tests. I've
>created tickets upstream for them. However, the package can still be
>reviewed.

* debian/libwww-mechanize-chrome-perl.docs -> can be removed,
   contains examples
* demo/ -> install as examples?
   and, as these are binary files, the usual questions: preferred form
   for modfication, copright/license?
   Ok, I watched one, looks like a harmless screen recording by
   upstream :)
* libfuture-http-perl (>=3D 0.06) --> no version constraint needed
* d/control: description: typo: WWW::Mechianize::Chrome
* libexperimental-perl (>=3D 0.031) -> satisfied in oldstable

(I didn't build this one)

>libselenium-remote-driver-perl
>https://salsa.debian.org/perl-team/modules/packages/libselenium-remote-dri=
ver-perl

* 1.50 is available upstreams :)
* debian/patches/fix-env-path.patch:
   I'd use "#!/usr/bin/perl" and I'd do the fixup in d/rules instead
   of a patch, cf. https://perl-team.pages.debian.net/debhelper.html#Fixing=
_Interpreter_Shebang_Lines
   and many packages
* git mv debian/lintian-overrides debian/libselenium-remote-driver-perl.lli=
ntian-overrides
   (I think future debhelper will enforce the <pkg>.something variant)
* d/copyright: (not checked now)
* The package installs
   ./usr/share/perl5/Selenium/Remote/driver-example.pl
   Is this a good/the right place?
* lib/Selenium/Firefox/webdriver.xpi sounds interesting :)
   And looking into it is even more interesting, e.g. there's a
   Windows .dll in it =E2=80=A6


Cheers,
gregor

--=20
  .''`.  https://info.comodo.priv.at -- Debian Developer https://www.debian=
.org
  : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D  85FA BB3A 6801 8649 =
AA06
  `. `'  Member VIBE!AT & SPI Inc. -- Supporter Free Software Foundation Eu=
rope
    `-=20=20=20

------------=_1760276468-10014-40
Content-Type: application/pgp-signature; name="signature.asc"
Content-Disposition: inline; filename="signature.asc"
Content-Transfer-Encoding: 7bit
Content-Description: Digital Signature

-----BEGIN PGP SIGNATURE-----

iQKTBAEBCgB9FiEE0eExbpOnYKgQTYX6uzpoAYZJqgYFAmjrr/RfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEQx
RTEzMTZFOTNBNzYwQTgxMDREODVGQUJCM0E2ODAxODY0OUFBMDYACgkQuzpoAYZJ
qgbauw/9Fe4ZyrA2Z9q4gRYttg1DhxAT7W5W1G0F5IaGkI5gTw63VDQd3MofxeUW
s6fSh6uwZd0zAtcKr05tFRQ7UY1LTHjYcM7urdCI71lvqlXHqoiEb4DwNNw2y7jw
xAx/b2mMMOLzboI28IKyVrNwFgW3Je3krOoIzFtnT9CgBjTEubDWK3l4Igm9pyhJ
RYmXSrTm/ZRW9WM1wC6McBHtx2d8Qz6+kK+pymZVNnh2DzhVtwJAGA6Ge5v5HZqy
L2OaNZpziIKIdqkCKjwVuriPC4kP5L/yxi6+mrBjSxeR+/vcVfCsupjCkEG2zz5d
OxrWzYbylylyvml+78f6YUJxtbKIgerJTebPWwHIGOl7q6TpB982AnaqoemVUpAb
ytO1Ek2BxPr6U5zJq+MIy69+N2Od21LovtfFOrWsZTye5zGs99eRKrHBj2fPaeRw
EiOSyQHGRI/TIGxp46taUBw1XPRYok9z1vvJk8xAANwajnRiYTlTriASKa5Ue3pz
Hvpyf3oCkaYMA+GMHP4u1jWWsrrPgNwaTECT1hQCH/gJ1+HUC8QzlCaNI1Zn5I8x
w5B0GVscGdoGxzD7n9vYukg4RTDCztG2CKxHHHyZs1GXyCaoMqjaSzOASKB2N7Qj
TOsW8eTiXU5kJtYEtynT01XFlyT76WVyv9L3LmFhX1mgSRKKB24=
=V1JL
-----END PGP SIGNATURE-----

------------=_1760276468-10014-40--