* [PATCH 1/2] libtool: Only package libltdl for the target
@ 2021-08-27 21:57 Saul Wold
2021-08-27 21:57 ` [PATCH 2/2] quilt: Only package guard " Saul Wold
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Saul Wold @ 2021-08-27 21:57 UTC (permalink / raw)
To: openembedded-core
The libltdl package is not needed for -native or -cross, so make it's
only packaged for the target class.
Signed-off-by: Saul Wold <saul.wold@windriver.com>
---
meta/recipes-devtools/libtool/libtool-2.4.6.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/libtool/libtool-2.4.6.inc b/meta/recipes-devtools/libtool/libtool-2.4.6.inc
index 6748d7468e..38982bf67f 100644
--- a/meta/recipes-devtools/libtool/libtool-2.4.6.inc
+++ b/meta/recipes-devtools/libtool/libtool-2.4.6.inc
@@ -43,7 +43,7 @@ CACHED_CONFIGUREVARS += "ac_cv_path_GREP=grep"
DEPENDS = "libtool-native"
-PACKAGES =+ "libltdl"
+PACKAGES:prepend:class-target =+ "libltdl"
FILES:${PN} += "${datadir}/aclocal"
FILES:${PN}-dev:remove = "${datadir}/aclocal"
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] quilt: Only package guard for the target
2021-08-27 21:57 [PATCH 1/2] libtool: Only package libltdl for the target Saul Wold
@ 2021-08-27 21:57 ` Saul Wold
2021-08-27 23:13 ` [OE-core] " Khem Raj
2021-08-30 20:19 ` Richard Purdie
2021-08-27 23:12 ` [OE-core] [PATCH 1/2] libtool: Only package libltdl " Khem Raj
2021-08-30 20:18 ` Richard Purdie
2 siblings, 2 replies; 7+ messages in thread
From: Saul Wold @ 2021-08-27 21:57 UTC (permalink / raw)
To: openembedded-core
The guard binary and documentaiton should only be be packaged for the
target class.
Signed-off-by: Saul Wold <saul.wold@windriver.com>
---
meta/recipes-devtools/quilt/quilt.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/quilt/quilt.inc b/meta/recipes-devtools/quilt/quilt.inc
index f85de384d2..41d30e89aa 100644
--- a/meta/recipes-devtools/quilt/quilt.inc
+++ b/meta/recipes-devtools/quilt/quilt.inc
@@ -73,7 +73,7 @@ do_install_ptest() {
cp ${WORKDIR}/test.sh ${D}${PTEST_PATH}
}
-PACKAGES += "guards guards-doc"
+PACKAGES:append:class-target += "guards guards-doc"
FILES:${PN} = "${sysconfdir} ${datadir}/quilt \
${bindir}/quilt ${libdir}/quilt"
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [OE-core] [PATCH 1/2] libtool: Only package libltdl for the target
2021-08-27 21:57 [PATCH 1/2] libtool: Only package libltdl for the target Saul Wold
2021-08-27 21:57 ` [PATCH 2/2] quilt: Only package guard " Saul Wold
@ 2021-08-27 23:12 ` Khem Raj
2021-08-30 20:18 ` Richard Purdie
2 siblings, 0 replies; 7+ messages in thread
From: Khem Raj @ 2021-08-27 23:12 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
On Fri, Aug 27, 2021 at 2:58 PM Saul Wold <Saul.Wold@windriver.com> wrote:
>
> The libltdl package is not needed for -native or -cross, so make it's
> only packaged for the target class.
>
> Signed-off-by: Saul Wold <saul.wold@windriver.com>
> ---
> meta/recipes-devtools/libtool/libtool-2.4.6.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/libtool/libtool-2.4.6.inc b/meta/recipes-devtools/libtool/libtool-2.4.6.inc
> index 6748d7468e..38982bf67f 100644
> --- a/meta/recipes-devtools/libtool/libtool-2.4.6.inc
> +++ b/meta/recipes-devtools/libtool/libtool-2.4.6.inc
> @@ -43,7 +43,7 @@ CACHED_CONFIGUREVARS += "ac_cv_path_GREP=grep"
>
> DEPENDS = "libtool-native"
>
> -PACKAGES =+ "libltdl"
> +PACKAGES:prepend:class-target =+ "libltdl"
We dont need =+ with prepend and add appropriate space as well.
> FILES:${PN} += "${datadir}/aclocal"
>
> FILES:${PN}-dev:remove = "${datadir}/aclocal"
> --
> 2.25.1
>
>
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [PATCH 2/2] quilt: Only package guard for the target
2021-08-27 21:57 ` [PATCH 2/2] quilt: Only package guard " Saul Wold
@ 2021-08-27 23:13 ` Khem Raj
2021-08-30 20:19 ` Richard Purdie
1 sibling, 0 replies; 7+ messages in thread
From: Khem Raj @ 2021-08-27 23:13 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
On Fri, Aug 27, 2021 at 2:58 PM Saul Wold <Saul.Wold@windriver.com> wrote:
>
> The guard binary and documentaiton should only be be packaged for the
> target class.
>
> Signed-off-by: Saul Wold <saul.wold@windriver.com>
> ---
> meta/recipes-devtools/quilt/quilt.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/quilt/quilt.inc b/meta/recipes-devtools/quilt/quilt.inc
> index f85de384d2..41d30e89aa 100644
> --- a/meta/recipes-devtools/quilt/quilt.inc
> +++ b/meta/recipes-devtools/quilt/quilt.inc
> @@ -73,7 +73,7 @@ do_install_ptest() {
> cp ${WORKDIR}/test.sh ${D}${PTEST_PATH}
> }
>
> -PACKAGES += "guards guards-doc"
> +PACKAGES:append:class-target += "guards guards-doc"
>
append and += together is not necessary
> FILES:${PN} = "${sysconfdir} ${datadir}/quilt \
> ${bindir}/quilt ${libdir}/quilt"
> --
> 2.25.1
>
>
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [PATCH 1/2] libtool: Only package libltdl for the target
2021-08-27 21:57 [PATCH 1/2] libtool: Only package libltdl for the target Saul Wold
2021-08-27 21:57 ` [PATCH 2/2] quilt: Only package guard " Saul Wold
2021-08-27 23:12 ` [OE-core] [PATCH 1/2] libtool: Only package libltdl " Khem Raj
@ 2021-08-30 20:18 ` Richard Purdie
2 siblings, 0 replies; 7+ messages in thread
From: Richard Purdie @ 2021-08-30 20:18 UTC (permalink / raw)
To: Saul Wold, openembedded-core
On Fri, 2021-08-27 at 14:57 -0700, Saul Wold wrote:
> The libltdl package is not needed for -native or -cross, so make it's
> only packaged for the target class.
>
> Signed-off-by: Saul Wold <saul.wold@windriver.com>
> ---
> meta/recipes-devtools/libtool/libtool-2.4.6.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/libtool/libtool-2.4.6.inc b/meta/recipes-devtools/libtool/libtool-2.4.6.inc
> index 6748d7468e..38982bf67f 100644
> --- a/meta/recipes-devtools/libtool/libtool-2.4.6.inc
> +++ b/meta/recipes-devtools/libtool/libtool-2.4.6.inc
> @@ -43,7 +43,7 @@ CACHED_CONFIGUREVARS += "ac_cv_path_GREP=grep"
>
> DEPENDS = "libtool-native"
>
> -PACKAGES =+ "libltdl"
> +PACKAGES:prepend:class-target =+ "libltdl"
> FILES:${PN} += "${datadir}/aclocal"
>
> FILES:${PN}-dev:remove = "${datadir}/aclocal"
Since native and cross don't generate packages, why do we need to do this?
Rather that what the commit message says, this would seem to cause nativesdk to
be unpackaged?
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [PATCH 2/2] quilt: Only package guard for the target
2021-08-27 21:57 ` [PATCH 2/2] quilt: Only package guard " Saul Wold
2021-08-27 23:13 ` [OE-core] " Khem Raj
@ 2021-08-30 20:19 ` Richard Purdie
2021-08-31 14:35 ` Saul Wold
1 sibling, 1 reply; 7+ messages in thread
From: Richard Purdie @ 2021-08-30 20:19 UTC (permalink / raw)
To: Saul Wold, openembedded-core
On Fri, 2021-08-27 at 14:57 -0700, Saul Wold wrote:
> The guard binary and documentaiton should only be be packaged for the
> target class.
>
> Signed-off-by: Saul Wold <saul.wold@windriver.com>
> ---
> meta/recipes-devtools/quilt/quilt.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/quilt/quilt.inc b/meta/recipes-devtools/quilt/quilt.inc
> index f85de384d2..41d30e89aa 100644
> --- a/meta/recipes-devtools/quilt/quilt.inc
> +++ b/meta/recipes-devtools/quilt/quilt.inc
> @@ -73,7 +73,7 @@ do_install_ptest() {
> cp ${WORKDIR}/test.sh ${D}${PTEST_PATH}
> }
>
> -PACKAGES += "guards guards-doc"
> +PACKAGES:append:class-target += "guards guards-doc"
>
> FILES:${PN} = "${sysconfdir} ${datadir}/quilt \
> ${bindir}/quilt ${libdir}/quilt"
Again, why?
This should be fine/ignored for native/cross and applicable to nativesdk?
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [OE-core] [PATCH 2/2] quilt: Only package guard for the target
2021-08-30 20:19 ` Richard Purdie
@ 2021-08-31 14:35 ` Saul Wold
0 siblings, 0 replies; 7+ messages in thread
From: Saul Wold @ 2021-08-31 14:35 UTC (permalink / raw)
To: Richard Purdie, openembedded-core; +Cc: Joshua Watt
On 8/30/21 1:19 PM, Richard Purdie wrote:
> On Fri, 2021-08-27 at 14:57 -0700, Saul Wold wrote:
>> The guard binary and documentaiton should only be be packaged for the
>> target class.
>>
>> Signed-off-by: Saul Wold <saul.wold@windriver.com>
>> ---
>> meta/recipes-devtools/quilt/quilt.inc | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-devtools/quilt/quilt.inc b/meta/recipes-devtools/quilt/quilt.inc
>> index f85de384d2..41d30e89aa 100644
>> --- a/meta/recipes-devtools/quilt/quilt.inc
>> +++ b/meta/recipes-devtools/quilt/quilt.inc
>> @@ -73,7 +73,7 @@ do_install_ptest() {
>> cp ${WORKDIR}/test.sh ${D}${PTEST_PATH}
>> }
>>
>> -PACKAGES += "guards guards-doc"
>> +PACKAGES:append:class-target += "guards guards-doc"
>>
>> FILES:${PN} = "${sysconfdir} ${datadir}/quilt \
>> ${bindir}/quilt ${libdir}/quilt"
>
> Again, why?
>
I was doing some testing with the create_spdx code and found a condition
when I had built quilt (in this case) and then cleaned the quilt-native,
when I rebuilt quilt-native I got the following error:
ERROR: quilt-native-0.66-r0 do_create_spdx: Recipe quilt-native is
trying to create package guards which was already written by recipe
quilt. This will cause corruption, please resolve this and only provide
the package from one recipe or the other or only build one of the recipes.
> This should be fine/ignored for native/cross and applicable to nativesdk?
>
Maybe the issue is actually in the create_spdx() code,
Sau!
> Cheers,
>
> Richard
>
--
Sau!
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-08-31 14:35 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-27 21:57 [PATCH 1/2] libtool: Only package libltdl for the target Saul Wold
2021-08-27 21:57 ` [PATCH 2/2] quilt: Only package guard " Saul Wold
2021-08-27 23:13 ` [OE-core] " Khem Raj
2021-08-30 20:19 ` Richard Purdie
2021-08-31 14:35 ` Saul Wold
2021-08-27 23:12 ` [OE-core] [PATCH 1/2] libtool: Only package libltdl " Khem Raj
2021-08-30 20:18 ` Richard Purdie
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.