diff mbox series

insane: Remove redundant returns

Message ID 20240830072724.2719936-1-olani@axis.com
State Accepted, archived
Commit 1e49635f802b04acad14115640ce9fcd63cc32a7
Headers show
Series insane: Remove redundant returns | expand

Commit Message

Ola x Nilsson Aug. 30, 2024, 7:27 a.m. UTC 
From: Ola x Nilsson <olani@axis.com>

Some redundant return statements were left over from
insane: Drop oe.qa.add_message usage

Signed-off-by: Ola x Nilsson <olani@axis.com>
---
 meta/classes-global/insane.bbclass | 3 ---
 1 file changed, 3 deletions(-)
diff mbox series

Patch

diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass
index 32adcc722b..d7aba89b49 100644
--- a/meta/classes-global/insane.bbclass
+++ b/meta/classes-global/insane.bbclass
@@ -113,7 +113,6 @@  def package_qa_check_libexec(path,name, d, elf):
 
     if 'libexec' in path.split(os.path.sep):
         oe.qa.handle_error("libexec", "%s: %s is using libexec please relocate to %s" % (name, package_qa_clean_path(path, d, name), libexec), d)
-        return
 
 QAPATHTEST[rpaths] = "package_qa_check_rpath"
 def package_qa_check_rpath(file,name, d, elf):
@@ -932,7 +931,6 @@  def package_qa_check_deps(pkg, pkgdest, d):
 
 QAPKGTEST[usrmerge] = "package_qa_check_usrmerge"
 def package_qa_check_usrmerge(pkg, d):
-
     pkgdest = d.getVar('PKGDEST')
     pkg_dir = pkgdest + os.sep + pkg + os.sep
     merged_dirs = ['bin', 'sbin', 'lib'] + d.getVar('MULTILIB_VARIANTS').split()
@@ -941,7 +939,6 @@  def package_qa_check_usrmerge(pkg, d):
             msg = "%s package is not obeying usrmerge distro feature. /%s should be relocated to /usr." % (pkg, f)
             oe.qa.handle_error("usrmerge", msg, d)
             return
-    return
 
 QAPKGTEST[perllocalpod] = "package_qa_check_perllocalpod"
 def package_qa_check_perllocalpod(pkg, d):