| Message ID | 20230219193101.338397-5-alex@linutronix.de |
|---|---|
| State | New |
| Headers | show |
| Series | [01/39] scripts/oe-setup-layers: correct variable names - layers should be called repos | expand |
On Sun, 2023-02-19 at 20:30 +0100, Alexander Kanavin wrote: > Having it as an error breaks the devtool workflow > ('devtool modify' in particular) that is meant for fixing > the fuzz. > > Signed-off-by: Alexander Kanavin <alex@linutronix.de> > --- > meta/classes-global/insane.bbclass | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Should we tweak devtool to disable this check rather than disabling globally? I'd really prefer we stopped the fuzz issues... Cheers, Richard
On Mon, 20 Feb 2023 at 16:21, Richard Purdie <richard.purdie@linuxfoundation.org> wrote: > Should we tweak devtool to disable this check rather than disabling > globally? I'd really prefer we stopped the fuzz issues... I agree, I'll see if it can be tweaked via .bbappend that devtool writes into the workspace layer. Alex
diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index ee34d5208d..e60e693ca0 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass @@ -29,7 +29,7 @@ WARN_QA ?= " libdir xorg-driver-abi buildpaths \ textrel incompatible-license files-invalid \ infodir build-deps src-uri-bad symlink-to-sysroot multilib \ - invalid-packageconfig host-user-contaminated uppercase-pn \ + invalid-packageconfig host-user-contaminated uppercase-pn patch-fuzz \ mime mime-xdg unlisted-pkg-lics unhandled-features-check \ missing-update-alternatives native-last missing-ptest \ license-exists license-no-generic license-syntax license-format \ @@ -44,7 +44,7 @@ ERROR_QA ?= "dev-so debug-deps dev-deps debug-files arch pkgconfig la \ already-stripped installed-vs-shipped ldflags compile-host-path \ install-host-path pn-overrides unknown-configure-option \ useless-rpaths rpaths staticdev empty-dirs \ - patch-fuzz patch-status-core\ + patch-status-core\ " # Add usrmerge QA check based on distro feature ERROR_QA:append = "${@bb.utils.contains('DISTRO_FEATURES', 'usrmerge', ' usrmerge', '', d)}"
Having it as an error breaks the devtool workflow ('devtool modify' in particular) that is meant for fixing the fuzz. Signed-off-by: Alexander Kanavin <alex@linutronix.de> --- meta/classes-global/insane.bbclass | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)