]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core.git/commitdiff
insane: Add error for B = WORKDIR
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 22 May 2024 15:11:23 +0000 (16:11 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 23 May 2024 10:25:27 +0000 (11:25 +0100)
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>
meta/classes-global/insane.bbclass

index eeef43d00e220971b0c8a062bd177bb69700f355..42ed1a24a74fc14a9e385048dc17e4fa67246863 100644 (file)
@@ -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.