diff options
| author | Robert Yang <liezhi.yang@windriver.com> | 2013-05-08 05:06:46 -0400 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-05-16 00:09:47 +0300 |
| commit | 51c3e9df28619bfa5a6c22a7c91c7c7093080c87 (patch) | |
| tree | bc635ace3154602741df836574457ae976a5846d /meta/classes/package_rpm.bbclass | |
| parent | bfaf6c5278288eecefddaf3db1a0631bd20c33ce (diff) | |
| download | poky-51c3e9df28619bfa5a6c22a7c91c7c7093080c87.tar.gz | |
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>
Diffstat (limited to 'meta/classes/package_rpm.bbclass')
| -rw-r--r-- | meta/classes/package_rpm.bbclass | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/meta/classes/package_rpm.bbclass b/meta/classes/package_rpm.bbclass index 25b14dd79d..8935c9fc6a 100644 --- a/meta/classes/package_rpm.bbclass +++ b/meta/classes/package_rpm.bbclass | |||
| @@ -672,12 +672,10 @@ python write_specfile () { | |||
| 672 | pkgdest = d.getVar('PKGDEST', True) | 672 | pkgdest = d.getVar('PKGDEST', True) |
| 673 | if not pkgdest: | 673 | if not pkgdest: |
| 674 | bb.fatal("No PKGDEST") | 674 | bb.fatal("No PKGDEST") |
| 675 | return | ||
| 676 | 675 | ||
| 677 | outspecfile = d.getVar('OUTSPECFILE', True) | 676 | outspecfile = d.getVar('OUTSPECFILE', True) |
| 678 | if not outspecfile: | 677 | if not outspecfile: |
| 679 | bb.fatal("No OUTSPECFILE") | 678 | bb.fatal("No OUTSPECFILE") |
| 680 | return | ||
| 681 | 679 | ||
| 682 | # Construct the SPEC file... | 680 | # Construct the SPEC file... |
| 683 | srcname = strip_multilib(d.getVar('PN', True), d) | 681 | srcname = strip_multilib(d.getVar('PN', True), d) |
