bbclass: bb.fatal() clean up
authorRobert Yang <liezhi.yang@windriver.com>
Wed, 8 May 2013 09:06:46 +0000 (05:06 -0400)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 15 May 2013 21:09:47 +0000 (00:09 +0300)
commit635725cac6be1164809e559d9bb346cd710c9861
treee2bda42bd2477b626be16e713c1057c96368bcb1
parent476d818ec601e7d4c795e030d1bc9fa4e6d7a05f
bbclass: bb.fatal() clean up

The bb.fatal() is defined as:

def fatal(*args):
    logger.critical(''.join(args))
    sys.exit(1)

So anything after bb.fatal() in the same code block doesn't have any
effect, e.g.:

    bb.fatal("%s_%s: %s" % (var, pkg, e))
    raise e

The "raise e" should be removed.

I searched all the files which use bb.fatal(), only the following 3
classes have this issues:

  insane.bbclass
  package.bbclass
  package_rpm.bbclass

[YOCTO #4461]

(From OE-Core rev: 4c47525c5a171b1282615c9fbc7d84addef85f92)

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/insane.bbclass
meta/classes/package.bbclass
meta/classes/package_rpm.bbclass