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

[Frans Meulenbroeks]

2010/9/12 Holger Freyther <holger+oe at freyther.de>:
>
> Make conf/bitbake.conf load conf/insane.conf that will INHERIT the
> insane.bbclass. Remove the INHERIT from the distros currently using
> insane.bbclass. Change insane.bbclass to check QA_FATAL to see if errors
> are allowed to be fatal, make SHR/Kaelios and Angstrom use that new
> flag to continue having fatal QA Errors.
>
> Signed-off-by: Holger Hans Peter Freyther <zecke at selfish.org>

I am not really in favour of this.

One of the disadvantages of insane.bbclass is that it has:
PACKAGE_DEPENDS += "pax-utils-native desktop-file-utils-native"
This drags in several packages (e.g. sqlite3-native glib-2.0-native)
only to check if the .desktop file is valid.

By making this mandatory quite some additional build time is needed
(on a clean build), which is kind-of a nuisance and does not bring
anything if you are working on a display-less system (e.g. a NAS).

I''d be more in favour to split off the .desktop test and use that
only for those packages that generate .desktop files. (and make the
rest of insane.bbclass mandatory).
Alternately perhaps it is possible to create a var or so by which
people in their local or distro .conf can turn insane.bbclass (or the
.desktop part) off.

Frans