From bde5c8d7bc25438fa582fcc67b2041a581db7ccf Mon Sep 17 00:00:00 2001 From: Richard Purdie Date: Wed, 22 May 2024 16:40:50 +0100 Subject: base/insane: Move S/B checks to more logical place in insane class (From OE-Core rev: f8f3315d58f9ec7824961d1f6f96d39c449b9578) Signed-off-by: Richard Purdie --- meta/classes-global/insane.bbclass | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'meta/classes-global/insane.bbclass') diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index 42ed1a24a7..99736830b9 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass @@ -1609,6 +1609,10 @@ python () { bb.fatal("Using S = ${WORKDIR} is no longer supported") if builddir == workdir: bb.fatal("Using B = ${WORKDIR} is no longer supported") + if sourcedir[-1] == '/': + bb.warn("Recipe %s sets S variable with trailing slash '%s', remove it" % (d.getVar("PN"), d.getVar("S"))) + if builddir[-1] == '/': + bb.warn("Recipe %s sets B variable with trailing slash '%s', remove it" % (d.getVar("PN"), d.getVar("B"))) # Some people mistakenly use DEPENDS:${PN} instead of DEPENDS and wonder # why it doesn't work. -- cgit v1.2.3-54-g00ecf