Re: [OE-core] [PATCH] file: add bzip2-replacement-native to DEPENDS to fix sstate issue

2020-05-24 Thread Richard Purdie
On Sun, 2020-05-24 at 12:04 +0200, Gregor Zatko wrote: > From: Jan-Simon Moeller > > file-native when built on a Debian 10 host will embed a dependency to > 'libbz2.so.1.0' (instead of 'libbz2.so.1'). This can cause issues > when sharing the sstate between hosts e.g.: > >

[OE-core] ✗ patchtest: failure for 'inherit' directive may not be used in conf files as it's supposed to be used for the inheritance of classes. Correct form in conf file is INHERIT.

2020-05-24 Thread Patchwork
== Series Details == Series: 'inherit' directive may not be used in conf files as it's supposed to be used for the inheritance of classes. Correct form in conf file is INHERIT. Revision: 1 URL : https://patchwork.openembedded.org/series/24295/ State : failure == Summary == Thank you for

Re: [OE-core] [PATCH] sanity.bbclass: Detect and fail if 'inherit' is used in conf file

2020-05-24 Thread Gregor Zatko
Ok, thanks for the info.I shall read the guidelines again as I started to contribute in the beginning of the year but was 3 months off then. I'll add that next time. On Sun, 2020-05-24 at 11:27 +0100, Paul Barker wrote: > On Sun, 24 May 2020 at 11:24, Gregor Zatko wrote: > > 'inherit' directive

Re: [OE-core] [PATCH] sanity.bbclass: Detect and fail if 'inherit' is used in conf file

2020-05-24 Thread Paul Barker
On Sat, 23 May 2020 at 21:27, Gregor Zatko wrote: > > 'inherit' directive may not be used in conf files as it's supposed > to be used for the inheritance of classes. > Correct form in conf file is INHERIT. > > This commit adds: > - a sanity check to find whether the wrong case exists > - fail the

[OE-core] [PATCH] file: add bzip2-replacement-native to DEPENDS to fix sstate issue

2020-05-24 Thread Gregor Zatko
From: Jan-Simon Moeller file-native when built on a Debian 10 host will embed a dependency to 'libbz2.so.1.0' (instead of 'libbz2.so.1'). This can cause issues when sharing the sstate between hosts e.g.: recipe-sysroot-native/usr/lib/rpm/rpmdeps: error while loading shared libraries:

[OE-core] [PATCH] 'inherit' directive may not be used in conf files as it's supposed to be used for the inheritance of classes. Correct form in conf file is INHERIT.

2020-05-24 Thread Gregor Zatko
This commit adds: - a sanity check to find whether the wrong case exists - fail the build if so - tell user about the difference in directives https://bugzilla.yoctoproject.org/show_bug.cgi?id=5426 Signed-off-by: Gregor Zatko --- meta/classes/sanity.bbclass | 6 ++ 1 file changed, 6

[OE-core] [PATCH] sanity.bbclass: Detect and fail if 'inherit' is used in conf file

2020-05-24 Thread Gregor Zatko
'inherit' directive may not be used in conf files as it's supposed to be used for the inheritance of classes. Correct form in conf file is INHERIT. This commit adds: - a sanity check to find whether the wrong case exists - fail the build if so - tell user about the difference in directives

Re: [OE-core] [PATCH] sanity.bbclass: Detect and fail if 'inherit' is used in conf file

2020-05-24 Thread Paul Barker
On Sun, 24 May 2020 at 11:24, Gregor Zatko wrote: > > 'inherit' directive may not be used in conf files as it's supposed > to be used for the inheritance of classes. > Correct form in conf file is INHERIT. > > This commit adds: > - a sanity check to find whether the wrong case exists > - fail the

Re: [OE-core] [PATCH] strace: upgrade 5.5 -> 5.6

2020-05-24 Thread Richard Purdie
On Sat, 2020-05-23 at 19:05 +0800, zhengruoqin wrote: > Signed-off-by: Zheng Ruoqin > --- > meta/recipes-devtools/strace/{strace_5.5.bb => strace_5.6.bb} | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > rename meta/recipes-devtools/strace/{strace_5.5.bb => strace_5.6.bb} (92%) > >

Re: [OE-core] [PATCH] docbook-xml: update recipe to stick with the upstream

2020-05-24 Thread Richard Purdie
On Sat, 2020-05-23 at 21:12 +0200, Gregor Zatko wrote: > From: Gregor Zatko > > Until now a Debian package has been used as a source. > This change just switches it to project's upstream. > > https://bugzilla.yoctoproject.org/show_bug.cgi?id=13270 > > Signed-off-by: Gregor Zatko > --- >

[OE-core] [PATCH] initramfs-framework/udev: umount /run/media/XXX

2020-05-24 Thread hongxu
In initramfs-framework, before swith_root rootfs, if udev have time to run (before being killed), it will invoke hook mount.sh (which is provided by udev-extraconf) to mount /run/media/XXX. Release the mounted disk after udev being killed. Signed-off-by: Hongxu Jia ---

Re: [OE-core] [PATCH] u-boot: change ${PN} to u-boot when install/deploy u-boot-init-env

2020-05-24 Thread Denys Dmytriyenko
On Sun, May 24, 2020 at 06:17:51PM +0200, Ming Liu wrote: > From: Ming Liu > > It's really inconvenient to have ${PN} as part of the name of > u-boot-init-env, for instance, in BSP layers, it could be u-boot-fslc, > u-boot-qoriq, u-boot-toradex, and so on. > > Considerring there is only one

[OE-core] [PATCH] u-boot: change ${PN} to u-boot when install/deploy u-boot-init-env

2020-05-24 Thread Ming Liu
From: Ming Liu It's really inconvenient to have ${PN} as part of the name of u-boot-init-env, for instance, in BSP layers, it could be u-boot-fslc, u-boot-qoriq, u-boot-toradex, and so on. Considerring there is only one virtual/bootloader for a build, we could replace ${PN} with u-boot to not

[OE-core] [PATCH] u-boot.inc: fix some inconsistent coding style

2020-05-24 Thread Ming Liu
From: Ming Liu Signed-off-by: Ming Liu --- meta/recipes-bsp/u-boot/u-boot.inc | 44 ++ 1 file changed, 21 insertions(+), 23 deletions(-) diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc index 80f828df52..718435f13f 100644 ---

[OE-core] [PATCH] at-spi2-atk: dont inherit features_check

2020-05-24 Thread Ming Liu
From: Ming Liu In commit be18dabd: [ at-spi2: Make X11 support truly optional ] REQUIRED_DISTRO_FEATURES was dropped, so it does not have to inherit features_check.bbclass. Signed-off-by: Ming Liu --- meta/recipes-support/atk/at-spi2-atk_2.34.2.bb | 2 +- 1 file changed, 1 insertion(+), 1

Re: [OE-core] [PATCH] u-boot: change ${PN} to u-boot when install/deploy u-boot-init-env

2020-05-24 Thread Konrad Weihmann
I know of a project which requires building multiple u-boot-versions within a single run (secure and not-secure) so this hardcoding esp. of the DEPLOY fragments doesn't work there and may lead to completely nondeterministic outputs in the DEPLOY dir. If you could extract the currently hardcoded

Re: [OE-core] [PATCH] u-boot: change ${PN} to u-boot when install/deploy u-boot-init-env

2020-05-24 Thread Ming Liu
Yes, that is a very good suggestion, will do that in V2. //Ming Liu I know of a project which requires building multiple u-boot-versions > within a single run (secure and not-secure) so this hardcoding esp. of > the DEPLOY fragments doesn't work there and may lead to completely >