Re: CVS commit: pkgsrc/sysutils/pflogsumm

[Leonardo Taccari <leot%NetBSD.org@localhost>]

Hello Sean,

"Sean Cole" writes:
> [...]
> Module Name:  pkgsrc
> Committed By: scole
> Date:         Sat Feb  2 18:10:24 UTC 2019
>
> Modified Files:
>       pkgsrc/sysutils/pflogsumm: Makefile distinfo
>
> Log Message:
> Updated sysutils/pflogsumm to 1.1.3
> [...]

Thank you for updating it!

If possible please always provide a changelog or a summary of changes
in the commit message.

E.g. from ${WRKSRC}/Changelog:

------------------------8<-------------------8<------------------------
rel-1.1.3       20100320

    Added long-awaited switches to optionally reduce detail reporting:
    --bounce_detail=N, --deferral_detail=N, --reject_detail=N,
    --smtp_detail=N, smtpd_warning_detail=N, and --detail=N.  Setting
    any of them to 0 suppresses that detail entirely.  --detail=N sets
    the default for all of them, as well as for -u=N and -h=N.

    With the above enhancements, the following switches are depreciated,
    and will eventually be removed: --no_bounce_detail,
    --no_deferral_detail, --no_reject_detail and --no_smtpd_warnings.
    They are replaced by setting the desired --*_detail=0.  They still
    work, but using them generates a warning.

    Added support for parsing logs with RFC 3339 timestamps.  Thanks
    and a tip o' the hat to sftf-at-yandrex-dot-ru for the heads-up
    and the code contribution.  (N.B.: My code does not require a
    command-line switch.  The format is detected automatically.)

    Fixed some --ignore-case inconsistincies.  Thanks and a tip o'
    the hat to Richard Blanchet (richard-dot-blanchet-at-free-dot-fr)
    for the heads-up and the diff.

    Fixed parsing bug that resulted in attempts to treat
    kind-of-IPv4-looking strings as IPv4 addresses.  (I really need to
    improve reject/defer/etc. "reason" parsing to fix this properly.)
    Thanks to Joseph Vit (jvit-at-certicon-dot-cz) for the bug
    report.

rel-1.1.2       20080629

    Fixed bug with calculating yesterday's date in vicinity of DST
    changes.  (Thanks and a tip o' the hat to Wieland Chmielewski
    for bringing the problem to my attention.)

    Added missing "underlining" to some (sub-)section titles for
    consistency.


rel-1.1.1       20070406

    Fixed to parse Postfix-2.3 (and beyond) logfiles.  Thanks to
    whomever contributed to

        http://bugs.gentoo.org/show_bug.cgi?id=144236

    Removed support for vmailer.

    Removed "SMTPD_STATS_SUPPORT" "fences" in code in favour of code
    to automatically detect the availability of Date::Calc.  If
    --smtpd_stats is specified and Date::Calc is not installed, now
    bails-out with friendly message.  (Adapted from suggestion and
    examples provided by David Landgren <david-at-landgren-dot-net>.
    Thanks!)

    Removed rem_smtpd_stats_supp.pl utility from distribution.  (No
    longer needed.)

    Memory footprint improvement: Pflogsumm no longer stores data for
    reports that are supressed via --no_<mumble> switches.

    Removed extraneous arguments in two calls to print_nested_hash
    that would result in the "quiet" flag being ignored.  Thanks to
    Pavel Urban (pupu-at-pupu-dot-cz) for bringing that to my
    attention.

    Added notes to FAQ about translations and i18n, about mismatching
    "received"/"delivered" counts, about bug in calculating "yesterday,"
    and about John Fawcett's "prepflog."
------------------------8<-------------------8<------------------------


Thanks again!