tsntool: Add SECTION, tidy SRC_URI and document findings per oelint

Address the oelint findings for the recipe:

- Add SECTION = "console/network".
- Drop the redundant FILESEXTRAPATHS:prepend. The only local file (the
  BIN_LDFLAGS patch) lives in ${THISDIR}/${PN}, which is already part of
  the default FILESPATH (${FILE_DIRNAME}/${BPN}), so the override was a
  no-op and tripped outofcontext/fileextrapaths in this plain recipe.
- Fold the separate SRC_URI:append patch entry into the main SRC_URI so
  it precedes SRCREV (order.SRC_URI).
- Keep the explicit FILES:${PN} (the unversioned libtsn.so is
  intentionally shipped in the main package, matching the dev-so skip)
  and the INSANE_SKIP, accepting both with an inline # nooelint.

No functional change.

Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>

1 files changed, 9 insertions, 7 deletions

diff --git a/recipes-extended/tsntool/tsntool_git.bb b/recipes-extended/tsntool/tsntool_git.bb
index f2911ddab..87066fa6c 100644
--- a/recipes-extended/tsntool/tsntool_git.bb
+++ b/recipes-extended/tsntool/tsntool_git.bb
@@ -1,6 +1,7 @@
 SUMMARY = "Configure TSN funtionalitie"
 DESCRIPTION = "A tool to configure TSN funtionalities in user space"
 HOMEPAGE = "https://github.com/nxp-qoriq/tsntool"
+SECTION = "console/network"
 LICENSE = "GPL-2.0-only"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=ef58f855337069acd375717db0dbbb6d"
 
@@ -8,14 +9,10 @@ DEPENDS = "cjson libnl readline"
 
 inherit pkgconfig
 
-FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:"
-
-SRC_URI = "git://github.com/nxp-qoriq/tsntool;protocol=https;nobranch=1"
-SRCREV = "a0769e23304957a22f0cbeee6d1f547b20c3c21e"
-
-SRC_URI:append = " \
-    file://0001-tsntool-remove-redundant-parameters-from-BIN_LDFLAGS.patch \
+SRC_URI = "git://github.com/nxp-qoriq/tsntool;protocol=https;nobranch=1 \
+           file://0001-tsntool-remove-redundant-parameters-from-BIN_LDFLAGS.patch \
 "
+SRCREV = "a0769e23304957a22f0cbeee6d1f547b20c3c21e"
 
 do_configure[depends] += "virtual/kernel:do_shared_workdir"
 
@@ -30,7 +27,12 @@ do_install() {
 }
 
 PACKAGES = "${PN}-dbg ${PN}"
+# libtsn.so is an unversioned runtime .so deliberately kept in the main
+# package (hence the dev-so skip), so FILES:${PN} is set explicitly.
+# nooelint: oelint.var.filesoverride
 FILES:${PN} = "${libdir}/libtsn.so ${bindir}/*"
+# Prebuilt-style .so with vendored rpaths and runtime deps QA can't map.
+# nooelint: oelint.vars.insaneskip
 INSANE_SKIP:${PN} += "file-rdeps rpaths dev-so"
 COMPATIBLE_MACHINE = "(qoriq)"
 PARALLEL_MAKE = ""