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

Andrew Gregory andrew.gregory.8 at gmail.com
Wed Dec 24 22:27:14 UTC 2014


On 12/24/14 at 04:07pm, Allan McRae wrote:
> 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?

No.  I had initially included the documentation file; unless somebody
wants it added back, the script can be moved up a level.

apg


More information about the pacman-dev mailing list