On Fri, Sep 18, 2015 at 03:41:47PM +0200, Mike Looijmans wrote:
> On 18-09-15 15:14, Martin Jansa wrote:
> > * show PN as other QA checks
> > * strip PKGDEST prefix from shown path as other QA checks
> >
> > Signed-off-by: Martin Jansa <martin.ja...@gmail.com>
> > ---
> >   meta/classes/insane.bbclass | 6 ++++--
> >   1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
> > index 5c8629a..a35e546 100644
> > --- a/meta/classes/insane.bbclass
> > +++ b/meta/classes/insane.bbclass
> > @@ -980,6 +980,7 @@ def package_qa_check_host_user(path, name, d, elf, 
> > messages):
> >           return
> >
> >       dest = d.getVar('PKGDEST', True)
> > +    pn = d.getVar('PN', True)
> >       home = os.path.join(dest, 'home')
> >       if path == home or path.startswith(home + os.sep):
> >           return
> > @@ -991,14 +992,15 @@ def package_qa_check_host_user(path, name, d, elf, 
> > messages):
> >           if exc.errno != errno.ENOENT:
> >               raise
> >       else:
> > +        rootfs_path = path[len(dest):]
> >           check_uid = int(d.getVar('HOST_USER_UID', True))
> >           if stat.st_uid == check_uid:
> > -            messages["host-user-contaminated"] = "%s is owned by uid %d, 
> > which is the same as the user running bitbake. This may be due to host 
> > contamination" % (path, check_uid)
> > +            messages["host-user-contaminated"] = "%s: %s is owned by uid 
> > %d, which is the same as the user running bitbake. This may be due to host 
> > contamination" % (pn, rootfs_path, check_uid)
> >               return False
> >
> >           check_gid = int(d.getVar('HOST_USER_GID', True))
> >           if stat.st_gid == check_gid:
> > -            messages["host-user-contaminated"] = "%s is owned by gid %d, 
> > which is the same as the user running bitbake. This may be due to host 
> > contamination" % (path, check_gid)
> > +            messages["host-user-contaminated"] = "$s: %s is owned by gid 
> > %d, which is the same as the user running bitbake. This may be due to host 
> > contamination" % (pn, rootfs_path, check_gid)
> 
> I think "$s" is a typo (that will lead to a Python exception).

You're right, but doesn't lead to python exception here

even the pn was printed out in my test build with this, e.g.:

NOTE: recipe e-wm-0.18.8-r0: task do_package_qa: Started
WARNING: QA Issue: e-wm: 
/e-wm-dbg/usr/src/debug/e-wm/0.18.8-r0/build/src/bin/e_fm_shared_types.h is 
owned by uid 1026, which is the same as the user running bitbake. This may be 
due to host contamination [host-user-contaminated]
NOTE: recipe e-wm-0.18.8-r0: task do_package_qa: Succeeded

But v2 comming with %s. Thanks

-- 
Martin 'JaMa' Jansa     jabber: martin.ja...@gmail.com

Attachment: signature.asc
Description: Digital signature

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to