[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Pan-devel] [PATCH] Downloading headers from the command line
From: |
Charles Kerr |
Subject: |
Re: [Pan-devel] [PATCH] Downloading headers from the command line |
Date: |
Wed, 28 Mar 2007 11:37:42 -0500 (CDT) |
User-agent: |
SquirrelMail/1.4.9a |
>
> 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