fetchmail-devel

[Fetchmail-devel] Option, help message and man page fixes

[Vitezslav C. <vcr...@re...>]

Hello Matthias,

I went through usage message/man page and added a few missing option
descriptions, fixed wrong indentation ("The ISP can make the...")
and minor inaccuracy ("bsmtp" option is not associated with '-o').

I also removed '-A' option (which required an argument). It seems
to do nothing at all and didn't find any documentation about it.
Or is it useful somehow?

Patch attached. Feel free to tweak it as you need (e. g. better
wording).

Best regards,
Vitezslav Crhonek

Re: [Fetchmail-devel] Option, help message and man page fixes

[grarpamp <gra...@gm...>]

On 4/27/16, Vitezslav Crhonek <vcr...@re...> wrote:
> I also removed '-A' option (which required an argument). It seems
> to do nothing at all and didn't find any documentation about it.
> Or is it useful somehow?

May have been related / intended for...?

  {"auth",      required_argument, (int *) 0, LA_AUTH},

Re: [Fetchmail-devel] Option, help message and man page fixes

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

Am 27.04.2016 um 21:36 schrieb grarpamp:
> On 4/27/16, Vitezslav Crhonek <vcr...@re...> wrote:
>> I also removed '-A' option (which required an argument). It seems
>> to do nothing at all and didn't find any documentation about it.
>> Or is it useful somehow?
> 
> May have been related / intended for...?
> 
>   {"auth",      required_argument, (int *) 0, LA_AUTH},

Possibly, but I'll rather not use short names for dangerous matters.
I checked Eric's last release, 6.2.5, which also appeared to ignore 'A
<arg>' if I read the code properly.

Re: [Fetchmail-devel] Option, help message and man page fixes

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

Am 27.04.2016 um 14:07 schrieb Vitezslav Crhonek:
> Hello Matthias,
> 
> I went through usage message/man page and added a few missing option
> descriptions, fixed wrong indentation ("The ISP can make the...")
> and minor inaccuracy ("bsmtp" option is not associated with '-o').
> 
> I also removed '-A' option (which required an argument). It seems
> to do nothing at all and didn't find any documentation about it.
> Or is it useful somehow?
> 
> Patch attached. Feel free to tweak it as you need (e. g. better
> wording).
> 
> Best regards,
> Vitezslav Crhonek

Hi Vitezslav,

Thank you. I've applied the patch on the legacy_64 branch, but it does
have a different has now because I had to recreate it, "git am" did not
take it (missing reference to the tree it was generated from I presume).

-A wasn't parsed out as far as I can see, so either it was a typo or
some kind of mistake - seems this was silently ignored.

Thanks again.

Best regards,
Matthias