diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-04-25 17:00:20 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-05-22 22:26:30 +0100 |
commit | c174208db939d372ca8ab5ff37b9ec694c400ac7 (patch) | |
tree | 197cf0cb377d4c22553ef6da1b737562beb133ad | |
parent | 569f8e09f0b077045d112dd569cbc4d47fdae08e (diff) | |
download | poky-c174208db939d372ca8ab5ff37b9ec694c400ac7.tar.gz |
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.
(From OE-Core rev: 32cba1cc916ad530c5e6630a927e74ca6f06289b)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/classes-global/insane.bbclass | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index 03ac06b102..eeef43d00e 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass | |||
@@ -1602,6 +1602,11 @@ python () { | |||
1602 | if prog.search(pn): | 1602 | if prog.search(pn): |
1603 | oe.qa.handle_error("uppercase-pn", 'PN: %s is upper case, this can result in unexpected behavior.' % pn, d) | 1603 | oe.qa.handle_error("uppercase-pn", 'PN: %s is upper case, this can result in unexpected behavior.' % pn, d) |
1604 | 1604 | ||
1605 | sourcedir = d.getVar("S") | ||
1606 | workdir = d.getVar("WORKDIR") | ||
1607 | if sourcedir == workdir: | ||
1608 | bb.fatal("Using S = ${WORKDIR} is no longer supported") | ||
1609 | |||
1605 | # Some people mistakenly use DEPENDS:${PN} instead of DEPENDS and wonder | 1610 | # Some people mistakenly use DEPENDS:${PN} instead of DEPENDS and wonder |
1606 | # why it doesn't work. | 1611 | # why it doesn't work. |
1607 | if (d.getVar(d.expand('DEPENDS:${PN}'))): | 1612 | if (d.getVar(d.expand('DEPENDS:${PN}'))): |