John Cremona on Wed, 21 Jul 2010 23:10:51 +0200


[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

Re: some glitches in make test-all


On 21 July 2010 21:55, Bill Allombert
<Bill.Allombert@math.u-bordeaux1.fr> wrote:
> On Wed, Jul 21, 2010 at 12:04:59PM +0100, John Cremona wrote:
>> I was able to fix my specific problem (opting not to test
>> ellglobalred) by changing line 7 of config/get_tests from
>>
>> test_extra_out="ploth"
>>
>> to
>>
>> test_extra_out="ploth ellglobalred"
>>
>> Now, it should not be hard to make this automatic by testing the
>> existence of the data directory $GP_DATA_DIR/elldata and similarly to
>> include ell-sea in that list iff  $GP_DATA_DIR/seadata does not exist.
>>  By scripting skills are not so good, and I'm sure the person who
>> wrote get_tests could do that faster and more reliably than me!
>
> Well, the purpose of 'make test-all' is to test everything (as long as we have
> written tests), so this requires potentially that all the optional packages are
> installed.  Making a commitment that 'make test-all' will work with
> such-and-such subset of the packages would not make sense.  So would blindly
> omiting tests, because the user would assume the tests did pass while they were
> not performed at all.

I disagree!     A user who does not need the elliptic curve database
will clearly not install it;  but he is then told that there is a bug
in ellglobalred, which is misleading.  I am just suggesting that
make-all acts like make-all-installed.

>
> What you actually want is another 'make test-*' target that do what you want.
>

Yes: make-all-installed.

Or: for a test that fails only because of a not-installed data file,
the output could read something other than "BUG" (and the result code
should perhaps be OK).

John

> Cheers,
> Bill.
>