<div dir="ltr"><div>Hi Robert,</div><div><br></div><div>I understand. Though LICENSE_CREATE_PACKAGE is disabled by default in OE-core, if some users want to create ${PN}-lic package for each recipe</div><div>via LICENSE_CREATE_PACKAGE = "1", then they are going to have host contamination warnings for each recipe.</div><div><br></div><div>The above commit indicated by Martin does create ${PN}-lic package when</div><div>LICENSE_CREATE_PACKAGE = "1".</div><div><br></div><div>Regards,</div><div>Jagadeesh</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 25, 2015 at 4:04 PM, Martin Jansa <span dir="ltr"><<a href="mailto:martin.jansa@gmail.com" target="_blank">martin.jansa@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Nov 25, 2015 at 04:42:57PM +0800, Robert Yang wrote:<br>
><br>
> Hi Jagadeesh,<br>
><br>
> Thanks, usually, patch should go into master branch before go into stable<br>
> branch such as jethro. IMHO, jethro branch may not take this fix even if<br>
> master merges it. The problem is that oe-core doesn't create package for<br>
> ${PN}-lic atm, this patch is treated as part of the enhancement which is<br>
> used for working with creating ${PN}-lic.<br>
<br>
</span>Why do you think it isn't able to create ${PN}-lic when configured to do<br>
so?<br>
<br>
commit dc78ef91a2bf01efb8028c9afbe69e506e016265<br>
Author: Martin Jansa <<a href="mailto:martin.jansa@gmail.com">martin.jansa@gmail.com</a>><br>
Date: Thu Jan 17 12:49:44 2013 +0100<br>
<br>
license.bbclass: package license texts to PN-lic when<br>
LICENSE_CREATE_PACKAGE is enabled<br>
<br>
is already in dylan release.<br>
<div class="HOEnZb"><div class="h5"><br>
> // Robert<br>
><br>
> On 11/25/2015 02:54 PM, Jagadeesh Krishnanjanappa wrote:<br>
> > We get below host contamination warnings of license files for<br>
> > each recipe, when we try to create a separate ${PN}-lic package (which<br>
> > contains license files), by setting LICENSE_CREATE_PACKAGE equal to "1"<br>
> > in local.conf.<br>
> ><br>
> > -- snip --<br>
> > WARNING: QA Issue: libcgroup: /libcgroup-lic/usr/share/licenses/libcgroup/generic_LGPLv2.1 is owned by uid 5001, which is the same as the user running bitbake. This may be due to host contamination [host-user-contaminated]<br>
> > WARNING: QA Issue: attr: /attr-lic/usr/share/licenses/attr/libattr.c is owned by uid 5001, which is the same as the user running bitbake. This may be due to host contamination [host-user-contaminated]<br>
> > WARNING: QA Issue: bash: /bash-lic/usr/share/licenses/bash/COPYING is owned by uid 5001, which is the same as the user running bitbake. This may be due to host contamination [host-user-contaminated]<br>
> > -- CUT --<br>
> ><br>
> > Since the license files from source and OE-core, are populated in a normal<br>
> > shell environment rather in pseudo environment (fakeroot); the ownership of<br>
> > these files will be same as host user running bitbake. During the do_package<br>
> > task (which runs in pseudo environment (fakeroot)), os.link preserves the<br>
> > ownership of these license files as host user instead of root user.<br>
> > This causes license files to have UID same as host user id and resulting in<br>
> > above warnings during do_package_qa task.<br>
> ><br>
> > Changing ownership of license files to root user under fakeroot environment<br>
> > will solve above warnings, and on exiting fakeroot environment the license<br>
> > files will continue to be owned by host user.<br>
> ><br>
> > Signed-off-by: Jagadeesh Krishnanjanappa <<a href="mailto:jkrishnanjanappa@mvista.com">jkrishnanjanappa@mvista.com</a>><br>
> > ---<br>
> > meta/classes/license.bbclass | 1 +<br>
> > 1 file changed, 1 insertion(+)<br>
> ><br>
> > diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass<br>
> > index c616a20..d400ac0 100644<br>
> > --- a/meta/classes/license.bbclass<br>
> > +++ b/meta/classes/license.bbclass<br>
> > @@ -185,6 +185,7 @@ def copy_license_files(lic_files_paths, destdir):<br>
> > os.remove(dst)<br>
> > if os.access(src, os.W_OK) and (os.stat(src).st_dev == os.stat(destdir).st_dev):<br>
> > os.link(src, dst)<br>
> > + os.chown(dst,os.getuid(),os.getgid())<br>
> > else:<br>
> > shutil.copyfile(src, dst)<br>
> > except Exception as e:<br>
> ><br>
</div></div><span class="HOEnZb"><font color="#888888">> --<br>
> _______________________________________________<br>
> Openembedded-core mailing list<br>
> <a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a><br>
> <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
<br>
--<br>
Martin 'JaMa' Jansa jabber: <a href="mailto:Martin.Jansa@gmail.com">Martin.Jansa@gmail.com</a><br>
</font></span></blockquote></div><br></div>