bug#27075: [PATCH] gnu: Add multitail

[Marius Bakke <mbakke@fastmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1373 bytes --]

Stefan Reichör <stefan@xsteve.at> writes:

> Hi there,
>
> I am not sure what category fits best for multitail. I have added it to
> less as a first attempt.

Thanks for this! I think "logging.scm" works well for this package. A
few comments:

> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'patch-curses-lib
> +           (lambda* (#:key outputs #:allow-other-keys)
> +             (let ((out (assoc-ref outputs "out")))
> +               (substitute* "mt.h"
> +                 (("ncursesw\\/panel.h") "panel.h")
> +                 (("ncursesw\\/ncurses.h") "ncurses.h")))
> +             #t))
> +         (delete 'configure)
> +         (delete 'check)))) ; no test suite (make check just runs cppcheck)

Instead of deleting the 'check' phase, you can set the #:tests? keyword
to #f.

> +    (inputs `(("ncurses" ,ncurses)))
> +    (home-page "https://vanheusden.com/multitail/")
> +    (synopsis "Monitor multiple logfiles")
> +    (description
> +     "MultiTail allows you to monitor logfiles and command output in multiple
> +windows in a terminal, colorize, filter and merge.")
> +    (license gpl2)))

When there is no version information indicated in the source code
headers, we default to "or later" (i.e. gpl2+ in this instance).

The rest LGTM. Can you send an updated patch?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]