On 16 Mar 2023 at 09:21:11 CET, "Julien ÉLIE"
<iulius@nom-de-mon-site.com.invalid> wrote:

> Hi Eli,
>
>> Can multiple pullnews instances be launched side by side?
>
> Yes, though you have to use a different set of newsgroups for each
> instance. Otherwise, they would do the same thing and it won't run much
> faster.
>
> For instance:
>
> pullnews -t 3 -c pullnews.marks1
> pullnews -t 3 -c pullnews.marks2

In the pulldown logs I see many of these lines:
x DEBUGGING 55508 421
x DEBUGGING 55509 421

What does this mean and what causes it?
After each such line it takes about 2 minutes until the next article is
downloaded and this slows down the download enormously.

I use debugging level 4.

Thanks.

Hi Eli,

>> pullnews -t 3 -c pullnews.marks1
>> pullnews -t 3 -c pullnews.marks2
>
> In the pulldown logs I see many of these lines:
> x
> DEBUGGING 55508 421
> x
> DEBUGGING 55509 421
>
> What does this mean and what causes it?
> After each such line it takes about 2 minutes until the next article is
> downloaded and this slows down the download enormously.

It means that article numbers 55508 and 55509 were not found on the
server (x). My guess is that the connection has timed out (421 special
internal code).

Jesse reported a bug which sounds like that a few months ago.

Could you please download the latest version of pullnews, and try it please?
https://raw.githubusercontent.com/InterNetNews/inn/main/frontends/pullnews.in

Just grab that file, rename it without .in, and change the first 2 lines
to fit what your current pullnews script has (it is the path to Perl and
the INN::Config module).
Then you can run that script. It will work with your version of INN.

--
Julien ÉLIE

« Campagne électorale : c'est l'art de gagner les voix des pauvres avec
l'argent des riches en promettant à chacun des deux de les protéger
contre l'autre. » (Oscar Ameringer)

On 17 Mar 2023 at 20:21:45 CET, "Julien ÉLIE"
<iulius@nom-de-mon-site.com.invalid> wrote:

>> In the pulldown logs I see many of these lines:
>> x
>> DEBUGGING 55508 421
>> x
>> DEBUGGING 55509 421
>>
>> What does this mean and what causes it?
>> After each such line it takes about 2 minutes until the next article is
>> downloaded and this slows down the download enormously.
>
> It means that article numbers 55508 and 55509 were not found on the
> server (x). My guess is that the connection has timed out (421 special
> internal code).
>
> Jesse reported a bug which sounds like that a few months ago.
>
> Could you please download the latest version of pullnews, and try it please?
>
> https://raw.githubusercontent.com/InterNetNews/inn/main/frontends/pullnews.in
>
> Just grab that file, rename it without .in, and change the first 2 lines
> to fit what your current pullnews script has (it is the path to Perl and
> the INN::Config module).
> Then you can run that script. It will work with your version of INN.

Hello Julien,

The latest version seems to have fixed it.

Thank you

Hi Eli,

> The latest version seems to have fixed it.

Glad to hear it. Thanks for the confirmation the fix works. (It has
not been released yet; the INN 2.7.1 release is scheduled in April.)

--
Julien ÉLIE

« Sol attigit talos. »

On 17 Mar 2023 at 21:21:45 CET, "Julien ÉLIE"
<iulius@nom-de-mon-site.com.invalid> wrote:

> Could you please download the latest version of pullnews, and try it please?
>
> https://raw.githubusercontent.com/InterNetNews/inn/main/frontends/pullnews.in
>
> Just grab that file, rename it without .in, and change the first 2 lines
> to fit what your current pullnews script has (it is the path to Perl and
> the INN::Config module).
> Then you can run that script. It will work with your version of INN.

Hi Julien,

I think there is a bug in this latest version.

When pullnews starts, it writes a PID file.
If pullnews is accidentally restarted, it will report that pullnews is already
running and that's perfect.
But after this message pullnews deletes the PID file :(

That's very annoying as I restart pullnews from cron every 15 minutes because
it occasionally stops for some unknown reason.

Other than that it works smoothly ;)

Eli.

Hi Eli,

> I think there is a bug in this latest version.
>
> When pullnews starts, it writes a PID file.
> If pullnews is accidentally restarted, it will report that pullnews is already
> running and that's perfect.
> But after this message pullnews deletes the PID file :(

Thanks again for your bug report; it's really appreciated to make
pullnews better.

When you download a new version of pullnews from the Git repository, do
not forget to manually edit the first 2 lines of the file which should
reflect the paths to Perl and INN::Config. (You can take the same as
the ones in other Perl scripts like mailpost.)
It gives something like:

#! /usr/bin/perl -w
use lib '/usr/share/perl5'; use INN::Config;

The issue you're facing is only triggered when you run pullnews outside
an INN installation (which is a possible use of pullnews, as it could be
run from a separate server).
There are 2 branches at several places in the code. PID file handling
was wrong in one of these branches, as you noticed.

I believe the issue is now fixed, and you can download the updated
pullnews script from the Git repository.

--
Julien ÉLIE

« Les amis de la vérité sont ceux qui la cherchent, et non ceux qui se
vantent de l'avoir trouvée. » (Condorcet)

On 30 Mar 2023 at 13:21:16 CEST, "Julien ÉLIE"
<iulius@nom-de-mon-site.com.invalid> wrote:
>
> The issue you're facing is only triggered when you run pullnews outside
> an INN installation (which is a possible use of pullnews, as it could be
> run from a separate server).
> There are 2 branches at several places in the code. PID file handling
> was wrong in one of these branches, as you noticed.
>
> I believe the issue is now fixed, and you can download the updated
> pullnews script from the Git repository.

Hi Julien,

The problem has been solved :)

Thank you for the speedy fix. It works great now.