[bitbake-devel] [PATCH 0/9 v3][PULL] Hob: Bug fixes: image_types, runqemu/deploy functionality filter, and DISTRO setting

[Xu, Dongxiao]

On Sun, 2012-03-25 at 12:07 +0100, Richard Purdie wrote:
> On Sat, 2012-03-24 at 19:49 +0800, Dongxiao Xu wrote:
> > Hi Richard,
> > 
> > This is the third pull request for Hob bug fixes, please help to review and pull.
> > 
> > Changes from v2:
> > 1) Change the logic in set_distro(), that is, if defaultsetup is selected in Hob advanced setting dialog, we should leave DISTRO variable not set in bitbake server.
> > 2) Add a new patch to avoid hardcode TMPDIR when executing runqemu.
> > 
> > Thanks,
> > Dongxiao
> > 
> > The following changes since commit 0348ef08db9ac425d83126d2e6eb465adc28b110:
> > 
> >   Hob: Change the descriptions for view_recipe_button and view_package_button (2012-03-23 16:09:35 +0000)
> > 
> > are available in the git repository at:
> >   git://git.pokylinux.org/poky-contrib dxu4/hob-bugfix
> >   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dxu4/hob-bugfix
> > 
> > Dongxiao Xu (9):
> >   Hob: Remove unnecessary PARSE_BBFILES command
> >   Hob: Cleanup for hobeventhandler.py
> >   Hob: Initial an empty label in recipeselectionpage.py
> >   Hob: Change the Hob initialization process
> >   Hob: Fix the workaround to get image types
> >   Hob: Add a supported image type "vmdk"
> >   Hob: runqemu and deployment functionality filter
> >   Hob: Set empty DISTRO if "defaultsetup" is selected
> >   Hob: Avoid getting TMPDIR in hardcode
> 
> Merged to master, thanks.
> 
> I do have some concerns around the "runqemu and deployment functionality
> filter" in that I'd prefer it if we treated the
> RUNNABLE_MACHINE_PATTERNS as a proper regexp and matched it against
> MACHINE rather than the image name. I'd prefer to see if behave like
> COMPATIBLE_MACHINE for example.

We can change the RUNNABLE_MACHINE_PATTERNS to be regexp style, but
maybe we still need to match it with image name, since we will have two
ways to access to the image details page:
1) Through a normal build, where we are able to get the MACHINE
variable.
2) Through clicking "My images" button and load an already built image,
from which we could not get the MACHINE variable but only image name.

BTW, to implement the filter functionality, we still have a pending
patch for OE-Core that is not in master yet, see:

http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=dxu4/hob-bugfix-oecore&id=10968f8c0cf425011983ef7e0fe4c14b7da5ef74

Thanks,
Dongxiao

> 
> This is a detail we can fix in subsequent patches though.
> 
> Cheers,
> 
> Richard
>