| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
| |
bison-native is already added to DEPENDS by the required
u-boot-fslc-common include, so listing it again here is redundant.
Remove the duplicate, clearing oelint's vars.duplicate.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
| |
The recipe set only DESCRIPTION, tripping oelint's mandatoryvar.SUMMARY
and suggestedvar.SECTION. Add a SUMMARY, set SECTION to "bootloaders"
(matching the OE-core u-boot convention for boot infrastructure), and
expand DESCRIPTION so it is distinct from and longer than SUMMARY.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |\
| |
| |
| |
| | |
lucianogdittgen/oelint-cleanup-11-recipe-metadata-7
oelint: Recipe metadata, do_install and split-package FILES cleanups
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a SUMMARY (mandatory), expand the terse DESCRIPTION, and move
PACKAGE_ARCH above the PACKAGES/FILES block (order.PACKAGE_ARCH).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a SUMMARY (mandatory) and move PACKAGE_ARCH above the PACKAGES/FILES
block (order.PACKAGE_ARCH).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a DESCRIPTION (mandatory) and move LICENSE/LIC_FILES_CHKSUM above
DEPENDS so the metadata follows the canonical order (order.LICENSE).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Set HOMEPAGE (mandatory) and accept the explicit FILES:${PN} = "/" with
an inline # nooelint: the prebuilt SECO firmware is deployed under the
firmware root, so the main package claims that path deliberately
(filesoverride).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a SUMMARY (mandatory) and move SECTION above LICENSE so the metadata
follows the canonical order (order.SECTION).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Set SECTION = "bsp" and move PACKAGE_ARCH above the PACKAGES/FILES block
so it follows the canonical order (suggestedvar.SECTION,
order.PACKAGE_ARCH).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a SUMMARY (mandatory) and expand the DESCRIPTION, and move
PACKAGE_ARCH above the PACKAGES/FILES block so it follows the canonical
order (order.PACKAGE_ARCH).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The recipe installs the SOF firmware and topology tree and packs it into
the main package, so oelint flags the explicit FILES:${PN} as
oelint.var.filesoverride. Accept it with an inline # nooelint.
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The recipe deploys the prebuilt EdgeLock Enclave firmware blob and packs
the firmware directory into the main package, so oelint flags the
explicit FILES:${PN} as oelint.var.filesoverride. Accept it with an
inline # nooelint.
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The firmware is split into per-device sub-packages (epdc, hdmi, and the
various VPU codecs). oelint flags each FILES:${PN}-<device> = as
oelint.var.filesoverride. These package-specific FILES vars are never
otherwise set, so switch them to += : the assignment is behaviourally
identical and no longer trips the override check, so no suppression is
needed.
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The firmware is split into per-chip sub-packages, and each block keeps
its FILES assignment beside the matching RDEPENDS/RPROVIDES/RREPLACES/
RCONFLICTS so a package's definition reads as a unit. oelint flags this
two ways:
- oelint.var.filesoverride on every FILES:${PN}-<pkg> = assignment.
Switch these to += (the vars are never otherwise set, so this is
behaviourally identical) and the override check no longer fires.
- oelint.var.order.FILES, because interleaving each package's FILES with
its own R* vars means a later FILES lands after an earlier RDEPENDS/
RCONFLICTS. Clearing it would mean scattering all FILES to the top and
breaking the per-package grouping, so keep an inline # nooelint:
oelint.var.order.FILES per block instead.
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |\ \
| | |
| | | |
oelint metadata cleanup: add SUMMARY/DESCRIPTION/SECTION and version recipe filenames
|
| | |/
| |
| |
| |
| |
| |
| | |
Add a DESCRIPTION describing the NAND boot-image writer
(mandatoryvar.DESCRIPTION). No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Move the DEPENDS block (INHIBIT_DEFAULT_DEPS, DEPENDS and the
toolchain-clang append) above the inherit line so DEPENDS precedes
PACKAGECONFIG, resolving oelint.var.order.DEPENDS and the
oelint.vars.dependsappend 'only append after include/inherit' error.
Keep the two conditional ANNOTATED_NAME:append tokens as separate,
readable assignments. They deliberately join without a leading space to
build the contiguous bl31 filename, which trips oelint.vars.inconspaces
on the append form; suppress it with a per-line # nooelint on each append
plus a comment explaining why the leading-space convention does not apply
here. (A per-line marker is required: nooelint scopes to the single line
below it, so it cannot cover both appends at once.)
Accept the remaining LIC_FILES_CHKSUM common-licenses reference inline
(the fetched source ships no standalone BSD-3-Clause file).
No functional change; deferred override assignments are applied at
finalization regardless of textual position.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
OE-core now parses LICENSE as an SPDX expression, so the legacy
"&" and "|" operators and non-SPDX names trigger warnings/errors during
parsing.
- pktgen-dpdk, skmm-ep, kernel-module-ar and skmm-host actually failed
parsing with a QA error (LicenseRef-BSD has no generic license file
since it was dropped from oe-core). Add NO_GENERIC_LICENSE[BSD] entries
pointing at their existing license text file to fix them
- Prefix non-SPDX license names with "LicenseRef-"
- Replace "&" with "AND" and "|" with "OR"
Signed-off-by: Antoine Gouby <antoine.gouby@toradex.com>
|
| |\
| |
| |
| |
| | |
lucianogdittgen/oelint-cleanup-10-recipe-metadata-6
oelint: Class/anon-python docstrings, image and packagegroup metadata
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Move the PACKAGECONFIG block up beside the fetch variables so it
precedes PACKAGE_ARCH (oelint.var.order.PACKAGECONFIG).
Accept two findings inline: PV:append must join without a leading space
to remain a valid version string (oelint.vars.inconspaces false
positive), and the arm-cot/secure consistency check needs a parse-time
anonymous python (oelint.task.noanonpython).
No functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
For the dummy imx-test_00.00.00 placeholder, give DESCRIPTION distinct
wording (no longer identical to SUMMARY) and set an explicit empty
SRC_URI (it is an ALLOW_EMPTY package with no upstream sources), which
satisfies the mandatory-variable check honestly.
For imx-test_git, move FILES:${PN}-dbg above RDEPENDS into the canonical
variable order.
In both recipes the LIC_FILES_CHKSUM references the common-licenses copy
because the license text is not shipped in the source tree; this
intentional local (non-remote) reference is documented with an inline
nooelint suppression.
No functional change; pure recipe-metadata cleanup.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Expand DESCRIPTION so it is no longer briefer than SUMMARY, and in the
.bb move FILES:${PN}-crrm above RDEPENDS (canonical order) while using an
append so it no longer overrides the package file list.
The base DEPENDS assignment in the .inc keeps a plain '=' and is
documented with an inline nooelint suppression: the .inc is the canonical
definition of the library's build dependencies and the require sits at
the top of the .bb, so oelint's 'append after include' heuristic is a
false positive here.
No functional change; pure recipe-metadata cleanup.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |/
|
|
|
|
|
|
|
|
|
|
| |
Add a SUMMARY (distinct from the existing DESCRIPTION) to resolve the
mandatory-variable finding, and reorder into the canonical oelint
variable order: SECTION before LICENSE/LIC_FILES_CHKSUM, SRC_URI before
SRCREV, and EXTRA_OECONF moved below the PACKAGECONFIG definitions it
consumes (PACKAGECONFIG before EXTRA_OECONF).
No functional change; pure recipe-metadata cleanup.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
oelint-adv flagged a missing DESCRIPTION, a suggested SECTION, and the
INSANE_SKIP use.
Add DESCRIPTION and SECTION. The INSANE_SKIP is required for the
prebuilt PFE firmware blobs (foreign target arch, already stripped), so
accept it with an inline '# nooelint: oelint.vars.insaneskip' and a
rationale comment. Metadata-only change; the built package is unaffected.
oelint-adv now reports no findings for this recipe.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
oelint-adv flagged a missing DESCRIPTION, a suggested SECTION,
PACKAGE_ARCH after FILES, and 'cp' used in do_install.
Add DESCRIPTION and SECTION, move PACKAGE_ARCH ahead of the PACKAGES/
FILES block, and replace the recursive 'cp' in do_install with
'install -m 0644' (the upstream firmware tree is flat, so no recursion
is needed). Functionally equivalent; the same files land in /boot.
oelint-adv now reports no findings for this recipe.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
oelint-adv flagged a missing DESCRIPTION, a suggested SECTION, DEPENDS
appearing after SRCREV, and PACKAGE_ARCH after FILES.
Add DESCRIPTION and SECTION, move DEPENDS ahead of the fetch variables,
and move PACKAGE_ARCH ahead of the PACKAGES/FILES block as oelint
expects. Metadata-only change; the built package is unaffected.
oelint-adv now reports no findings for this recipe.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
oelint-adv flagged a missing DESCRIPTION, a suggested SECTION, and
PACKAGE_ARCH appearing after FILES.
Add DESCRIPTION and SECTION and move PACKAGE_ARCH ahead of the PACKAGES/
FILES block as oelint expects. Metadata-only change; the built package
is unaffected.
oelint-adv now reports no findings for this recipe.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
oelint-adv flagged a missing SUMMARY, a suggested SECTION, and
LICENSE/SRC_URI/PV appearing out of the expected order.
Add SUMMARY and SECTION, move LICENSE ahead of DEPENDS, and reorder the
fetch variables to PV / SRC_URI / SRCREV as oelint expects. Metadata-only
change; the built package is unaffected.
oelint-adv now reports no findings for this recipe.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE and set SECTION.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE and set SECTION.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory SUMMARY, expand DESCRIPTION, and move SECTION
ahead of LICENSE/LIC_FILES_CHKSUM.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory SUMMARY and move SECTION ahead of
LICENSE/LIC_FILES_CHKSUM.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory SUMMARY and expand DESCRIPTION accordingly.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
| |
Add the missing mandatory SUMMARY and set SECTION in the shared
qoriq-atf-2.12.inc, covering both qoriq-atf and qoriq-atf-tools, and
expand DESCRIPTION so it is more descriptive than the summary.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE, expand DESCRIPTION, and
move SECTION ahead of LICENSE/LIC_FILES_CHKSUM.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE, expand DESCRIPTION, and
move SECTION ahead of LICENSE.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory DESCRIPTION.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory SUMMARY and expand DESCRIPTION so it is more
descriptive than the summary.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory HOMEPAGE.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory DESCRIPTION and HOMEPAGE, set SECTION, and
move PACKAGE_ARCH ahead of the PACKAGES/FILES block.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE and set SECTION.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE, set SECTION and move
PACKAGE_ARCH ahead of FILES.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE and set SECTION.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE, set SECTION, give
DESCRIPTION distinct wording, and move PACKAGE_ARCH ahead of FILES.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Remove a consecutive blank line (oelint.newline.consecutive). No
functional change.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Remove a consecutive blank line (oelint.newline.consecutive). No
functional change.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Remove a consecutive blank line (oelint.newline.consecutive). No
functional change.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Order DEPENDS alphabetically per oelint.vars.dependsordered. No
functional change.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Order DEPENDS alphabetically per oelint.vars.dependsordered. No
functional change.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|