From: Richard Purdie <richard.purdie@linuxfoundation.org>
Date: Wed, 22 May 2024 15:11:23 +0000 (+0100)
Subject: insane: Add error for B = WORKDIR
X-Git-Tag: uninative-4.6~513
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e3c2c1fac904bb518d85e10a2ac0177c81cbf7e8;p=thirdparty%2Fopenembedded%2Fopenembedded-core.git

insane: Add error for B = WORKDIR

This was never a good idea and would have mostly happened from S = WORKDIR
however explictly disallow it and error if anyone tries.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass
index eeef43d00e2..42ed1a24a74 100644
--- a/meta/classes-global/insane.bbclass
+++ b/meta/classes-global/insane.bbclass
@@ -1603,9 +1603,12 @@ python () {
         oe.qa.handle_error("uppercase-pn", 'PN: %s is upper case, this can result in unexpected behavior.' % pn, d)
 
     sourcedir = d.getVar("S")
+    builddir = d.getVar("B")
     workdir = d.getVar("WORKDIR")
     if sourcedir == workdir:
         bb.fatal("Using S = ${WORKDIR} is no longer supported")
+    if builddir == workdir:
+        bb.fatal("Using B = ${WORKDIR} is no longer supported")
 
     # Some people mistakenly use DEPENDS:${PN} instead of DEPENDS and wonder
     # why it doesn't work.