[PATCH macros] Add a test suite

[Gaetan Nadon]

On Tue, 2010-04-13 at 13:56 -0500, Yaakov (Cygwin/X) wrote:

> On 2010-04-13 13:46, Gaetan Nadon wrote:
> > Have you had a look at Automake Test Suite support? (I haven't yet)
> > http://www.gnu.org/software/automake/manual/automake.html#index-g_t_0040code_007bmkdir-_002dp_007d_002c-macro-check-325
> >
> > It looks like there could be some benefits from the pre-defined "check"
> > primary variable prefix.
> 
> Do you mean:
> 
> dist_check_SCRIPTS = test/runtest
> TESTS = $(dist_check_SCRIPTS)
> 
> You still need two lines, one to run the test and one to dist the 
> script.  I think TESTS and EXTRA_DIST is a slightly better combination, 
> only because EXTRA_DIST may be used for other files in the future. 
> Otherwise I see no difference.
> 

I didn't mean anything specific, just pointing out that automake had
some thoughts about adding test cases and perhaps there was something in
there to leverage out, you know, the usual about not reinventing the
wheel. It looks like you already had a look into it. 

Reviewed-by: Gaetan Nadon <memsize at videotron.ca>

Thanks

> 
> Yaakov
> Cygwin/X
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100413/df56fa46/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100413/df56fa46/attachment.pgp>