[PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[Tom Hochstein]

The build tool wayland-scanner was missing from the native toolchain
of the Yocto Project SDK build.

Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
---
 meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
index 0c6a530..2c2abed 100644
--- a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
+++ b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
@@ -24,6 +24,7 @@ RDEPENDS_${PN} = "\
     nativesdk-makedevs \
     nativesdk-smartpm \
     nativesdk-postinst-intercept \
+    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', '', d)} \
     "
 
 RDEPENDS_${PN}_darwin = "\
-- 
1.9.1

Re: [PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[Otavio Salvador]

On Thu, Jul 7, 2016 at 8:24 PM, Tom Hochstein <tom.hochstein@nxp.com> wrote:
> The build tool wayland-scanner was missing from the native toolchain
> of the Yocto Project SDK build.
>
> Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>

Acked-by: Otavio Salvador <otavio@ossystems.com.br>

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

Re: [PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[Richard Purdie]

On Tue, 2018-08-21 at 16:10 +0900, Seokha Ko wrote:
> currently nativesdk-wayland-dev package copies bin directory which 
> includes wayland-scanner only.
> 
> And, IMHO, if there are other tools for dev package, we need them
> for 
> nativesdk target also.

wayland-dev has all kinds of dependencies which we do not need in the
SDK. This patch will significantly increase the size of the SDK
unnecessarily.

We need to put the scanner into a separate package and include that.

Cheers,

Richard

Re: [PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[Seokha Ko]

currently nativesdk-wayland-dev package copies bin directory which 
includes wayland-scanner only.

And, IMHO, if there are other tools for dev package, we need them for 
nativesdk target also.

Thank you.


On 08/21/2018 09:50 AM, Khem Raj wrote:
> On Mon, Aug 20, 2018 at 4:26 PM Seokha Ko <sukdo399@daum.net> wrote:
>> The wayland-scanner was missing from the native toolchain for SDK build
>> because it's copied from wayland dev package.
>>
>> Signed-off-by: Seokha Ko <sukdo399@daum.net>
>> ---
>>   .../packagegroups/nativesdk-packagegroup-sdk-host.bb            | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
>> index e2f6169994..4dc6eea23e 100644
>> --- a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
>> +++ b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
>> @@ -23,7 +23,7 @@ RDEPENDS_${PN} = "\
>>       nativesdk-makedevs \
>>       nativesdk-dnf \
>>       nativesdk-cmake \
>> -    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', '', d)} \
>> +    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland-dev', '', d)} \
> Do we need the whole dev package ? if not I would suggest to package
> it into a separate output package
> and include that along with nativesdk-wayland
>
>>       nativesdk-sdk-provides-dummy \
>>       "
>>
>> --
>> 2.17.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[Seokha Ko]

[-- Attachment #1: Type: text/html, Size: 4561 bytes --]

Re: [PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[Khem Raj]

On Mon, Aug 20, 2018 at 4:26 PM Seokha Ko <sukdo399@daum.net> wrote:
>
> The wayland-scanner was missing from the native toolchain for SDK build
> because it's copied from wayland dev package.
>
> Signed-off-by: Seokha Ko <sukdo399@daum.net>
> ---
>  .../packagegroups/nativesdk-packagegroup-sdk-host.bb            | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
> index e2f6169994..4dc6eea23e 100644
> --- a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
> +++ b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
> @@ -23,7 +23,7 @@ RDEPENDS_${PN} = "\
>      nativesdk-makedevs \
>      nativesdk-dnf \
>      nativesdk-cmake \
> -    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', '', d)} \
> +    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland-dev', '', d)} \

Do we need the whole dev package ? if not I would suggest to package
it into a separate output package
and include that along with nativesdk-wayland

>      nativesdk-sdk-provides-dummy \
>      "
>
> --
> 2.17.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

[PATCH] nativesdk-packagegroup-sdk-host: Add wayland-scanner to the SDK native toolchain

[Seokha Ko]

The wayland-scanner was missing from the native toolchain for SDK build
because it's copied from wayland dev package.

Signed-off-by: Seokha Ko <sukdo399@daum.net>
---
 .../packagegroups/nativesdk-packagegroup-sdk-host.bb            | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
index e2f6169994..4dc6eea23e 100644
--- a/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
+++ b/meta/recipes-core/packagegroups/nativesdk-packagegroup-sdk-host.bb
@@ -23,7 +23,7 @@ RDEPENDS_${PN} = "\
     nativesdk-makedevs \
     nativesdk-dnf \
     nativesdk-cmake \
-    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', '', d)} \
+    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland-dev', '', d)} \
     nativesdk-sdk-provides-dummy \
     "
 
-- 
2.17.1