[pacman-dev] [PATCH 2/3] use tap.sh for bash tests

Allan McRae allan at archlinux.org
Wed Dec 24 06:07:09 UTC 2014


On 24/12/14 07:53, Andrew Gregory wrote:
> tap.sh is a reusable TAP library that handles test counting and provides
> useful diagnostic messages on test failures.
> 
> Signed-off-by: Andrew Gregory <andrew.gregory.8 at gmail.com>
> ---
> 
> I'm maintaining tap.sh outside of pacman (hence the MIT license) at
> https://github.com/andrewgregory/tap.sh
> Documentation is available there.

I did a review of the script and have no comments.

> I have no idea if EXTRA_DIST is the correct place to list tap.sh, but distcheck
> seems to work with it.

That seems fine.

>  Makefile.am                        |   2 +-
>  test/scripts/human_to_size_test.sh |  39 ++-------
>  test/scripts/parseopts_test.sh     |  44 ++--------
>  test/tap.sh/tap.sh                 | 163 +++++++++++++++++++++++++++++++++++++

Do we need the tap.sh/ directory here?


More information about the pacman-dev mailing list