[oe] [PATCH] insane.bbclass: Enable the insane.bbclass for everyone in a warning mode

[Frans Meulenbroeks]

2010/9/12 Chris Larson <clarson at kergoth.com>:

>
> I think we should reorganize insane.bbclass and split up its functionality,
> either into multiple classes or configurable via variables, as others have
> suggested in this thread.  If that's done, we can enable the core by
> default, but leave the bits that require additional dependencies as
> optional.
> --

Agree.

One of the ways to do so is to add more semantics to SECTION where
whatever is in SECTION triggers a dependency on the sanity tester part
for that section.
Alternately we could decide that every lib file has an inherit lib
clause or something like that.
Disadvantage is that we need to check/update all recipes and there is
a chance that someone forgets to add such a line.
(of course we could work around that by saying that e.g. SECTION is
mandatory and can contain only a predefined number of values. Still
that does not beat the case that someone just uses a wrong value).

BTW I feel that whether (QA or other) tests are executed should be
ultimately controllable by the person who builds (e.g. in the
local.conf). He/she is the one who has to pay for these (in terms of
time and cycles and he/she gets the benefits (or gets burned) by using
it.

Frans