Logcheck devel - Jul 2004 - Bug#260330: Further bugs in command line parsing

Hi,
there appear to be further bugs in command line parsing:
I tried:

logcheck -t -T -o -l /var/log/syslog

Which is expected to report to stdout. But it tries to execute sendmail.

logcheck -t -l /var/log/syslog -o -T -d

Did neither report to stdout (-o) nor run in debug mode.

The help message is also incorrect:

usage: logcheck [-c CFG] [-d] [-h] [-l CFG] [-o] [-r DIR] [-s|-p|-w] [-
S] [-t]
... missing, inconsistent or incorrect are:
 -l LOG       = run logfile through logcheck
 -L CFG       = overrule default logfiles list
 -m           = email the report to recipient
 -R           = adds Reboot: to email subject
 -T           = do not remove the TMPDIR
 -u           = enable syslog-summary

Have you noted, that the "mktemp" call contains unneded Xes?
you give the pattern logcheck.XXXXXXXX, but only the last 6 X will be
replaced by a random value.

Greetings,
Erich

hello,

On Tue, 20 Jul 2004, Erich Schubert wrote:
> Hi,
> there appear to be further bugs in command line parsing:
> I tried:
> 
> logcheck -t -T -o -l /var/log/syslog
> 
> Which is expected to report to stdout. But it tries to execute sendmail.
> 
> logcheck -t -l /var/log/syslog -o -T -d
> 
> Did neither report to stdout (-o) nor run in debug mode.
uups -T does not need any args, corrected in current cvs.
now both examples work as expected.
 
> The help message is also incorrect:
> 
> usage: logcheck [-c CFG] [-d] [-h] [-l CFG] [-o] [-r DIR] [-s|-p|-w] [-
> S] [-t]
> ... missing, inconsistent or incorrect are:
>  -l LOG       = run logfile through logcheck
>  -L CFG       = overrule default logfiles list
>  -m           = email the report to recipient
>  -R           = adds Reboot: to email subject
>  -T           = do not remove the TMPDIR
>  -u           = enable syslog-summary
corrected in cvs, also be consistent to name the args for all
switches that need one.
 
> Have you noted, that the "mktemp" call contains unneded Xes?
> you give the pattern logcheck.XXXXXXXX, but only the last 6 X will be
> replaced by a random value.
no hadn't noticed, but thanks for your attention.
also corrected in cvs. will be in next imminent release.

still that won't help you so much,
but i'll take a look at your "real" problem.
thanks for the bug reports.

a++ maks




-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url :
http://lists.alioth.debian.org/pipermail/logcheck-devel/attachments/20040720/01e89cee/attachment.pgp