From 32cba1cc916ad530c5e6630a927e74ca6f06289b Mon Sep 17 00:00:00 2001 From: Richard Purdie Date: Thu, 25 Apr 2024 17:00:20 +0100 Subject: [PATCH] insane: Error for S == WORKDIR Where a recipe uses WORKDIR as S, exit with a fatal error since the code is no longer safe for this layout. Signed-off-by: Richard Purdie --- meta/classes-global/insane.bbclass | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index 03ac06b102f..eeef43d00e2 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass @@ -1602,6 +1602,11 @@ python () { if prog.search(pn): oe.qa.handle_error("uppercase-pn", 'PN: %s is upper case, this can result in unexpected behavior.' % pn, d) + sourcedir = d.getVar("S") + workdir = d.getVar("WORKDIR") + if sourcedir == workdir: + bb.fatal("Using S = ${WORKDIR} is no longer supported") + # Some people mistakenly use DEPENDS:${PN} instead of DEPENDS and wonder # why it doesn't work. if (d.getVar(d.expand('DEPENDS:${PN}'))): -- 2.47.3