[Freeciv-Dev] Re: (PR#8662) [PATCH] tests/Makefile.am fixes

["Marko Lindqvist" <marko.lindqvist@xxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=8662 >

Marko Lindqvist wrote:

...without checking his facts, obviously

> test scripts distributed I added them to check_SCRIPTS. But those 
> scripts are _not_ automake compliant check scripts. Since target 'check' 
> is manually overridden, automake never tries to actually use them, but 
> this is still no good. (For now) they should appear under noinst_SCRIPTS.

  By rereading automake manual a bit I got impression that scripts 
listed in check_SCRIPTS are just dependencies for actual check, automake 
does not try run scripts just because they are listed in check_SCRIPTS. 
For that, they have to appear in TESTS as well. So check_SCRIPTS is ok 
after all.
  But I'm not making any absolute statements just now since I already 
(probably) were wrong once.


  - Caz