[OE-core] [PATCH 3/3] prelink: do not use 'exit 0' to incorrectly postpone to first boot

[Richard Purdie]

On Wed, 2018-09-05 at 08:36 -0400, Hongxu Jia wrote:
> After postinst was executed at do_rootfs successfully,
> there will be no first boot to redo.
> 
> Since `229f4e9 package.bbclass: add support for
> pkg_postinst_ontarget()' applied in oe-core, use
> pkg_postinst_ontarget to instead.
> 
> Signed-off-by: Hongxu Jia <hongxu.jia at windriver.com>
> ---
>  meta/recipes-devtools/prelink/prelink_git.bb | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/meta/recipes-devtools/prelink/prelink_git.bb
> b/meta/recipes-devtools/prelink/prelink_git.bb
> index c5eaedd..c110bd2 100644
> --- a/meta/recipes-devtools/prelink/prelink_git.bb
> +++ b/meta/recipes-devtools/prelink/prelink_git.bb
> @@ -158,13 +158,7 @@ do_install_append () {
>  # Prelinking during a cross install should be handled by the image-
> prelink
>  # bbclass.  If the user desires this to run on the target at first
> boot
>  # they will need to create a custom boot script.
> -pkg_postinst_prelink() {
> -#!/bin/sh
> -
> -if [ "x$D" != "x" ]; then
> -  exit 0
> -fi
> -
> +pkg_postinst_ontarget_prelink() {
>  prelink -a
>  }

I'm not sure this is correct. The idea here is that no postinst runs,
on target or at rootfs time (since we prelink the rootfs elsewhere at
rootfs time). The only time a postinst runs is if the package is
actually installed onto a target system later on a running system.

I think you change the behaviour here so that it would run on the
target system at first boot?

Cheers,

Richard