Re: 04script-wrapable-autotests

[Derek Robert Price]

Akim Demaille wrote:

> I am firmly against this patch, and ask for it to be reversed.  You
> will first have to explain it, and teach us why we'd want such a code
> bloat.  The very first problem with this patch is its ChangeLog.
>  

It allows scripts to appear in autotest sources, making an autotest 
script behave much more like configure.in.

It allows the CVS test suite to set some variables, for instance 
$CVSROOT, for the tests to share.  In conjunction with my option parsing 
patch, this can be switched depending on whether the user wanted to test 
the client/server or local mode of CVS.

It will also allow some constructs such as:

   for remote in false :; do
       m4_include([tests.at])
   done

or:

   for server in cvs-1.10 cvs-1.11 cvs-1.11.5; do
       m4_include([tests.at])
   done
      
Where tests.at contains AT_SETUP...AT_CHECK...AT_CLEANUP sets.  Tests 
defined in tests.at have access to all the environment variables defined 
previously, including the `remote' and `server' variables from the above 
examples.  The first loop is a convenience to allow testing of local and 
remote versions of CVS.  The second could be used to test a client 
against multiple versions of a server to ensure protocol compatibility.

This patch increases the size of the test suite slightly, but disk is 
cheap and portability is priceless.

Derek

--
               *8^)

Email: address@hidden

Get CVS support at <http://ximbiot.com>!
--
Actual listing in the TV section of the Marin (CA) Independent-Journal:

Movie _The Wizard of Oz_: Transported to a surreal landscape, a young
girl kills the first woman she meets, then teams up with three complete
strangers to kill again.