All of lore.kernel.org
 help / color / mirror / Atom feed
* [thud][PATCH 1/2] libaio: Extend to native
@ 2018-12-22 22:13 Peter Kjellerstedt
  2018-12-22 22:13 ` [thud][PATCH 2/2] meson: Correct use of the _append operator Peter Kjellerstedt
  2019-02-22 11:00 ` [thud][PATCH 1/2] libaio: Extend to native Peter Kjellerstedt
  0 siblings, 2 replies; 6+ messages in thread
From: Peter Kjellerstedt @ 2018-12-22 22:13 UTC (permalink / raw)
  To: openembedded-core

From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>

lvm2 currently requires libaio. So building lvm2-native will result in
the following error.

  ERROR: Required build target 'lvm2-native' has no buildable providers.
  Missing or unbuildable dependency chain was: ['lvm2-native', 'libaio-native']

Extend libaio to native to fix this issue.

Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

This is a backport from master.

 meta/recipes-extended/libaio/libaio_0.3.111.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/libaio/libaio_0.3.111.bb b/meta/recipes-extended/libaio/libaio_0.3.111.bb
index 04b50b61ec..8e1cd349a0 100644
--- a/meta/recipes-extended/libaio/libaio_0.3.111.bb
+++ b/meta/recipes-extended/libaio/libaio_0.3.111.bb
@@ -20,4 +20,4 @@ do_install () {
     oe_runmake install DESTDIR=${D}
 }
 
-BBCLASSEXTEND = "nativesdk"
+BBCLASSEXTEND = "native nativesdk"
-- 
2.12.0



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [thud][PATCH 2/2] meson: Correct use of the _append operator
  2018-12-22 22:13 [thud][PATCH 1/2] libaio: Extend to native Peter Kjellerstedt
@ 2018-12-22 22:13 ` Peter Kjellerstedt
  2018-12-23 16:48   ` akuster808
  2019-02-22 11:00 ` [thud][PATCH 1/2] libaio: Extend to native Peter Kjellerstedt
  1 sibling, 1 reply; 6+ messages in thread
From: Peter Kjellerstedt @ 2018-12-22 22:13 UTC (permalink / raw)
  To: openembedded-core

From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>

The value to SRC_URI_append_class-native was not prefixed with a space.
This was not noticed as the SRC_URI before applying the _append contains
trailing spaces. However, if one, e.g., has a .bbappend and adds to the
SRC_URI using SRC_URI += "file://foo.patch", then there no longer is any
trailing space and the _append concatenates the two URIs together,
leading to a build failue.

Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

This is a backport from master.

 meta/recipes-devtools/meson/meson.inc | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-devtools/meson/meson.inc b/meta/recipes-devtools/meson/meson.inc
index b7fb3e698a..40b29d5802 100644
--- a/meta/recipes-devtools/meson/meson.inc
+++ b/meta/recipes-devtools/meson/meson.inc
@@ -15,9 +15,10 @@ SRC_URI = "https://github.com/mesonbuild/meson/releases/download/${PV}/meson-${P
 SRC_URI[sha256sum] = "92d8afd921751261e36151643464efd3394162f69efbe8cd53e0a66b1cf395eb"
 SRC_URI[md5sum] = "31bda3519d8c0eb3438267268a78085e"
 
-SRC_URI_append_class-native = "file://0002-Make-CPU-family-warnings-fatal.patch \
-                               file://0001-Support-building-allarch-recipes-again.patch \
-                               "
+SRC_URI_append_class-native = " \
+    file://0002-Make-CPU-family-warnings-fatal.patch \
+    file://0001-Support-building-allarch-recipes-again.patch \
+"
 
 UPSTREAM_CHECK_URI = "https://github.com/mesonbuild/meson/releases"
 
-- 
2.12.0



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [thud][PATCH 2/2] meson: Correct use of the _append operator
  2018-12-22 22:13 ` [thud][PATCH 2/2] meson: Correct use of the _append operator Peter Kjellerstedt
@ 2018-12-23 16:48   ` akuster808
  0 siblings, 0 replies; 6+ messages in thread
From: akuster808 @ 2018-12-23 16:48 UTC (permalink / raw)
  To: Peter Kjellerstedt, openembedded-core



On 12/22/18 2:13 PM, Peter Kjellerstedt wrote:
> From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
>
> The value to SRC_URI_append_class-native was not prefixed with a space.
> This was not noticed as the SRC_URI before applying the _append contains
> trailing spaces. However, if one, e.g., has a .bbappend and adds to the
> SRC_URI using SRC_URI += "file://foo.patch", then there no longer is any
> trailing space and the _append concatenates the two URIs together,
> leading to a build failue.
>
> Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

This is stagged in stable/thud-nmut.

thanks.

> ---
>
> This is a backport from master.
>
>  meta/recipes-devtools/meson/meson.inc | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/meta/recipes-devtools/meson/meson.inc b/meta/recipes-devtools/meson/meson.inc
> index b7fb3e698a..40b29d5802 100644
> --- a/meta/recipes-devtools/meson/meson.inc
> +++ b/meta/recipes-devtools/meson/meson.inc
> @@ -15,9 +15,10 @@ SRC_URI = "https://github.com/mesonbuild/meson/releases/download/${PV}/meson-${P
>  SRC_URI[sha256sum] = "92d8afd921751261e36151643464efd3394162f69efbe8cd53e0a66b1cf395eb"
>  SRC_URI[md5sum] = "31bda3519d8c0eb3438267268a78085e"
>  
> -SRC_URI_append_class-native = "file://0002-Make-CPU-family-warnings-fatal.patch \
> -                               file://0001-Support-building-allarch-recipes-again.patch \
> -                               "
> +SRC_URI_append_class-native = " \
> +    file://0002-Make-CPU-family-warnings-fatal.patch \
> +    file://0001-Support-building-allarch-recipes-again.patch \
> +"
>  
>  UPSTREAM_CHECK_URI = "https://github.com/mesonbuild/meson/releases"
>  



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [thud][PATCH 1/2] libaio: Extend to native
  2018-12-22 22:13 [thud][PATCH 1/2] libaio: Extend to native Peter Kjellerstedt
  2018-12-22 22:13 ` [thud][PATCH 2/2] meson: Correct use of the _append operator Peter Kjellerstedt
@ 2019-02-22 11:00 ` Peter Kjellerstedt
  2019-02-22 15:20   ` akuster808
  1 sibling, 1 reply; 6+ messages in thread
From: Peter Kjellerstedt @ 2019-02-22 11:00 UTC (permalink / raw)
  To: Armin Kuster; +Cc: openembedded-core

*ping*

//Peter

> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org <openembedded-core-bounces@lists.openembedded.org> On Behalf Of Peter Kjellerstedt
> Sent: den 22 december 2018 23:13
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [thud][PATCH 1/2] libaio: Extend to native
> 
> From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> 
> lvm2 currently requires libaio. So building lvm2-native will result in
> the following error.
> 
>   ERROR: Required build target 'lvm2-native' has no buildable providers.
>   Missing or unbuildable dependency chain was: ['lvm2-native', 'libaio-native']
> 
> Extend libaio to native to fix this issue.
> 
> Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> 
> This is a backport from master.
> 
>  meta/recipes-extended/libaio/libaio_0.3.111.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-extended/libaio/libaio_0.3.111.bb b/meta/recipes-extended/libaio/libaio_0.3.111.bb
> index 04b50b61ec..8e1cd349a0 100644
> --- a/meta/recipes-extended/libaio/libaio_0.3.111.bb
> +++ b/meta/recipes-extended/libaio/libaio_0.3.111.bb
> @@ -20,4 +20,4 @@ do_install () {
>      oe_runmake install DESTDIR=${D}
>  }
> 
> -BBCLASSEXTEND = "nativesdk"
> +BBCLASSEXTEND = "native nativesdk"
> --
> 2.12.0



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [thud][PATCH 1/2] libaio: Extend to native
  2019-02-22 11:00 ` [thud][PATCH 1/2] libaio: Extend to native Peter Kjellerstedt
@ 2019-02-22 15:20   ` akuster808
  2019-02-25 11:38     ` Peter Kjellerstedt
  0 siblings, 1 reply; 6+ messages in thread
From: akuster808 @ 2019-02-22 15:20 UTC (permalink / raw)
  To: Peter Kjellerstedt; +Cc: openembedded-core



On 2/22/19 3:00 AM, Peter Kjellerstedt wrote:
> *ping*
>
> //Peter

you mean this?
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-extended/libaio?h=thud&id=40350b46d5c053b4336cc128922f44ce80e7da9a
>
>> -----Original Message-----
>> From: openembedded-core-bounces@lists.openembedded.org <openembedded-core-bounces@lists.openembedded.org> On Behalf Of Peter Kjellerstedt
>> Sent: den 22 december 2018 23:13
>> To: openembedded-core@lists.openembedded.org
>> Subject: [OE-core] [thud][PATCH 1/2] libaio: Extend to native
>>
>> From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
>>
>> lvm2 currently requires libaio. So building lvm2-native will result in
>> the following error.
>>
>>   ERROR: Required build target 'lvm2-native' has no buildable providers.
>>   Missing or unbuildable dependency chain was: ['lvm2-native', 'libaio-native']
>>
>> Extend libaio to native to fix this issue.
>>
>> Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> ---
>>
>> This is a backport from master.
>>
>>  meta/recipes-extended/libaio/libaio_0.3.111.bb | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-extended/libaio/libaio_0.3.111.bb b/meta/recipes-extended/libaio/libaio_0.3.111.bb
>> index 04b50b61ec..8e1cd349a0 100644
>> --- a/meta/recipes-extended/libaio/libaio_0.3.111.bb
>> +++ b/meta/recipes-extended/libaio/libaio_0.3.111.bb
>> @@ -20,4 +20,4 @@ do_install () {
>>      oe_runmake install DESTDIR=${D}
>>  }
>>
>> -BBCLASSEXTEND = "nativesdk"
>> +BBCLASSEXTEND = "native nativesdk"
>> --
>> 2.12.0



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [thud][PATCH 1/2] libaio: Extend to native
  2019-02-22 15:20   ` akuster808
@ 2019-02-25 11:38     ` Peter Kjellerstedt
  0 siblings, 0 replies; 6+ messages in thread
From: Peter Kjellerstedt @ 2019-02-25 11:38 UTC (permalink / raw)
  To: akuster808; +Cc: openembedded-core

> -----Original Message-----
> From: akuster808 <akuster808@gmail.com>
> Sent: den 22 februari 2019 16:21
> To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [thud][PATCH 1/2] libaio: Extend to native
> 
> On 2/22/19 3:00 AM, Peter Kjellerstedt wrote:
> > *ping*
> >
> > //Peter
> 
> you mean this?
> http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-
> extended/libaio?h=thud&id=40350b46d5c053b4336cc128922f44ce80e7da9a

No. That one extended the recipe to nativesdk, my patch extends it 
for native as well.

This is the one I want:

https://patchwork.openembedded.org/patch/157392/

//Peter

> >> -----Original Message-----
> >> From: openembedded-core-bounces@lists.openembedded.org
> <openembedded-core-bounces@lists.openembedded.org> On Behalf Of Peter
> Kjellerstedt
> >> Sent: den 22 december 2018 23:13
> >> To: openembedded-core@lists.openembedded.org
> >> Subject: [OE-core] [thud][PATCH 1/2] libaio: Extend to native
> >>
> >> From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> >>
> >> lvm2 currently requires libaio. So building lvm2-native will result
> in
> >> the following error.
> >>
> >>   ERROR: Required build target 'lvm2-native' has no buildable
> providers.
> >>   Missing or unbuildable dependency chain was: ['lvm2-native',
> 'libaio-native']
> >>
> >> Extend libaio to native to fix this issue.
> >>
> >> Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> >> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >> ---
> >>
> >> This is a backport from master.
> >>
> >>  meta/recipes-extended/libaio/libaio_0.3.111.bb | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/meta/recipes-extended/libaio/libaio_0.3.111.bb
> b/meta/recipes-extended/libaio/libaio_0.3.111.bb
> >> index 04b50b61ec..8e1cd349a0 100644
> >> --- a/meta/recipes-extended/libaio/libaio_0.3.111.bb
> >> +++ b/meta/recipes-extended/libaio/libaio_0.3.111.bb
> >> @@ -20,4 +20,4 @@ do_install () {
> >>      oe_runmake install DESTDIR=${D}
> >>  }
> >>
> >> -BBCLASSEXTEND = "nativesdk"
> >> +BBCLASSEXTEND = "native nativesdk"
> >> --
> >> 2.12.0


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-02-25 11:38 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-22 22:13 [thud][PATCH 1/2] libaio: Extend to native Peter Kjellerstedt
2018-12-22 22:13 ` [thud][PATCH 2/2] meson: Correct use of the _append operator Peter Kjellerstedt
2018-12-23 16:48   ` akuster808
2019-02-22 11:00 ` [thud][PATCH 1/2] libaio: Extend to native Peter Kjellerstedt
2019-02-22 15:20   ` akuster808
2019-02-25 11:38     ` Peter Kjellerstedt

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.