Re: make check-expensive [was: [PATCH] rm, du, chmod, chown, chgrp: use much less memory for large directories]

[Jim Meyering]

Bernhard Voelker wrote:

> On 08/24/2011 10:49 AM, Jim Meyering wrote:
>> Voelker, Bernhard wrote:
>>> BTW: Wouldn't this test deserve a proper make target, e.g.
>>> "make check-expensive"?
>>
>> Yes, good idea.
>> That would make it easier to run just those test.
>>
>> However, hard-coding the list of expensive and very-expensive
>> tests would require doing the same sort of thing as is done
>> for root_tests (see check-root) in tests/Makefile.am,
>> where there'd be a hand-maintained list of expensive and very-expensive
>> tests (in tests/Makefile.am) as well as rules to run them and rules
>> to cross-check that the lists are complete, as is done in cfg.mk's
>> sc-root_tests rule.
>
> Wouldn't it be sufficient to add something like the following
> to the top-level Makefile.am?
>
> check-expensive:
>       env RUN_EXPENSIVE_TESTS=yes make check
>
> check-very-expensive:
>       env RUN_VERY_EXPENSIVE_TESTS=yes make check

That would be sufficient if you don't mind running all of
the other tests, too.  Does a name like "check-expensive",
imply "all regular tests, plus the expensive ones"?
Would you expect "check-very-expensive" also to run the
"merely expensive" tests?  From what I recall, you'd have
to set RUN_EXPENSIVE_TESTS=yes, too, if you want that.

BTW, you don't need "env", and it should use $(MAKE), not "make", e.g.,

check-expensive:
        $(MAKE) check RUN_EXPENSIVE_TESTS=yes