diff options
| -rw-r--r-- | meta/classes/oelint.bbclass | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/meta/classes/oelint.bbclass b/meta/classes/oelint.bbclass index 07a7ed9d7c..d00f468d9a 100644 --- a/meta/classes/oelint.bbclass +++ b/meta/classes/oelint.bbclass | |||
| @@ -64,4 +64,22 @@ python do_lint() { | |||
| 64 | bb.warn("%s: %s doesn't have Signed-off-by" % (pkgname, item)) | 64 | bb.warn("%s: %s doesn't have Signed-off-by" % (pkgname, item)) |
| 65 | if findKey(path, "Upstream-Status"): | 65 | if findKey(path, "Upstream-Status"): |
| 66 | bb.warn("%s: %s doesn't have Upstream-Status" % (pkgname, item)) | 66 | bb.warn("%s: %s doesn't have Upstream-Status" % (pkgname, item)) |
| 67 | |||
| 68 | |||
| 69 | ############################## | ||
| 70 | # Check for ${PN} or ${P} usage in SRC_URI or S | ||
| 71 | # Should use ${BPN} or ${BP} instead to avoid breaking multilib | ||
| 72 | # | ||
| 73 | for s in srcuri: | ||
| 74 | if not s.startswith("file://"): | ||
| 75 | if not s.find("{PN}") == -1: | ||
| 76 | bb.warn("%s: should use BPN instead of PN in SRC_URI" % pkgname) | ||
| 77 | if not s.find("{P}") == -1: | ||
| 78 | bb.warn("%s: should use BP instead of P in SRC_URI" % pkgname) | ||
| 79 | |||
| 80 | srcpath = d.getVar("S") | ||
| 81 | if not srcpath.find("{PN}") == -1: | ||
| 82 | bb.warn("%s: should use BPN instead of PN in S" % pkgname) | ||
| 83 | if not srcpath.find("{P}") == -1: | ||
| 84 | bb.warn("%s: should use BP instead of P in S" % pkgname) | ||
| 67 | } | 85 | } |
