Re: [Bug-wget] contributing to wget

[vivi]

The recovery of wget crossed my mind too but I presumed in such an
instance the user would install their own copy of wget or speak to the
administrator. Perhaps asking the user how to proceed if the system
wgetrc parse failed would suitable. e.g

"Parsing system wgetrc file failed; please check /etc/wgetrc"
"Attempt to parse user wgetrc file? (y/n)"

Any thoughts?

Thanks for the comments Giuseppe, I've made the alterations.

        -Reza

On 11/06/10 11:24, Giuseppe Scrivano wrote:
> Hrvoje Niksic <address@hidden> writes:
> 
>> "address@hidden" <address@hidden> writes:
>>
>>> Thanks for the encouragement! I've attached a patch which should tell
>>> the user there is a problem with the system wgetrc file and exit. Seems
>>> suspiciously simple, can anyone spot any problems with it?
>> [...]
>>> +  /*If there are any problems parsing the system wgetrc file, tell the 
>>> user and exit*/
>>> +  if (!ok) 
>>> +    {
>>> +   fprintf(stderr, _("Parsing system wgetrc file failed; please check 
>>> '%s'.\n"), SYSTEM_WGETRC);
>>> +           exit(2);
>>> +    }              
>>
>> Is there a way for Wget to recover from this error?  The user who runs
>> Wget might not have superuser privileges needed to correct this problem.
> 
> Good point.  It is a good idea to don't exit here, at least until #20635
> is not done.
> 
> Thanks,
> Giuseppe