kernel.bbclass: add coreutils-native to the DEPENDS list

Message ID 20181204134233.12495-1-anders.roxell@linaro.org
State New
Headers show
Series
  • kernel.bbclass: add coreutils-native to the DEPENDS list
Related show

Commit Message

Anders Roxell Dec. 4, 2018, 1:42 p.m.
In newer kernels fold gets used, see output from a build:

.../kernel-source/scripts/atomic/atomic-tbl.sh:
   line 183: fold: command not found

Rework so that coreutils-native is in the DEPENDS list.

Signed-off-by: Anders Roxell <anders.roxell@linaro.org>

---
 meta/classes/kernel.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.19.2

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Comments

Ross Burton Dec. 4, 2018, 2:23 p.m. | #1
Can't we just add this to HOSTTOOLS instead?
On Tue, 4 Dec 2018 at 13:43, Anders Roxell <anders.roxell@linaro.org> wrote:
>

> In newer kernels fold gets used, see output from a build:

>

> .../kernel-source/scripts/atomic/atomic-tbl.sh:

>    line 183: fold: command not found

>

> Rework so that coreutils-native is in the DEPENDS list.

>

> Signed-off-by: Anders Roxell <anders.roxell@linaro.org>

> ---

>  meta/classes/kernel.bbclass | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

>

> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass

> index 880c5ef17283..8d83ca4843e3 100644

> --- a/meta/classes/kernel.bbclass

> +++ b/meta/classes/kernel.bbclass

> @@ -4,7 +4,7 @@ KERNEL_PACKAGE_NAME ??= "kernel"

>  KERNEL_DEPLOYSUBDIR ??= "${@ "" if (d.getVar("KERNEL_PACKAGE_NAME") == "kernel") else d.getVar("KERNEL_PACKAGE_NAME") }"

>

>  PROVIDES += "${@ "virtual/kernel" if (d.getVar("KERNEL_PACKAGE_NAME") == "kernel") else "" }"

> -DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc kmod-native bc-native lzop-native bison-native"

> +DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc kmod-native bc-native lzop-native bison-native coreutils-native"

>  PACKAGE_WRITE_DEPS += "depmodwrapper-cross"

>

>  do_deploy[depends] += "depmodwrapper-cross:do_populate_sysroot"

> --

> 2.19.2

>

> --

> _______________________________________________

> Openembedded-core mailing list

> Openembedded-core@lists.openembedded.org

> http://lists.openembedded.org/mailman/listinfo/openembedded-core

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
Anders Roxell Dec. 13, 2018, 10:47 a.m. | #2
On Tue, 4 Dec 2018 at 15:24, Burton, Ross <ross.burton@intel.com> wrote:
>

> Can't we just add this to HOSTTOOLS instead?


That makes sense, I'll send out a patch shortly.

Cheers,
Anders

> On Tue, 4 Dec 2018 at 13:43, Anders Roxell <anders.roxell@linaro.org> wrote:

> >

> > In newer kernels fold gets used, see output from a build:

> >

> > .../kernel-source/scripts/atomic/atomic-tbl.sh:

> >    line 183: fold: command not found

> >

> > Rework so that coreutils-native is in the DEPENDS list.

> >

> > Signed-off-by: Anders Roxell <anders.roxell@linaro.org>

> > ---

> >  meta/classes/kernel.bbclass | 2 +-

> >  1 file changed, 1 insertion(+), 1 deletion(-)

> >

> > diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass

> > index 880c5ef17283..8d83ca4843e3 100644

> > --- a/meta/classes/kernel.bbclass

> > +++ b/meta/classes/kernel.bbclass

> > @@ -4,7 +4,7 @@ KERNEL_PACKAGE_NAME ??= "kernel"

> >  KERNEL_DEPLOYSUBDIR ??= "${@ "" if (d.getVar("KERNEL_PACKAGE_NAME") == "kernel") else d.getVar("KERNEL_PACKAGE_NAME") }"

> >

> >  PROVIDES += "${@ "virtual/kernel" if (d.getVar("KERNEL_PACKAGE_NAME") == "kernel") else "" }"

> > -DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc kmod-native bc-native lzop-native bison-native"

> > +DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc kmod-native bc-native lzop-native bison-native coreutils-native"

> >  PACKAGE_WRITE_DEPS += "depmodwrapper-cross"

> >

> >  do_deploy[depends] += "depmodwrapper-cross:do_populate_sysroot"

> > --

> > 2.19.2

> >

> > --

> > _______________________________________________

> > Openembedded-core mailing list

> > Openembedded-core@lists.openembedded.org

> > http://lists.openembedded.org/mailman/listinfo/openembedded-core

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Patch

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 880c5ef17283..8d83ca4843e3 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -4,7 +4,7 @@  KERNEL_PACKAGE_NAME ??= "kernel"
 KERNEL_DEPLOYSUBDIR ??= "${@ "" if (d.getVar("KERNEL_PACKAGE_NAME") == "kernel") else d.getVar("KERNEL_PACKAGE_NAME") }"
 
 PROVIDES += "${@ "virtual/kernel" if (d.getVar("KERNEL_PACKAGE_NAME") == "kernel") else "" }"
-DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc kmod-native bc-native lzop-native bison-native"
+DEPENDS += "virtual/${TARGET_PREFIX}binutils virtual/${TARGET_PREFIX}gcc kmod-native bc-native lzop-native bison-native coreutils-native"
 PACKAGE_WRITE_DEPS += "depmodwrapper-cross"
 
 do_deploy[depends] += "depmodwrapper-cross:do_populate_sysroot"