There is currently both an incompatible-license and a
license-incompatible QA message. This is very confusing.
However, license-incompatible is only used to output a message when a
package is included in an image despite it having a license that is
normally incompatible (by using the INCOMPATIBLE_LICENSE_EXCEPTIONS
variable). To better match how it is used and to distinguish it from
incompatible-license, rename it to license-exception.

Signed-off-by: Peter Kjellerstedt <[email protected]>
---
 meta/classes-global/insane.bbclass               | 2 +-
 meta/classes-recipe/license_image.bbclass        | 2 +-
 meta/lib/oeqa/selftest/cases/incompatible_lic.py | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/classes-global/insane.bbclass 
b/meta/classes-global/insane.bbclass
index 04eb88364e5..6a84c4aa74d 100644
--- a/meta/classes-global/insane.bbclass
+++ b/meta/classes-global/insane.bbclass
@@ -34,7 +34,7 @@ ERROR_QA ?= "dev-so debug-deps dev-deps debug-files arch 
pkgconfig la \
             mime mime-xdg unlisted-pkg-lics unhandled-features-check \
             missing-update-alternatives missing-ptest \
             license-exists license-no-generic license-syntax license-format \
-            license-incompatible license-file-missing obsolete-license \
+            license-exception license-file-missing obsolete-license \
             libdir xorg-driver-abi buildpaths \
             dep-cmp pkgvarcheck perm-config perm-line perm-link \
             packages-list pkgv-undefined var-undefined \
diff --git a/meta/classes-recipe/license_image.bbclass 
b/meta/classes-recipe/license_image.bbclass
index 19b3dc55ba2..0e953856a63 100644
--- a/meta/classes-recipe/license_image.bbclass
+++ b/meta/classes-recipe/license_image.bbclass
@@ -72,7 +72,7 @@ def write_license_files(d, license_manifest, pkg_dic, 
rootfs=True):
             else:
                 incompatible_licenses = incompatible_pkg_license(d, 
bad_licenses, pkg_dic[pkg]["LICENSE"])
                 if incompatible_licenses:
-                    oe.qa.handle_error('license-incompatible', "Including %s 
with incompatible license(s) %s into the image, because it has been allowed by 
exception list." %(pkg, ' '.join(incompatible_licenses)), d)
+                    oe.qa.handle_error('license-exception', "Including %s with 
incompatible license(s) %s into the image, because it has been allowed by 
exception list." %(pkg, ' '.join(incompatible_licenses)), d)
             try:
                 (pkg_dic[pkg]["LICENSE"], pkg_dic[pkg]["LICENSES"]) = \
                     oe.license.manifest_licenses(pkg_dic[pkg]["LICENSE"],
diff --git a/meta/lib/oeqa/selftest/cases/incompatible_lic.py 
b/meta/lib/oeqa/selftest/cases/incompatible_lic.py
index 4c228de6a33..439c1b9f0ec 100644
--- a/meta/lib/oeqa/selftest/cases/incompatible_lic.py
+++ b/meta/lib/oeqa/selftest/cases/incompatible_lic.py
@@ -128,7 +128,7 @@ INCOMPATIBLE_LICENSE:pn-core-image-minimal = "GPL-3.0* 
LGPL-3.0*"
         bitbake('core-image-minimal')
 
     def test_bash_license_exceptions(self):
-        self.write_config(self.default_config() + 
'\nINCOMPATIBLE_LICENSE_EXCEPTIONS:pn-core-image-minimal = 
"bash:GPL-3.0-or-later"\nERROR_QA:remove = "license-incompatible"')
+        self.write_config(self.default_config() + 
'\nINCOMPATIBLE_LICENSE_EXCEPTIONS:pn-core-image-minimal = 
"bash:GPL-3.0-or-later"\nERROR_QA:remove = "license-exception"')
 
         bitbake('core-image-minimal')
 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#203233): 
https://lists.openembedded.org/g/openembedded-core/message/203233
Mute This Topic: https://lists.openembedded.org/mt/107864351/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to