* [PATCH] coreutils: Fix -dev package dependencies
@ 2020-03-11 11:44 Richard Purdie
2020-03-11 12:02 ` ✗ patchtest: failure for " Patchwork
2020-03-11 18:29 ` [PATCH] " Khem Raj
0 siblings, 2 replies; 5+ messages in thread
From: Richard Purdie @ 2020-03-11 11:44 UTC (permalink / raw)
To: openembedded-core
The new ptest dependencies present some challenges, in particular libmodule-build-perl
which effectively depends on gcc. In multilib images, this results in both
libXX-gcc-symlinks and libYY-gcc-symlinks being installed which conflict. This also
makes little sense.
The easiest way to fix this is to disable the automatic -dev package dependencies
and manually specify the correct ones.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
meta/recipes-core/coreutils/coreutils_8.31.bb | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/meta/recipes-core/coreutils/coreutils_8.31.bb b/meta/recipes-core/coreutils/coreutils_8.31.bb
index cba0bfe15cc..eac016319eb 100644
--- a/meta/recipes-core/coreutils/coreutils_8.31.bb
+++ b/meta/recipes-core/coreutils/coreutils_8.31.bb
@@ -149,6 +149,10 @@ inherit ptest
RDEPENDS_${PN}-ptest += "bash findutils gawk liberror-perl libmodule-build-perl make perl perl-module-file-stat python3-core sed shadow"
+# -dev automatic dependencies fails as we don't want libmodule-build-perl-dev, its too heavy
+RRECOMMENDS_coreutils-dev[nodeprrecs] = "1"
+RRECOMMENDS_coreutils-dev = "acl-dev attr-dev gmp-dev libcap-dev bash-dev findutils-dev gawk-dev shadow-dev"
+
do_install_ptest () {
install -d ${D}${PTEST_PATH}/tests
cp -r ${S}/tests/* ${D}${PTEST_PATH}/tests
--
2.25.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* ✗ patchtest: failure for coreutils: Fix -dev package dependencies
2020-03-11 11:44 [PATCH] coreutils: Fix -dev package dependencies Richard Purdie
@ 2020-03-11 12:02 ` Patchwork
2020-03-11 18:29 ` [PATCH] " Khem Raj
1 sibling, 0 replies; 5+ messages in thread
From: Patchwork @ 2020-03-11 12:02 UTC (permalink / raw)
To: Richard Purdie; +Cc: openembedded-core
== Series Details ==
Series: coreutils: Fix -dev package dependencies
Revision: 1
URL : https://patchwork.openembedded.org/series/23199/
State : failure
== Summary ==
Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:
* Issue Series does not apply on top of target branch [test_series_merge_on_head]
Suggested fix Rebase your series on top of targeted branch
Targeted branch master (currently at 04ee0e8b95)
If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).
---
Guidelines: https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] coreutils: Fix -dev package dependencies
2020-03-11 11:44 [PATCH] coreutils: Fix -dev package dependencies Richard Purdie
2020-03-11 12:02 ` ✗ patchtest: failure for " Patchwork
@ 2020-03-11 18:29 ` Khem Raj
2020-03-11 18:31 ` Richard Purdie
1 sibling, 1 reply; 5+ messages in thread
From: Khem Raj @ 2020-03-11 18:29 UTC (permalink / raw)
To: Richard Purdie, openembedded-core
On 3/11/20 4:44 AM, Richard Purdie wrote:
> The new ptest dependencies present some challenges, in particular libmodule-build-perl
> which effectively depends on gcc. In multilib images, this results in both
> libXX-gcc-symlinks and libYY-gcc-symlinks being installed which conflict. This also
> makes little sense.
>
> The easiest way to fix this is to disable the automatic -dev package dependencies
> and manually specify the correct ones.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> meta/recipes-core/coreutils/coreutils_8.31.bb | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/meta/recipes-core/coreutils/coreutils_8.31.bb b/meta/recipes-core/coreutils/coreutils_8.31.bb
> index cba0bfe15cc..eac016319eb 100644
> --- a/meta/recipes-core/coreutils/coreutils_8.31.bb
> +++ b/meta/recipes-core/coreutils/coreutils_8.31.bb
> @@ -149,6 +149,10 @@ inherit ptest
>
> RDEPENDS_${PN}-ptest += "bash findutils gawk liberror-perl libmodule-build-perl make perl perl-module-file-stat python3-core sed shadow"
>
> +# -dev automatic dependencies fails as we don't want libmodule-build-perl-dev, its too heavy
> +RRECOMMENDS_coreutils-dev[nodeprrecs] = "1"
> +RRECOMMENDS_coreutils-dev = "acl-dev attr-dev gmp-dev libcap-dev bash-dev findutils-dev gawk-dev shadow-dev"
> +
this also means that any indirect dependency in future has to be added
here manually right ? or will that situation never happen, if it will
then perhaps a note in comments might be useful about that
> do_install_ptest () {
> install -d ${D}${PTEST_PATH}/tests
> cp -r ${S}/tests/* ${D}${PTEST_PATH}/tests
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] coreutils: Fix -dev package dependencies
2020-03-11 18:29 ` [PATCH] " Khem Raj
@ 2020-03-11 18:31 ` Richard Purdie
2020-03-11 18:42 ` Khem Raj
0 siblings, 1 reply; 5+ messages in thread
From: Richard Purdie @ 2020-03-11 18:31 UTC (permalink / raw)
To: Khem Raj, openembedded-core
On Wed, 2020-03-11 at 11:29 -0700, Khem Raj wrote:
>
> On 3/11/20 4:44 AM, Richard Purdie wrote:
> > The new ptest dependencies present some challenges, in particular
> > libmodule-build-perl
> > which effectively depends on gcc. In multilib images, this results
> > in both
> > libXX-gcc-symlinks and libYY-gcc-symlinks being installed which
> > conflict. This also
> > makes little sense.
> >
> > The easiest way to fix this is to disable the automatic -dev
> > package dependencies
> > and manually specify the correct ones.
> >
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > ---
> > meta/recipes-core/coreutils/coreutils_8.31.bb | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/meta/recipes-core/coreutils/coreutils_8.31.bb
> > b/meta/recipes-core/coreutils/coreutils_8.31.bb
> > index cba0bfe15cc..eac016319eb 100644
> > --- a/meta/recipes-core/coreutils/coreutils_8.31.bb
> > +++ b/meta/recipes-core/coreutils/coreutils_8.31.bb
> > @@ -149,6 +149,10 @@ inherit ptest
> >
> > RDEPENDS_${PN}-ptest += "bash findutils gawk liberror-perl
> > libmodule-build-perl make perl perl-module-file-stat python3-core
> > sed shadow"
> >
> > +# -dev automatic dependencies fails as we don't want libmodule-
> > build-perl-dev, its too heavy
> > +RRECOMMENDS_coreutils-dev[nodeprrecs] = "1"
> > +RRECOMMENDS_coreutils-dev = "acl-dev attr-dev gmp-dev libcap-dev
> > bash-dev findutils-dev gawk-dev shadow-dev"
> > +
>
> this also means that any indirect dependency in future has to be
> added here manually right ? or will that situation never happen, if
> it will then perhaps a note in comments might be useful about that
If DEPENDS change, we may need to tweak this, yes.
The whole -dev thing has a bug open about reworking it and its a mess
already so this is really a bandaid for now.
I'd rather have this and coreutils-ptest than not have the ptests.
I'm trying to be pragmatic about it! :)
Cheers,
Richard
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] coreutils: Fix -dev package dependencies
2020-03-11 18:31 ` Richard Purdie
@ 2020-03-11 18:42 ` Khem Raj
0 siblings, 0 replies; 5+ messages in thread
From: Khem Raj @ 2020-03-11 18:42 UTC (permalink / raw)
To: Richard Purdie, openembedded-core
On 3/11/20 11:31 AM, Richard Purdie wrote:
> On Wed, 2020-03-11 at 11:29 -0700, Khem Raj wrote:
>>
>> On 3/11/20 4:44 AM, Richard Purdie wrote:
>>> The new ptest dependencies present some challenges, in particular
>>> libmodule-build-perl
>>> which effectively depends on gcc. In multilib images, this results
>>> in both
>>> libXX-gcc-symlinks and libYY-gcc-symlinks being installed which
>>> conflict. This also
>>> makes little sense.
>>>
>>> The easiest way to fix this is to disable the automatic -dev
>>> package dependencies
>>> and manually specify the correct ones.
>>>
>>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>>> ---
>>> meta/recipes-core/coreutils/coreutils_8.31.bb | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/meta/recipes-core/coreutils/coreutils_8.31.bb
>>> b/meta/recipes-core/coreutils/coreutils_8.31.bb
>>> index cba0bfe15cc..eac016319eb 100644
>>> --- a/meta/recipes-core/coreutils/coreutils_8.31.bb
>>> +++ b/meta/recipes-core/coreutils/coreutils_8.31.bb
>>> @@ -149,6 +149,10 @@ inherit ptest
>>>
>>> RDEPENDS_${PN}-ptest += "bash findutils gawk liberror-perl
>>> libmodule-build-perl make perl perl-module-file-stat python3-core
>>> sed shadow"
>>>
>>> +# -dev automatic dependencies fails as we don't want libmodule-
>>> build-perl-dev, its too heavy
>>> +RRECOMMENDS_coreutils-dev[nodeprrecs] = "1"
>>> +RRECOMMENDS_coreutils-dev = "acl-dev attr-dev gmp-dev libcap-dev
>>> bash-dev findutils-dev gawk-dev shadow-dev"
>>> +
>>
>> this also means that any indirect dependency in future has to be
>> added here manually right ? or will that situation never happen, if
>> it will then perhaps a note in comments might be useful about that
>
> If DEPENDS change, we may need to tweak this, yes.
>
> The whole -dev thing has a bug open about reworking it and its a mess
> already so this is really a bandaid for now.
>
> I'd rather have this and coreutils-ptest than not have the ptests.
>
> I'm trying to be pragmatic about it! :)
I think change is fine, I am just thinking if we will assist keeping it
sane with a comment, until dev package logic is reworked.
>
> Cheers,
>
> Richard
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-03-11 18:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-11 11:44 [PATCH] coreutils: Fix -dev package dependencies Richard Purdie
2020-03-11 12:02 ` ✗ patchtest: failure for " Patchwork
2020-03-11 18:29 ` [PATCH] " Khem Raj
2020-03-11 18:31 ` Richard Purdie
2020-03-11 18:42 ` Khem Raj
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.