diff options
| author | Luciano Dittgen <luciano.dittgen@ossystems.com.br> | 2026-07-29 09:34:36 -0300 |
|---|---|---|
| committer | github-actions[bot] <github-actions[bot]@users.noreply.github.com> | 2026-08-04 11:22:12 +0000 |
| commit | a1f556e5a96da2569ec590b5b5ce23c50f0a7877 (patch) | |
| tree | 0b2822c703d34c3e9bbb41b6c89a370bcf2ef6f3 /dynamic-layers | |
| parent | 62fa97dbb025bd0627822fada4b4490c7446abe9 (diff) | |
| download | meta-freescale-a1f556e5a96da2569ec590b5b5ce23c50f0a7877.tar.gz | |
opencv: Drop the redundant leading space in SRC_URI overrides
oelint.vars.notneededspace flagged three SRC_URI assignments in the
i.MX fork whose value opens with a space before the line continuation.
None of them needs it:
- SRC_URI += already inserts a separating space between the old and
the appended value, so the extra one only produced a double space.
- SRC_URI:remove and SRC_URI:prepend values are whitespace-split by
BitBake, so a leading space is not significant.
The trailing space on the SRC_URI:prepend value is deliberately kept:
:prepend is a plain string concatenation, so that space is what
separates the prepended entries from the pre-existing SRC_URI, and
removing it would join two URLs into one.
These three lines are i.MX-fork-local (they exist purely to swap the
upstream tag for IMX_BASE_VERSION), so the change does not add drift
against the meta-openembedded recipe.
Validation: oelint-adv rescan of the recipe; oelint.vars.notneededspace
is absent afterwards. The expanded value of SRC_URI is unchanged apart
from collapsed inter-entry whitespace, which BitBake discards when it
splits the variable.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
(cherry picked from commit e042636a232f02e286051f64fcdd17720de6a5a2)
Diffstat (limited to 'dynamic-layers')
| -rw-r--r-- | dynamic-layers/openembedded-layer/recipes-support/opencv/opencv_4.13.0.imx.bb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/dynamic-layers/openembedded-layer/recipes-support/opencv/opencv_4.13.0.imx.bb b/dynamic-layers/openembedded-layer/recipes-support/opencv/opencv_4.13.0.imx.bb index d509ac53e..d051b3628 100644 --- a/dynamic-layers/openembedded-layer/recipes-support/opencv/opencv_4.13.0.imx.bb +++ b/dynamic-layers/openembedded-layer/recipes-support/opencv/opencv_4.13.0.imx.bb | |||
| @@ -251,15 +251,15 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57" | |||
| 251 | 251 | ||
| 252 | # Need to override opencv and contrib URL because they include PV | 252 | # Need to override opencv and contrib URL because they include PV |
| 253 | IMX_BASE_VERSION = "${@'.'.join((d.getVar('PV') or '').split('.')[:3])}" | 253 | IMX_BASE_VERSION = "${@'.'.join((d.getVar('PV') or '').split('.')[:3])}" |
| 254 | SRC_URI:remove = " \ | 254 | SRC_URI:remove = "\ |
| 255 | git://github.com/opencv/opencv.git;name=opencv;branch=4.x;protocol=https;tag=${PV} \ | 255 | git://github.com/opencv/opencv.git;name=opencv;branch=4.x;protocol=https;tag=${PV} \ |
| 256 | git://github.com/opencv/opencv_contrib.git;destsuffix=${BB_GIT_DEFAULT_DESTSUFFIX}/contrib;name=contrib;branch=4.x;protocol=https;tag=${PV}" | 256 | git://github.com/opencv/opencv_contrib.git;destsuffix=${BB_GIT_DEFAULT_DESTSUFFIX}/contrib;name=contrib;branch=4.x;protocol=https;tag=${PV}" |
| 257 | SRC_URI:prepend = " \ | 257 | SRC_URI:prepend = "\ |
| 258 | git://github.com/opencv/opencv.git;name=opencv;branch=4.x;protocol=https;tag=${IMX_BASE_VERSION} \ | 258 | git://github.com/opencv/opencv.git;name=opencv;branch=4.x;protocol=https;tag=${IMX_BASE_VERSION} \ |
| 259 | git://github.com/opencv/opencv_contrib.git;destsuffix=${BB_GIT_DEFAULT_DESTSUFFIX}/contrib;name=contrib;branch=4.x;protocol=https;tag=${IMX_BASE_VERSION} " | 259 | git://github.com/opencv/opencv_contrib.git;destsuffix=${BB_GIT_DEFAULT_DESTSUFFIX}/contrib;name=contrib;branch=4.x;protocol=https;tag=${IMX_BASE_VERSION} " |
| 260 | 260 | ||
| 261 | # i.MX patches | 261 | # i.MX patches |
| 262 | SRC_URI += " \ | 262 | SRC_URI += "\ |
| 263 | file://0101-MGS-6470-ccc-Modify-host-ptr-alignment-size-in-UMAT.patch \ | 263 | file://0101-MGS-6470-ccc-Modify-host-ptr-alignment-size-in-UMAT.patch \ |
| 264 | file://0102-MGS-6470-ccc-Add-configuration-parameter-to-force-en.patch \ | 264 | file://0102-MGS-6470-ccc-Add-configuration-parameter-to-force-en.patch \ |
| 265 | file://0103-MGS-6470-ccc-Change-configuration-to-enable-hostptr-.patch \ | 265 | file://0103-MGS-6470-ccc-Change-configuration-to-enable-hostptr-.patch \ |
