Re: [PATCH 1/4] image.bbclass: Separate out image generation into a new task, do_image

[Jonathan Liu <net147@gmail.com>]

Hi Richard,

I have issues building a core-image-minimal Raspberry Pi 2 SD image 
after this series of patches using openembedded-core master 
(b3b679d5be86f73d1a06c7230cb00872f0a407b5) and meta-raspberrypi master 
(0d8e9271855ccb9bc21bdc0743f8e0f6ff0d2d5c).

| DEBUG: Executing python function set_image_size
| DEBUG: Python function set_image_size finished
| DEBUG: Executing shell function do_image_rpi-sdimg
| 
/home/build/git/pi2/build/tmp/work/raspberrypi2-custom-linux-gnueabi/core-image-minimal/1.0-r0/temp/run.do_image_rpi-sdimg.8349: 
line 108: do_image_rpi-sdimg: command not found
| WARNING: 
/home/build/git/pi2/build/tmp/work/raspberrypi2-custom-linux-gnueabi/core-image-minimal/1.0-r0/temp/run.do_image_rpi-sdimg.8349:1 
exit 127 from
|   do_image_rpi-sdimg
| ERROR: Function failed: do_image_rpi-sdimg (log file is located at 
/home/build/git/pi2/build/tmp/work/raspberrypi2-custom-linux-gnueabi/core-image-minimal/1.0-r0/temp/log.do_image_rpi-sdimg.8349)

openembedded-core 38237b7ac53c416f85c4a70a61acafc3404c8b5f - ok
openembedded-core b3b679d5be86f73d1a06c7230cb00872f0a407b5 - fails

Any ideas?

Regards,
Jonathan

On 7/01/2016 9:57 AM, Richard Purdie wrote:
> I've heard complaints from people trying to create more interesting image
> types about how hard it is to understand the rootfs/image generation code
> and that its a pain to develop/test/debug.
>
> Having looked at it myself, the internal construction of shell functions which
> then gets passed into a multiprocessing pool is rather convoluted and it places
> rather odd constraints on when variables are expanded. Its therefore no wonder
> people find it confusing/complex.
>
> This patch starts the process of splitting this up by separating out image
> generation from the do_rootfs task into a new do_image task.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>
> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
> index 81971fe..77a8548 100644
> --- a/meta/classes/image.bbclass
> +++ b/meta/classes/image.bbclass
> @@ -207,7 +207,6 @@ PACKAGE_EXCLUDE[type] = "list"
>   
>   fakeroot python do_rootfs () {
>       from oe.rootfs import create_rootfs
> -    from oe.image import create_image
>       from oe.manifest import create_manifest
>   
>       # Handle package exclusions
> @@ -244,9 +243,6 @@ fakeroot python do_rootfs () {
>   
>       # Generate rootfs
>       create_rootfs(d)
> -
> -    # generate final images
> -    create_image(d)
>   }
>   do_rootfs[dirs] = "${TOPDIR}"
>   do_rootfs[lockfiles] += "${IMAGE_ROOTFS}.lock"
> @@ -254,6 +250,16 @@ do_rootfs[cleandirs] += "${S}"
>   do_rootfs[umask] = "022"
>   addtask rootfs before do_build
>   
> +fakeroot python do_image () {
> +    from oe.image import create_image
> +
> +    # generate final images
> +    create_image(d)
> +}
> +do_image[dirs] = "${TOPDIR}"
> +do_image[umask] = "022"
> +addtask do_image after do_rootfs before do_build
> +
>   MULTILIBRE_ALLOW_REP =. "${base_bindir}|${base_sbindir}|${bindir}|${sbindir}|${libexecdir}|${sysconfdir}|${nonarch_base_libdir}/udev|/lib/modules/[^/]*/modules.*|"
>   MULTILIB_CHECK_FILE = "${WORKDIR}/multilib_check.py"
>   MULTILIB_TEMP_ROOTFS = "${WORKDIR}/multilib"
>
>