Re: [Pan-devel] [PATCH] Downloading headers from the command line

[Calin A. Culianu]

Hi Eric (and others),

Please also note bug #424248 which complaned about the pan process hanging 
around in --nogui mode.  I attached my patch to solve this situation 
there.

It is entirely compatible with your patch.  So you may want to either 
combine my patch with yours or something or wait for Charles to include 
both patches in svn.

-Calin


On Thu, 29 Mar 2007, Eric Lauriault wrote:

> Hi,
>
> I opened the bugzilla tickets. Thanks again for the best newsreader around.
>
> later,
>
> Eric
>
> Charles Kerr wrote:
> > > On Sun, 25 Mar 2007, Eric Lauriault wrote:
> > >
> > > > Hi guys,
> > > >
> > > > I've attached a small patch to download headers from the command line.
> > > > I've wanted to do this to download headers for a few groups
> > > > automatically without a gui while I'm away using crond. It's pretty much
> > > > the same idea as using pan as an nzb reader. The problem I'm
> > > > experiencing is that when the queue empties, the process just hangs
> > > > around until it gets receives a signal. The same behavior can be
> > > > observed while downloading using an nzb file from the command line. Is
> > > > this expected?
> > > >
> > > > Eric
> >
> > > Yeah that's due to the fact that the g_main_loop() is still running.
> > >
> > > The only way to really solve this is to create a class that implements
> > > Queue::Listener and when it receives the on_queue_task_removed() event,
> > > to query the queue and see if it's empty.  If it is, it should call
> > > g_main_loop_quit().
> > >
> > > I can go ahead and modify your patch to do that, but I am not sure what
> > > the consensus is about this feature (you may want to ask Charles if he
> > > agrees with the approach I outlined above?).
> >
> > Calin, your approach is fine.  You could use on_queue_size_changed()
> > instead, since it passes the queue size as an argument.
> >
> > Eric, could you open a bugzilla ticket for this so that it's easier
> > to track the patches?
> >
> > cheers,
> > Charles
> >
> >
> >
> > _______________________________________________
> > Pan-devel mailing list
> > address@hidden
> > http://lists.nongnu.org/mailman/listinfo/pan-devel
>
>
>
> _______________________________________________
> Pan-devel mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/pan-devel