Re: [PATCH 0/5] [BUG# 227] rpm/deb support for meta-toolchain.

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Fri, 2011-01-28 at 18:01 +0800, Lianhao Lu wrote:
> From: Lianhao Lu <lianhao.lu@intel.com>
> 
> The purpose of this series of patches is to add rpm/deb support in meta-toolchain creation,
> and also to fix bug #227.
> 
> It makes the task populate_sdk in meta/recipes-core/meta/meta-toolchain.bb more generic
> to support installing packages from ipk, rpm, and deb files in creating meta-toolchain 
> tarball, and move the populate_sdk task out of meta-toolchain.bb recipe into a bbclass.
> 
> It added populate_sdk_xxx.bbclass for different package file format.
> 
> Since there are quite many code duplication between relevant popuplage_sdk_xxx and rootfs_xxx 
> bbclasses, it also moved the common code into the package_xxx.bbclass, based on RP's suggestion. 
> The generic function package_install_internal_xxx() may installs a list of packages, based on the
> information specified by various shell environment variables about where the target directory to
> be installed to, package architecture information, etc.
> 
> It also fixed a bug in cross-canadian.bbclass to enable the package_deb.bbclass generate correct
> deb package files for cross-canadian recipies.

These patches are good, if you have time could you:

a) rebase these patches against the merged rpm changes from Mark Hatle 
   from the rpm update to 5.4

b) Try hard to split out any logical changes you can from the bigger
patches into specific steps. For example, enhancing the log_check
function and moving those around could be a separate commit.

The idea is to make changes in clear logical steps where at all possible
so anyone reviewing the changes (such as Saul, Mark and myself) can then
more quickly "see" the changes are correct and also, if there are
problems, we can bisect the commits to track down which change the
problem is from.

Cheers,

Richard