<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);">
Hi all,</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);">
Unfortunately it was me that broke it in commit a985438ed248ff54383a9998bd7fd6293c628584 -
<a href="https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10" id="LPlnk">
https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10</a> fixes it.</div>
<div class="_Entity _EType_OWALinkPreview _EId_OWALinkPreview _EReadonly_1">
<div id="LPBorder_GTaHR0cHM6Ly9zYWxzYS5kZWJpYW4ub3JnL2wxMG4tdGVhbS9kbDEwbi8tL21lcmdlX3JlcXVlc3RzLzEw" class="LPBorder914940" style="width: 100%; margin-top: 16px; margin-bottom: 16px; position: relative; max-width: 800px; min-width: 424px;">
<table id="LPContainer914940" role="presentation" style="padding: 12px 36px 12px 12px; width: 100%; border-width: 1px; border-style: solid; border-color: rgb(200, 200, 200); border-radius: 2px;">
<tbody>
<tr valign="top" style="border-spacing: 0px;">
<td>
<div id="LPImageContainer914940" style="position: relative; margin-right: 12px; height: 160px; overflow: hidden;">
<a target="_blank" id="LPImageAnchor914940" href="https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10"><img id="LPThumbnailImageId914940" alt="" height="160" style="display: block;" width="160" src="https://salsa.debian.org/assets/twitter_card-570ddb06edf56a2312253c5872489847a0f385112ddbcd71ccfa1570febab5d2.jpg"></a></div>
</td>
<td style="width: 100%;">
<div id="LPTitle914940" style="font-size: 21px; font-weight: 300; margin-right: 8px; font-family: wf_segoe-ui_light, "Segoe UI Light", "Segoe WP Light", "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; margin-bottom: 12px;">
<a target="_blank" id="LPUrlAnchor914940" href="https://salsa.debian.org/l10n-team/dl10n/-/merge_requests/10" style="text-decoration: none; color: var(--themePrimary);">Fix the =encoding commenting/disabling the `package` statement (!10) · Merge requests ·
Debian i18n and l10n team / dl10n · GitLab</a></div>
<div id="LPDescription914940" style="font-size: 14px; max-height: 100px; color: rgb(102, 102, 102); font-family: wf_segoe-ui_normal, "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; margin-bottom: 12px; margin-right: 8px; overflow: hidden;">
Debian i18n - l10n development - dl10n robots</div>
<div id="LPMetadata914940" style="font-size: 14px; font-weight: 400; color: rgb(166, 166, 166); font-family: wf_segoe-ui_normal, "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif;">
salsa.debian.org</div>
</td>
</tr>
</tbody>
</table>
</div>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
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.</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);">
You can check this locally by doing <code>perl -Ilib dl10n-html</code> 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.</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 id="appendonsend"></div>
<div style="font-family:Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Laura Arjona Reina <larjona@debian.org><br>
<b>Sent:</b> 19 November 2024 2:38 PM<br>
<b>To:</b> 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>
<b>Subject:</b> Re: Undefined subroutine errors (spider-bts) Was: [Debian-l10n-devel] [Help] Re: dl10n-check: Can't locate Debian/Pkg/DebSrc.pm in @INC</font>
<div> </div>
</div>
<div>
<p>Hello</p>
<p>Thanks for the quick answer.<br>
</p>
<div class="x_moz-cite-prefix">El 19/11/24 a las 15:16, gregor herrmann escribió:<br>
</div>
<blockquote type="cite">
<pre class="x_moz-quote-pre">On Tue, 19 Nov 2024 15:01:27 +0100, Laura Arjona Reina wrote:
</pre>
<blockquote type="cite">
<pre class="x_moz-quote-pre">Looking at the log, it says:
Undefined subroutine &Html::html called at
/srv/i18n.debian.org//dl10n/git/dl10n-html line 140.
We have the file in dl10n/git/lib/Debian/L10n/Html.pm that includes the
subroutine hmtl() and it was working without issues. I think that now Perl
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" data-auth="NotApplicable" class="x_moz-txt-link-freetext">https://salsa.debian.org/l10n-team/dl10n/-/blob/master/cron/spiderbts?ref_type=heads</a>
) gets the configuration from file dl10n.conf as the other scripts.
</pre>
</blockquote>
<pre class="x_moz-quote-pre">
Was perl upgraded?</pre>
</blockquote>
No AFAIK, we have 11.11 in tye, and perl 5.32.1-4+deb11u4 <br>
<br>
<blockquote type="cite">
<pre class="x_moz-quote-pre">
Anyway, looks indeed like
./dl10n-html
doesn't find
./lib
<guess>
I'd try to add
use lib '.';
at the top of dl10n-html, e.g. after "use POSIX qw(strftime);"
in <a href="https://salsa.debian.org/l10n-team/dl10n/-/blob/master/dl10n-html?ref_type=heads#L32" data-auth="NotApplicable" class="x_moz-txt-link-freetext">https://salsa.debian.org/l10n-team/dl10n/-/blob/master/dl10n-html?ref_type=heads#L32</a>
</pre>
</blockquote>
<p>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>
</p>
<blockquote type="cite">
<pre class="x_moz-quote-pre">
or call dl10n-html from the cronjob as "perl -I<full path to the
lib/directory>, e.g.
PERL -I$DL10NDIR/lib $DL10NDIR/dl10n-html --sdb …
in <a href="https://salsa.debian.org/l10n-team/dl10n/-/blob/master/cron/spiderbts?ref_type=heads#L46" data-auth="NotApplicable" class="x_moz-txt-link-freetext">https://salsa.debian.org/l10n-team/dl10n/-/blob/master/cron/spiderbts?ref_type=heads#L46</a>
</guess>
</pre>
</blockquote>
<p>I also tried this, editing the spiderbts file in tye.debian.org (used "perl" instead of PERL) obtaining the same error:</p>
<p><span style="font-family:monospace"><span style="color:#000000; background-color:#ffffff">Undefined subroutine &Debian::L10n::Html::html called at /srv/i18n.debian.org//dl10n/git/dl10n-html line 140.</span><br>
<br>
(I also changed the message written in the .log file to be sure that the system was running my modified script).</span></p>
<p><span style="font-family:monospace">Kind regards,</span></p>
<pre class="x_moz-signature" cols="72">--
Laura Arjona Reina
<a href="https://wiki.debian.org/LauraArjona" data-auth="NotApplicable" class="x_moz-txt-link-freetext">https://wiki.debian.org/LauraArjona</a></pre>
</div>
</body>
</html>