Re: [fetchmail-devel] Re: [fetchmail-users] Incorrect header line and lost mails

[Matthias A. <mat...@gm...>]

Frederic Marchal <fre...@wo...> writes:

> Matthias (and others),
>
> I have some more questions and requests for guidance...
>
> - Should I patch the source from 6.3.2 or the one in the SVN
> repository ?

The SVN's branches/BRANCH_6-3/ would be the most useful place at this
time for fixes, and trunk/ for feature patches.

> - I will simply ignore the invalid lines (unless someone has a better
> idea).

Do you mean, skip the header, or include it without complaining?

> I think it should be user selectable. Is it suitable to have that
> option in the rcfile and not available from the command line ?

Personally, I do not appreciate such inconsistencies.

> - The option would be set with a SET statement in the rcfile. Is there
>any reason to make it configurable for each server ?

That depends on whether it's switching off the complaint (in that case,
global might be enough) or actually deleting the header. The better
option in any case is server or user configurable.

> - I intent to add the variable of the option in struct runctl if it is
> global and to struct hostdata if it is local to a server. Is it the good
> way of doing it ?

It's the only reasonable way of doing it. :-)

> - Does anybody have a suitable name for that option ?

That depends on what it does. It should default to "off" for compatibility.

-- 
Matthias Andree