[2/2] wayland: Fix wayland-scanner missing from SDK

Submitted by Tom Hochstein on March 19, 2020, 6:34 p.m. | Patch ID: 171165

Details

Message ID 20200319183425.12294-2-tom.hochstein@nxp.com
State New
Headers show

Commit Message

Tom Hochstein March 19, 2020, 6:34 p.m.
The FILES variable is customized to package wayland-scanner
in the target -dev package. However, this is not correct for
the nativesdk packages and causes wayland-scanner to be
missing from the SDK.

Fix the nativesdk packaging by properly limiting the
wayland-scanner target packaging.

Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
---
 meta/recipes-graphics/wayland/wayland_1.18.0.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Patch hide | download patch | download mbox

diff --git a/meta/recipes-graphics/wayland/wayland_1.18.0.bb b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
index 00be3aac27..c7098483bc 100644
--- a/meta/recipes-graphics/wayland/wayland_1.18.0.bb
+++ b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
@@ -54,8 +54,8 @@  sysroot_stage_all_append_class-target () {
 	cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 ${SYSROOT_DESTDIR}/${datadir}/aclocal/
 }
 
-FILES_${PN} = "${libdir}/*${SOLIBS}"
-FILES_${PN}-dev += "${bindir} ${datadir}/wayland"
+FILES_${PN}_class-target = "${libdir}/*${SOLIBS}"
+FILES_${PN}-dev_append_class-target = " ${bindir} ${datadir}/wayland"
 
 BBCLASSEXTEND = "native nativesdk"
 

Comments

Richard Purdie March 19, 2020, 11:42 p.m.
On Thu, 2020-03-19 at 13:34 -0500, Tom Hochstein wrote:
> The FILES variable is customized to package wayland-scanner
> in the target -dev package. However, this is not correct for
> the nativesdk packages and causes wayland-scanner to be
> missing from the SDK.
> 
> Fix the nativesdk packaging by properly limiting the
> wayland-scanner target packaging.
> 
> Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
> ---
>  meta/recipes-graphics/wayland/wayland_1.18.0.bb | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-graphics/wayland/wayland_1.18.0.bb b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
> index 00be3aac27..c7098483bc 100644
> --- a/meta/recipes-graphics/wayland/wayland_1.18.0.bb
> +++ b/meta/recipes-graphics/wayland/wayland_1.18.0.bb
> @@ -54,8 +54,8 @@ sysroot_stage_all_append_class-target () {
>  	cp ${STAGING_DATADIR_NATIVE}/aclocal/wayland-scanner.m4 ${SYSROOT_DESTDIR}/${datadir}/aclocal/
>  }
>  
> -FILES_${PN} = "${libdir}/*${SOLIBS}"
> -FILES_${PN}-dev += "${bindir} ${datadir}/wayland"
> +FILES_${PN}_class-target = "${libdir}/*${SOLIBS}"
> +FILES_${PN}-dev_append_class-target = " ${bindir} ${datadir}/wayland"

Nobody can look at that and tell that its an fix for nativesdk
packaging. At the very least it needs a comment about what its doing
but I can't help feeling there must be some neater way we can do this
that overrides for nativesdk instead of misleading with target
references?

Cheers,

Richard