<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Hurray!</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Also, phew! Sorry for the inconvenience.</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
I will note that this "package" is, as already said, not really a package but a collection of scripts, and doesn't have any tests. It's also a bit odd in having its modules not have the packages match the full pathnames.</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
However, it's rare that "proper" module tests actually test scripts, so that strategy wouldn't have caught this here. It's probably worth noting at the top of the README that the mode of use isn't via a Debian package, but something that gets
<code>git pull</code>ed and just used; the usual packaging apparatus gives a false sense of security that things will get checked before affecting production.</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Best regards,</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Ed</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<hr style="display: inline-block; width: 98%;">
<div dir="ltr" id="x_divRplyFwdMsg"><span style="font-family: Calibri, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);"><b>From:</b> Laura Arjona Reina <larjona@debian.org><br>
<b>Sent:</b> 19 November 2024 5:26 PM<br>
<b>To:</b> Ed . <ej_zg@hotmail.com>; 805378@bugs.debian.org <805378@bugs.debian.org>; Holger Wansing <hwansing@mailbox.org>; Andreas Tille <tille@debian.org>; Damyan Ivanov <dmn@debian.org>; debian-perl@lists.debian.org <debian-perl@lists.debian.org>; Package
Salvaging Team <team+salvage@tracker.debian.org>; Debian l10n development team <debian-l10n-devel@lists.alioth.debian.org><br>
<b>Subject:</b> Re: [Debian-l10n-devel] Bug#805378: Undefined subroutine errors (spider-bts) Was: [Help] Re: dl10n-check: Can't locate Debian/Pkg/DebSrc.pm in @INC</span>
<div> </div>
</div>
<div style="font-size: 11pt;">Hello Ed<br>
<br>
El 19/11/24 a las 18:14, Ed . escribió:<br>
> Hi all,<br>
><br>
> Unfortunately it was me that broke it in commit a985438ed248ff54383a9998bd7fd6293c628584 -
<a href="https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10" id="OWAcaa23f06-113c-b917-00a3-0111e74778a4" class="OWAAutoLink" data-auth="NotApplicable">
https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10</a> fixes it.<br>
> [<a href="https://salsa.debian.org/assets/twitter_card-570ddb06edf56a2312253c5872489847a0f385112ddbcd71ccfa1570febab5d2.jpg]<https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10" id="OWA0f9fd444-d291-f395-04ef-e42568baf930" class="OWAAutoLink" data-auth="NotApplicable">https://salsa.debian.org/assets/twitter_card-570ddb06edf56a2312253c5872489847a0f385112ddbcd71ccfa1570febab5d2.jpg]<https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10</a>><br>
> Fix the =encoding commenting/disabling the `package` statement (!10) · Merge requests · Debian i18n and l10n team / dl10n · GitLab<<a href="https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10" id="OWA98f0239c-e8a5-8102-49e4-04145d065833" class="OWAAutoLink" data-auth="NotApplicable">https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10</a>><br>
> Debian i18n - l10n development - dl10n robots<br>
> salsa.debian.org<br>
<br>
Thanks. I was going commit by commit and found out this was the<br>
responsible and was comparing Html.pm with Utils.pm to try to fix it<br>
when your mail arrived.<br>
<br>
> Explanation: I added the =encoding line, and didn't notice that in Html.pm (unlike the other two .pm files) there was substantive code (a package statement) instead of immediately a POD statement (=head1). I have fixed this more long-term by moving that package
statement to below all the POD, as in the other two.<br>
><br>
> You can check this locally by doing perl -Ilib dl10n-html with/without the change: without, you'll see the error Laura reported, with, you'll see it gets further, saying it can't find data.<br>
<br>
I've reviewed the MR and just merged it, ran a "git pull" in tye.d.o and<br>
the spiderbts scripts works again, thanks!<br>
<br>
Kind regards<br>
<br>
<br>
> ________________________________<br>
> From: Laura Arjona Reina <larjona@debian.org><br>
> Sent: 19 November 2024 2:38 PM<br>
> To: Holger Wansing <hwansing@mailbox.org>; Andreas Tille <tille@debian.org>; Damyan Ivanov <dmn@debian.org>; debian-perl@lists.debian.org <debian-perl@lists.debian.org>; Package Salvaging Team <team+salvage@tracker.debian.org>; 805378@bugs.debian.org <805378@bugs.debian.org>;
Debian l10n development team <debian-l10n-devel@lists.alioth.debian.org><br>
> Subject: Re: Undefined subroutine errors (spider-bts) Was: [Debian-l10n-devel] [Help] Re: dl10n-check: Can't locate Debian/Pkg/DebSrc.pm in @INC<br>
><br>
><br>
> Hello<br>
><br>
> Thanks for the quick answer.<br>
><br>
> El 19/11/24 a las 15:16, gregor herrmann escribió:<br>
><br>
> On Tue, 19 Nov 2024 15:01:27 +0100, Laura Arjona Reina wrote:<br>
><br>
><br>
><br>
> Looking at the log, it says:<br>
><br>
> Undefined subroutine &Html::html called at<br>
> /srv/i18n.debian.org//dl10n/git/dl10n-html line 140.<br>
><br>
> We have the file in dl10n/git/lib/Debian/L10n/Html.pm that includes the<br>
> subroutine hmtl() and it was working without issues. I think that now Perl<br>
> in tye.d.o cannot find it but I don't know why, because the cron job (script <a href="https://salsa.debian.org/l10n-team/dl10n/-/blob/master/cron/spiderbts?ref_type=heads" id="OWAc0113b96-a34f-d2ba-e8ee-113a15340159" class="OWAAutoLink" data-auth="NotApplicable">
https://salsa.debian.org/l10n-team/dl10n/-/blob/master/cron/spiderbts?ref_type=heads</a><br>
> ) gets the configuration from file dl10n.conf as the other scripts.<br>
><br>
><br>
><br>
> Was perl upgraded?<br>
><br>
> No AFAIK, we have 11.11 in tye, and perl 5.32.1-4+deb11u4<br>
><br>
><br>
><br>
><br>
> Anyway, looks indeed like<br>
> ./dl10n-html<br>
> doesn't find<br>
> ./lib<br>
><br>
> <guess><br>
> I'd try to add<br>
> use lib '.';<br>
> at the top of dl10n-html, e.g. after "use POSIX qw(strftime);"<br>
> in <a href="https://salsa.debian.org/l10n-team/dl10n/-/blob/master/dl10n-html?ref_type=heads#L32" id="OWA14673676-ef98-cadf-ed8f-bc9117d9f586" class="OWAAutoLink" data-auth="NotApplicable">
https://salsa.debian.org/l10n-team/dl10n/-/blob/master/dl10n-html?ref_type=heads#L32</a><br>
><br>
><br>
> I tried this (manually editing the file in tye.debian.org) and got the same error, also tried to write the line more on top, in L5, with no changes.<br>
><br>
><br>
> or call dl10n-html from the cronjob as "perl -I<full path to the<br>
> lib/directory>, e.g.<br>
> PERL -I$DL10NDIR/lib $DL10NDIR/dl10n-html --sdb …<br>
> in <a href="https://salsa.debian.org/l10n-team/dl10n/-/blob/master/cron/spiderbts?ref_type=heads#L46" id="OWA75cddce4-67fe-ecf4-bfd2-656ecaab6bf2" class="OWAAutoLink" data-auth="NotApplicable">
https://salsa.debian.org/l10n-team/dl10n/-/blob/master/cron/spiderbts?ref_type=heads#L46</a><br>
> </guess><br>
><br>
><br>
> I also tried this, editing the spiderbts file in tye.debian.org (used "perl" instead of PERL) obtaining the same error:<br>
><br>
> Undefined subroutine &Debian::L10n::Html::html called at /srv/i18n.debian.org//dl10n/git/dl10n-html line 140.<br>
><br>
> (I also changed the message written in the .log file to be sure that the system was running my modified script).<br>
><br>
> Kind regards,<br>
><br>
> --<br>
> Laura Arjona Reina<br>
> <a href="https://wiki.debian.org/LauraArjona" id="OWAc622e5d5-61cd-6ef7-3015-87c8bbb75d3c" class="OWAAutoLink" data-auth="NotApplicable">
https://wiki.debian.org/LauraArjona</a><br>
><br>
><br>
> _______________________________________________<br>
> Debian-l10n-devel mailing list<br>
> Debian-l10n-devel@alioth-lists.debian.net<br>
> <a href="https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/debian-l10n-devel" id="OWA9511ed1b-ece3-efee-677d-087bbe6b4c7c" class="OWAAutoLink" data-auth="NotApplicable">
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/debian-l10n-devel</a><br>
<br>
--<br>
Laura Arjona Reina<br>
<a href="https://wiki.debian.org/LauraArjona" id="OWAd4877e8a-909b-2505-e9d7-a7c53abb0e5c" class="OWAAutoLink" data-auth="NotApplicable">https://wiki.debian.org/LauraArjona</a><br>
<br>
</div>
</body>
</html>