]> code.ossystems Code Review - openembedded-core.git/commit
insane/package: let package.bbclass inherit insane.bbclass
authorRobert Yang <liezhi.yang@windriver.com>
Sat, 11 May 2013 22:46:10 +0000 (06:46 +0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 7 Jun 2013 15:41:02 +0000 (16:41 +0100)
commit852dead71387c66ec0cba7c71e3814a74e409560
tree041759af90bfb904b97575b4738ae6a357314f08
parentb1cd1c692d7b13e6865353c9bb2f7cf331c9f774
insane/package: let package.bbclass inherit insane.bbclass

RP's comment:
"What we're trying to do is move everything to use a standard mechanism
for reporting issues of this type (do_package). With insane.bbclass, you
can elect whether a given type of error is a warning or error and fails
the task."

* The package.bbclass had used package_qa_handle_error() which is from
  insane.bbclass, and we will use it for handling other warnings and
  errors, so let package.bbclass inherit insane.bbclass, this change will
  make the insane as a requirement (always included).

* Change the "PACKAGEFUNCS ?=" to "+=", otherwise there would be an
  error like:
  Exception: variable SUMMARY references itself!

  This is because we let package.bbclass inherit insane.bbclass, and
  PACKAGEFUNCS has been set in insane.bbclass, so the "PACKAGEFUNCS ?="
  will set nothing, then the "emit_pkgdata" doesn't run which will
  cause this error.

* Add a QA_SANE variable in insane.bbclass, once the error type
  is ERROR_QA, it will fail the task and stop the build.

[YOCTO #3190]
[YOCTO #4396]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
meta/classes/insane.bbclass
meta/classes/package.bbclass