diff options
| author | Luciano Dittgen <luciano.dittgen@ossystems.com.br> | 2026-07-14 17:29:31 -0300 |
|---|---|---|
| committer | Luciano Dittgen <luciano.dittgen@ossystems.com.br> | 2026-07-22 11:34:43 -0300 |
| commit | 48475a88cbb26edb79e87d6dc0a3bd02ca20df5e (patch) | |
| tree | e44774e7bf9e44f4d96b04e431ab4e2a62233dad /recipes-bsp | |
| parent | 448d053c495fc52b51196692d8d2242d71001787 (diff) | |
| download | meta-freescale-48475a88cbb26edb79e87d6dc0a3bd02ca20df5e.tar.gz | |
imx-mkimage: Fix license/patch metadata per oelint
Correct the 0001 patch Upstream-Status reason from the non-matching
'[OE-specific]' to '[oe-specific]' (oelint.file.inappropriatemsg).
Accept two findings inline: the LIC_FILES_CHKSUM common-licenses
reference (the fetched source ships no standalone GPL-2.0-only file) and
the canonical .inc DEPENDS that oelint only flags because the require
lives in the .bb (oelint.vars.dependsappend).
oelint.var.nativefilename is left as a documented false positive: the
recipe uses 'inherit native' directly, so renaming the file to include
'-native' would double the class suffix and break PN, and the finding
is anchored to line 1 so it cannot be inline-suppressed.
oelint.var.inheritdevtool.native is intentionally left unaddressed here:
the proper fix is to let imx-mkimage build a target binary via
BBCLASSEXTEND = "native" instead of deferring the native inherit, which
is a functional change handled separately.
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
(cherry picked from commit bb3945d1d07409bf93c59e0647272f8bafd9910b)
Diffstat (limited to 'recipes-bsp')
3 files changed, 9 insertions, 1 deletions
diff --git a/recipes-bsp/imx-mkimage/files/0001-iMX8M-soc.mak-use-native-mkimage-from-sysroot.patch b/recipes-bsp/imx-mkimage/files/0001-iMX8M-soc.mak-use-native-mkimage-from-sysroot.patch index f182a2f14..f6cbc38dd 100644 --- a/recipes-bsp/imx-mkimage/files/0001-iMX8M-soc.mak-use-native-mkimage-from-sysroot.patch +++ b/recipes-bsp/imx-mkimage/files/0001-iMX8M-soc.mak-use-native-mkimage-from-sysroot.patch | |||
| @@ -15,7 +15,7 @@ Use it from the build sysroot, and do not pull the local version of it. | |||
| 15 | 15 | ||
| 16 | Reinjected the original patch from Andrey Zhizhikin <andrey.z@gmail.com> | 16 | Reinjected the original patch from Andrey Zhizhikin <andrey.z@gmail.com> |
| 17 | 17 | ||
| 18 | Upstream-Status: Inappropriate [OE-specific] | 18 | Upstream-Status: Inappropriate [oe-specific] |
| 19 | Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> | 19 | Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> |
| 20 | --- | 20 | --- |
| 21 | 21 | ||
diff --git a/recipes-bsp/imx-mkimage/imx-mkimage_git.bb b/recipes-bsp/imx-mkimage/imx-mkimage_git.bb index 65f65002c..7b3886e9e 100644 --- a/recipes-bsp/imx-mkimage/imx-mkimage_git.bb +++ b/recipes-bsp/imx-mkimage/imx-mkimage_git.bb | |||
| @@ -1,6 +1,9 @@ | |||
| 1 | # Copyright (C) 2016 Freescale Semiconductor | 1 | # Copyright (C) 2016 Freescale Semiconductor |
| 2 | # Copyright 2017-2022,2026 NXP | 2 | # Copyright 2017-2022,2026 NXP |
| 3 | 3 | ||
| 4 | # NOTE: uses 'inherit native' directly, so a '-native' filename (as | ||
| 5 | # oelint.var.nativefilename suggests) would double the class suffix and break PN. | ||
| 6 | # That finding is a false positive here and is not inline-suppressible. | ||
| 4 | require imx-mkimage_git.inc | 7 | require imx-mkimage_git.inc |
| 5 | 8 | ||
| 6 | SUMMARY = "i.MX boot image generation tool" | 9 | SUMMARY = "i.MX boot image generation tool" |
| @@ -8,6 +11,8 @@ DESCRIPTION = "Tooling to assemble i.MX boot images (mkimage_imx8 and related)." | |||
| 8 | HOMEPAGE = "https://github.com/nxp-imx/imx-mkimage" | 11 | HOMEPAGE = "https://github.com/nxp-imx/imx-mkimage" |
| 9 | SECTION = "bsp" | 12 | SECTION = "bsp" |
| 10 | LICENSE = "GPL-2.0-only" | 13 | LICENSE = "GPL-2.0-only" |
| 14 | # Source ships no standalone license file; reference the common GPL-2.0-only text. | ||
| 15 | # nooelint: oelint.var.licenseremotefile | ||
| 11 | LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6" | 16 | LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6" |
| 12 | 17 | ||
| 13 | inherit deploy native | 18 | inherit deploy native |
diff --git a/recipes-bsp/imx-mkimage/imx-mkimage_git.inc b/recipes-bsp/imx-mkimage/imx-mkimage_git.inc index b8d5d58cc..1455d99cb 100644 --- a/recipes-bsp/imx-mkimage/imx-mkimage_git.inc +++ b/recipes-bsp/imx-mkimage/imx-mkimage_git.inc | |||
| @@ -1,5 +1,8 @@ | |||
| 1 | # Copyright 2017-2026 NXP | 1 | # Copyright 2017-2026 NXP |
| 2 | 2 | ||
| 3 | # Canonical DEPENDS for this native tool; the require sits in the .bb so oelint | ||
| 4 | # sees this base assignment as "after include". | ||
| 5 | # nooelint: oelint.vars.dependsappend | ||
| 3 | DEPENDS = "openssl-native zlib-native" | 6 | DEPENDS = "openssl-native zlib-native" |
| 4 | 7 | ||
| 5 | SRC_URI = "${IMX_MKIMAGE_SRC};branch=${SRCBRANCH} \ | 8 | SRC_URI = "${IMX_MKIMAGE_SRC};branch=${SRCBRANCH} \ |
