On Mon, 2020-08-24 at 18:29 -0500, Mark Hatle wrote:
> Sync up the task definitions with the other package classes. This may not
> have been strictly necessary but will make overall maintenance easier as
> the various package classes are now in sync.
>
> Additional, there was a missing deltask in the nopackages.bbclass related
> to the package_tar which has been corrected. This could cause problems on
> native recipes when package_tar was enabled.
>
> Signed-off-by: Mark Hatle <[email protected]>
> ---
> meta/classes/nopackages.bbclass | 1 +
> meta/classes/package_tar.bbclass | 11 ++++++-----
> 2 files changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/meta/classes/nopackages.bbclass b/meta/classes/nopackages.bbclass
> index 559f5078bd..7a4f632d71 100644
> --- a/meta/classes/nopackages.bbclass
> +++ b/meta/classes/nopackages.bbclass
> @@ -2,6 +2,7 @@ deltask do_package
> deltask do_package_write_rpm
> deltask do_package_write_ipk
> deltask do_package_write_deb
> +deltask do_package_write_tar
> deltask do_package_qa
> deltask do_packagedata
> deltask do_package_setscene
This part makes sense.
> diff --git a/meta/classes/package_tar.bbclass
> b/meta/classes/package_tar.bbclass
> index ce3ab4c8e2..8946bc212a 100644
> --- a/meta/classes/package_tar.bbclass
> +++ b/meta/classes/package_tar.bbclass
> @@ -57,10 +57,8 @@ python do_package_tar () {
>
> python () {
> if d.getVar('PACKAGES') != '':
> - deps = (d.getVarFlag('do_package_write_tar', 'depends') or
> "").split()
> - deps.append('tar-native:do_populate_sysroot')
> - deps.append('virtual/fakeroot-native:do_populate_sysroot')
> - d.setVarFlag('do_package_write_tar', 'depends', " ".join(deps))
> + deps = ' tar-native:do_populate_sysroot
> virtual/fakeroot-native:do_populate_sysroot'
> + d.appendVarFlag('do_package_write_tar', 'depends', deps)
This is effectively no change but standardises so ok...
> d.setVarFlag('do_package_write_tar', 'fakeroot', "1")
> }
>
> @@ -70,4 +68,7 @@ python do_package_write_tar () {
> bb.build.exec_func("do_package_tar", d)
> }
> do_package_write_tar[dirs] = "${D}"
> -addtask package_write_tar before do_build after do_packagedata do_package
> +do_package_write_tar[depends] +=
> "${@oe.utils.build_depends_string(d.getVar('PACKAGE_WRITE_DEPS'),
> 'do_populate_sysroot')}"
Does package_tar need those dependencies? it doesn't handle postinsts
so it basically doesn't.
> +addtask package_write_tar after do_packagedata do_package
> +
> +do_build[recrdeptask] += "do_package_write_tar"
This is also incorrect since tar's have no dependencies on other things
and I suspect this difference is deliberate.
Cheers,
Richard
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#141807):
https://lists.openembedded.org/g/openembedded-core/message/141807
Mute This Topic: https://lists.openembedded.org/mt/76396875/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
