As I wrote in my previous mail, when it comes to licenses, it is the distribution that is important. So if you build a package (bzip2-ptest) that includes code that is GPL-3.0, it does not matter if that code is only used as a binary blob, it is still distributed in the package, and thus the package needs to be covered by the GPL-3.0 license. //Peter From: openembedded-core@lists.openembedded.org On Behalf Of Rahul Kumar Sent: den 17 maj 2020 21:18 To: Randy MacLeod Cc: Alexander Kanavin ; Richard Purdie ; OE-core ; Trevor Gamblin Subject: Re: [OE-core] [PATCH v2] bzip2: Add test suite for bzip2 Hi Randy, patch has been accepted by bzip2 community with additional changes. https://sourceware.org/git/?p=bzip2-tests.git;a=commit;h=f9061c030a25de5b6829e1abf373057309c734c0 So I need your suggestion with respect to Yocto. Since required changes for ptest infrastructure is integrated to the bzip2-tests.git source code. So now we can just change the SRCREV with the latest commit instead of applying the patch to the bzip2-tests.git source code. And since we are just integrating the bzip2-tests source to the bzip2 recipe and not modifying any file so i think no need to add any License for it. Now patch will look like ===================================================================== --- a/meta/recipes-extended/bzip2/bzip2/Makefile.am +++ b/meta/recipes-extended/bzip2/bzip2/Makefile.am @@ -46,6 +46,7 @@ runtest: else echo "FAIL: sample2 decompress"; fi @if cmp sample3.tst sample3.ref; then echo "PASS: sample3 decompress";\ else echo "FAIL: sample3 decompress"; fi + ./bzip2-tests/run-tests.sh --tests-dir="$(PWD)/bzip2-tests" install-ptest: sed -n '/^runtest:/,/^install-ptest:/{/^install-ptest:/!p}' \ @@ -56,6 +57,7 @@ install-ptest: cp $(srcdir)/sample1.bz2 $(DESTDIR)/ cp $(srcdir)/sample2.bz2 $(DESTDIR)/ cp $(srcdir)/sample3.bz2 $(DESTDIR)/ + cp -rf $(srcdir)/../git $(DESTDIR)/bzip2-tests ln -s $(bindir)/bzip2 $(DESTDIR)/bzip2 install-exec-hook: diff --git a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb index d58f553..c5aa690 100644 --- a/meta/recipes-extended/bzip2/bzip2_1.0.8.bb +++ b/meta/recipes-extended/bzip2/bzip2_1.0.8.bb @@ -8,13 +8,17 @@ LICENSE = "bzip2-1.0.6" LIC_FILES_CHKSUM = "file://LICENSE;beginline=4;endline=37;md5=600af43c50f1fcb82e32f19b32df4664" SRC_URI = "https://sourceware.org/pub/${BPN}/${BPN}-${PV}.tar.gz \ + git://sourceware.org/git/bzip2-tests.git;name=bzip2-tests \ file://configure.ac;subdir=${BP} \ file://Makefile.am;subdir=${BP} \ file://run-ptest \ " + SRC_URI[md5sum] = "67e051268d0c475ea773822f7500d0e5" SRC_URI[sha256sum] = "ab5a03176ee106d3f0fa90e381da478ddae405918153cca248e682cd0c4a2269" +SRCREV_bzip2-tests = "f9061c030a25de5b6829e1abf373057309c734c0" << this commit Id includes latest charges for ptest infrastruct. + UPSTREAM_CHECK_URI = "https://www.sourceware.org/pub/bzip2/" PACKAGES =+ "libbz2" @@ -39,7 +43,7 @@ do_install_ptest () { FILES_libbz2 = "${libdir}/lib*${SOLIBS}" -RDEPENDS_${PN}-ptest += "make" +RDEPENDS_${PN}-ptest += "make bash" PROVIDES_append_class-native = " bzip2-replacement-native" BBCLASSEXTEND = "native nativesdk" ================================================================================= Kindly comment on it and feel free to point out, if i am wrong at any place. Thanks & Regards, Rahul Kumar Software Engineer,Linux Solutions Engineering Group,Montavista Software LLC Email Id: rahulk@mvista.com On Fri, May 1, 2020 at 6:56 AM Randy MacLeod > wrote: On 2020-04-27 3:39 p.m., Alexander Kanavin wrote: > You need to first see from the failure page which configuration is > failing, for example non-gpl3 is one such. > > Then you find that configuration in config.json. The below should > hopefully be self-explanatory in how you should set up the build? > > |"non-gpl3" : { "NEEDREPOS" : ["poky", "meta-gplv2"], "MACHINE" : > "qemux86", "BBTARGETS" : "core-image-minimal core-image-full-cmdline", > "extravars" : [ "INCOMPATIBLE_LICENSE = '*GPLv3'", "WARN_QA_remove = > 'incompatible-license'" ], "EXTRACMDS" : [ > "../../yocto-autobuilder-helper/scripts/check-gplv3" ] }, > > | > > | > | > > |Alex Hi Rahul, Sorry for my late reply. The commit log for v2 is very good now! Thanks for incorporating my --pedantic suggestions. ;-) It seems that you need a perl dependency for something (docs? $ cd .../bzip2.git $ grep -r "perl " * format.pl:#!/usr/bin/perl -w README.XML.STUFF:It uses format.pl, a perl script... Then we need to figure out how to deal with the GPLv3 issue. The buildbot output can be tedious to figure out. I haven't really spent enough time plugging away at it to be proficient yet either. Have you been able to reproduce the problems that Richard reported? If not, and you've tried for a bit, then just say so and I'll try to help tomorrow or early next week. It looks like you are packaging the test code/data with the main package not in bzip2-ptest. Have a look at: meta/recipes-support/libpcre/libpcre_8.44.bb for an example. There are many more. Also, if you look at oe-core.git: $ rgrep LICENSE_ * | grep PN you can see many examples of sub-packages with different licenses than the main package. One example is: meta/recipes-support/gnutls/gnutls_3.6.13.bb I hope that can address the buildbot problem but I haven't tried it myself yet. BTW, Trevor has gotten the YP autobuilder going at Wind River and he'll be sending a few documentation updates next week or so. That may help in case you want to reproduce the YP AB test infrastructure. I expect that you don't _have_ to do so but I think it would be good if more contributing organizations did have an instance with only limited builders of the YP AB so that we can do more testing before Richard runs our changes through the main system. Richard has cautioned that the YP AB has lots of builders each of which has many cores but I hope that we can at least do some AB checking ourselves. ../Randy > | > > > On Mon, 27 Apr 2020 at 20:54, Rahul Kumar > >> wrote: > > Hi Richard/Alexander, > > I am not able to understand how I can use the below file. > http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder-helper/tree/config.json > > did you mean to say that i have to set MACRO in local.conf based on > this file. > > *Thanks & Regards,* > Rahul Kumar > Software Engineer,Linux Solutions Engineering > Group,Montavista Software LLC > Email Id: rahulk@mvista.com > > > > > On Mon, Apr 27, 2020 at 11:46 PM Richard Purdie > > >> wrote: > > On Mon, 2020-04-27 at 18:30 +0200, Alexander Kanavin wrote: > > You need to look at configurations defined here: > > > http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder-helper/tree/config.json > > and replicate them locally. Then you can reproduce the > failures that > > the AB gets in those configurations. > > That start of the failing logs on the autobuilder also list out the > configuration options for that build. > > Cheers, > > Richard > > > > -- # Randy MacLeod # Wind River Linux