Re: [oe-core][PATCH v2 2/2] valgrind: Disable ptest swapcontext.vgtest

2021-03-25 Thread Alexander Kanavin
Can you put the ticket link into the patch itself as well please?

Alex


On Thu, 25 Mar 2021 at 23:14, Yi Fan Yu  wrote:

> New test introduced in valgrind 3.17.0.
> Test fails on both qemuarm64 and qemux64.
>
> [YOCTO #14324]
>
> Signed-off-by: Yi Fan Yu 
> ---
>  ...orarily-drd-tests-swapcontext.vgtest.patch | 28 +++
>  .../valgrind/valgrind_3.17.0.bb   |  1 +
>  2 files changed, 29 insertions(+)
>  create mode 100644
> meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch
>
> diff --git
> a/meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch
> b/meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch
> new file mode 100644
> index 00..f217098650
> --- /dev/null
> +++
> b/meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch
> @@ -0,0 +1,28 @@
> +From 0f1814a618eff4233e9e8379a8cb2dededdc5a8b Mon Sep 17 00:00:00 2001
> +From: Yi Fan Yu 
> +Date: Tue, 23 Mar 2021 11:09:20 -0700
> +Subject: [PATCH] Disable temporarily drd/tests/swapcontext.vgtest
> +
> +New test introduced in valgrind 3.17.0.
> +Test fails on both qemuarm64 and qemux64.
> +
> +Upstream-Status: Pending [Needs more Investigation]
> +
> +Signed-off-by: Yi Fan Yu 
> +---
> + drd/tests/swapcontext.vgtest | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/drd/tests/swapcontext.vgtest b/drd/tests/swapcontext.vgtest
> +index 98e3712c4..5492da31d 100644
> +--- a/drd/tests/swapcontext.vgtest
>  b/drd/tests/swapcontext.vgtest
> +@@ -1,4 +1,4 @@
> +-prereq: test -e swapcontext && ./supported_libpthread
> ++prereq: false
> + vgopts: --read-var-info=yes --check-stack-var=yes --show-confl-seg=no
> --num-callers=2
> + prog: swapcontext
> + stderr_filter: filter_stderr
> +--
> +2.17.1
> +
> diff --git a/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb
> b/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb
> index 1e29896596..7a6b766121 100644
> --- a/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb
> +++ b/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb
> @@ -43,6 +43,7 @@ SRC_URI = "
> https://sourceware.org/pub/valgrind/valgrind-${PV}.tar.bz2 \
>
> file://0001-none-tests-fdleak_cmsg.stderr.exp-adjust-tmp-paths.patch \
> file://0001-memcheck-tests-Fix-timerfd-syscall-test.patch \
> file://0001-Add-missing-musl.supp.patch \
> +
>  file://0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch \
> "
>  SRC_URI[md5sum] = "afe11b5572c3121a781433b7c0ab741b"
>  SRC_URI[sha256sum] =
> "ad3aec668e813e40f238995f60796d9590eee64a16dff88421430630e69285a2"
> --
> 2.17.1
>
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149975): 
https://lists.openembedded.org/g/openembedded-core/message/149975
Mute This Topic: https://lists.openembedded.org/mt/81615169/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] populate_sdk_ext: Avoid copying and producing .pyc files

2021-03-25 Thread Alexander Kanavin
Wait, I have to say neither reason is convincing. Pyc files are not
system-specific, they are the same on all systems. And pseudo issues should
be dealt with by fixing the issue, not working around the symptoms, no?

Alex

On Thu, 25 Mar 2021 at 23:44, Mark Hatle 
wrote:

> Since pyc cache files are really system specific, no real reason to copy or
> generate them during the eSDK build process.  Also generating them has the
> possibility of re-using inodes that pseudo may have been tracking, leading
> a build failure.
>
> Signed-off-by: Mark Hatle 
> ---
>  meta/classes/populate_sdk_ext.bbclass | 4 +++-
>  meta/lib/oe/copy_buildsystem.py   | 6 +++---
>  2 files changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/meta/classes/populate_sdk_ext.bbclass
> b/meta/classes/populate_sdk_ext.bbclass
> index 9112ab6c5e..14689ec6ac 100644
> --- a/meta/classes/populate_sdk_ext.bbclass
> +++ b/meta/classes/populate_sdk_ext.bbclass
> @@ -251,7 +251,9 @@ python copy_buildsystem () {
>
>  # Create a layer for new recipes / appends
>  bbpath = d.getVar('BBPATH')
> -bb.process.run(['devtool', '--bbpath', bbpath, '--basepath',
> baseoutpath, 'create-workspace', '--create-only', os.path.join(baseoutpath,
> 'workspace')])
> +env = os.environ.copy()
> +env['PYTHONDONTWRITEBYTECODE'] = '1'
> +bb.process.run(['devtool', '--bbpath', bbpath, '--basepath',
> baseoutpath, 'create-workspace', '--create-only', os.path.join(baseoutpath,
> 'workspace')], env=env)
>
>  # Create bblayers.conf
>  bb.utils.mkdirhier(baseoutpath + '/conf')
> diff --git a/meta/lib/oe/copy_buildsystem.py
> b/meta/lib/oe/copy_buildsystem.py
> index 31a84f5b06..d97bf9d1b9 100644
> --- a/meta/lib/oe/copy_buildsystem.py
> +++ b/meta/lib/oe/copy_buildsystem.py
> @@ -20,7 +20,7 @@ def _smart_copy(src, dest):
>  mode = os.stat(src).st_mode
>  if stat.S_ISDIR(mode):
>  bb.utils.mkdirhier(dest)
> -cmd = "tar --exclude='.git' --xattrs --xattrs-include='*' -chf -
> -C %s -p . \
> +cmd = "tar --exclude='.git' --exclude='__pycache__' --xattrs
> --xattrs-include='*' -chf - -C %s -p . \
>  | tar --xattrs --xattrs-include='*' -xf - -C %s" % (src, dest)
>  subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT)
>  else:
> @@ -259,7 +259,7 @@ def create_locked_sstate_cache(lockedsigs,
> input_sstate_cache, output_sstate_cac
>  bb.note('Generating sstate-cache...')
>
>  nativelsbstring = d.getVar('NATIVELSBSTRING')
> -bb.process.run("gen-lockedsig-cache %s %s %s %s %s" % (lockedsigs,
> input_sstate_cache, output_sstate_cache, nativelsbstring, filterfile or ''))
> +bb.process.run("PYTHONDONTWRITEBYTECODE=1 gen-lockedsig-cache %s %s
> %s %s %s" % (lockedsigs, input_sstate_cache, output_sstate_cache,
> nativelsbstring, filterfile or ''))
>  if fixedlsbstring and nativelsbstring != fixedlsbstring:
>  nativedir = output_sstate_cache + '/' + nativelsbstring
>  if os.path.isdir(nativedir):
> @@ -286,7 +286,7 @@ def check_sstate_task_list(d, targets, filteroutfile,
> cmdprefix='', cwd=None, lo
>  logparam = '-l %s' % logfile
>  else:
>  logparam = ''
> -cmd = "%sBB_SETSCENE_ENFORCE=1 PSEUDO_DISABLED=1 oe-check-sstate %s
> -s -o %s %s" % (cmdprefix, targets, filteroutfile, logparam)
> +cmd = "%sPYTHONDONTWRITEBYTECODE=1 BB_SETSCENE_ENFORCE=1
> PSEUDO_DISABLED=1 oe-check-sstate %s -s -o %s %s" % (cmdprefix, targets,
> filteroutfile, logparam)
>  env = dict(d.getVar('BB_ORIGENV', False))
>  env.pop('BUILDDIR', '')
>  env.pop('BBPATH', '')
> --
> 2.17.1
>
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149974): 
https://lists.openembedded.org/g/openembedded-core/message/149974
Mute This Topic: https://lists.openembedded.org/mt/81615695/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] grub-efi: Fix build failure on RISCV32

2021-03-25 Thread Khem Raj
On Thu, Mar 25, 2021 at 12:48 PM Alistair Francis  wrote:
>
> On Thu, Mar 25, 2021 at 12:15 AM Khem Raj  wrote:
> >
> > Signed-off-by: Khem Raj 
> > Cc: Alistair Francis 
>
> I'm not sure if this is the right thing to do here.
>
> Have you tested this does GRUB boot with this check removed?
>

I have not run it no, but inspected the generate PE file and they look
ok. however, I think we should drop
this patch and instead just disable grub on rv32 until such time when
port is available. I have send a follow up
patch for that.


> From memory I don't think the original GRUB port would have been tested on 
> RV32.
>
> Alistair
>
> > ---
> >  ...sable-offset-overflow-check-for-rv32.patch | 45 +++
> >  meta/recipes-bsp/grub/grub2.inc   |  1 +
> >  2 files changed, 46 insertions(+)
> >  create mode 100644 
> > meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
> >
> > diff --git 
> > a/meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
> >  
> > b/meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
> > new file mode 100644
> > index 00..6547bf2261
> > --- /dev/null
> > +++ 
> > b/meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
> > @@ -0,0 +1,45 @@
> > +From ce1cea0563011698e4a70c9a60e7da66b57ab1d6 Mon Sep 17 00:00:00 2001
> > +From: Khem Raj 
> > +Date: Wed, 24 Mar 2021 20:55:56 -0700
> > +Subject: [PATCH] Disable offset overflow check for rv32
> > +
> > +Fixes
> > +/tmp/grub/grub-mkimage: error: target 17c6 not reachable from pc=936.
> > +
> > +Upstream-Status: Reported [https://savannah.gnu.org/bugs/index.php?60283]
> > +Signed-off-by: Khem Raj 
> > +---
> > + util/grub-mkimagexx.c | 6 --
> > + 1 file changed, 4 insertions(+), 2 deletions(-)
> > +
> > +diff --git a/util/grub-mkimagexx.c b/util/grub-mkimagexx.c
> > +index 00f49ccaa..6deb66226 100644
> > +--- a/util/grub-mkimagexx.c
> >  b/util/grub-mkimagexx.c
> > +@@ -1297,9 +1297,10 @@ SUFFIX (relocate_addrs) (Elf_Ehdr *e, struct 
> > section_metadata *smd,
> > +{
> > +  grub_uint32_t hi20, lo12;
> > +
> > ++#ifdef MKIMAGE_ELF64
> > +  if (off != (grub_int32_t)off)
> > +grub_util_error ("target %lx not reachable from 
> > pc=%lx", (long)sym_addr, (long)((char *)target - (char *)e));
> > +-
> > ++#endif
> > +  hi20 = (off + 0x800) & 0xf000;
> > +  lo12 = (off - hi20) & 0xfff;
> > +  t32[0] = grub_host_to_target32 
> > ((grub_target_to_host32 (t32[0]) & 0xfff) | hi20);
> > +@@ -1336,9 +1337,10 @@ SUFFIX (relocate_addrs) (Elf_Ehdr *e, struct 
> > section_metadata *smd,
> > +{
> > +  grub_int32_t hi20;
> > +
> > ++#ifdef MKIMAGE_ELF64
> > +  if (off != (grub_int32_t)off)
> > +grub_util_error ("target %lx not reachable from 
> > pc=%lx", (long)sym_addr, (long)((char *)target - (char *)e));
> > +-
> > ++#endif
> > +  hi20 = (off + 0x800) & 0xf000;
> > +  *t32 = grub_host_to_target32 ((grub_target_to_host32 
> > (*t32) & 0xfff) | hi20);
> > +}
> > +--
> > +2.31.0
> > +
> > diff --git a/meta/recipes-bsp/grub/grub2.inc 
> > b/meta/recipes-bsp/grub/grub2.inc
> > index bcff676c26..d8a556e06c 100644
> > --- a/meta/recipes-bsp/grub/grub2.inc
> > +++ b/meta/recipes-bsp/grub/grub2.inc
> > @@ -19,6 +19,7 @@ SRC_URI = 
> > "https://alpha.gnu.org/gnu/grub/grub-${REALPV}.tar.xz \
> > 
> > file://grub-module-explicitly-keeps-symbole-.module_license.patch \
> > file://0001-grub.d-10_linux.in-add-oe-s-kernel-name.patch \
> > file://determinism.patch \
> > +   file://0001-Disable-offset-overflow-check-for-rv32.patch \
> >  "
> >
> >  SRC_URI[sha256sum] = 
> > "2c87f1f21e2ab50043e6cd9163c08f1b6c3a6171556bf23ff9ed65b074145484"
> > --
> > 2.31.0
> >
> >
> > 
> >

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149973): 
https://lists.openembedded.org/g/openembedded-core/message/149973
Mute This Topic: https://lists.openembedded.org/mt/81595170/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] grub2: Disable for RISCV32

2021-03-25 Thread Khem Raj
A full working port is not available yet, until such time disable it

Signed-off-by: Khem Raj 
Cc: Alistair Francis 
---
 meta/recipes-bsp/grub/grub2.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-bsp/grub/grub2.inc b/meta/recipes-bsp/grub/grub2.inc
index bcff676c26..d88c3fd3ad 100644
--- a/meta/recipes-bsp/grub/grub2.inc
+++ b/meta/recipes-bsp/grub/grub2.inc
@@ -33,7 +33,7 @@ UPSTREAM_CHECK_REGEX = 
"grub-(?P\d+(\.\d+)+)\.tar\.(gz|xz)"
 
 DEPENDS = "flex-native bison-native gettext-native"
 
-COMPATIBLE_HOST = 
'(x86_64.*|i.86.*|arm.*|aarch64.*|riscv.*)-(linux.*|freebsd.*)'
+COMPATIBLE_HOST = 
'(x86_64.*|i.86.*|arm.*|aarch64.*|riscv64)-(linux.*|freebsd.*)'
 COMPATIBLE_HOST_armv7a = 'null'
 COMPATIBLE_HOST_armv7ve = 'null'
 
-- 
2.31.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149972): 
https://lists.openembedded.org/g/openembedded-core/message/149972
Mute This Topic: https://lists.openembedded.org/mt/81621162/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] wpebackend-fdo: upgrade 1.8.0 -> 1.8.2

2021-03-25 Thread zhengruoqin
Signed-off-by: Zheng Ruoqin 
---
 .../webkit/{wpebackend-fdo_1.8.0.bb => wpebackend-fdo_1.8.2.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-sato/webkit/{wpebackend-fdo_1.8.0.bb => 
wpebackend-fdo_1.8.2.bb} (89%)

diff --git a/meta/recipes-sato/webkit/wpebackend-fdo_1.8.0.bb 
b/meta/recipes-sato/webkit/wpebackend-fdo_1.8.2.bb
similarity index 89%
rename from meta/recipes-sato/webkit/wpebackend-fdo_1.8.0.bb
rename to meta/recipes-sato/webkit/wpebackend-fdo_1.8.2.bb
index 344aa9717c..2ce631a73c 100644
--- a/meta/recipes-sato/webkit/wpebackend-fdo_1.8.0.bb
+++ b/meta/recipes-sato/webkit/wpebackend-fdo_1.8.2.bb
@@ -13,7 +13,7 @@ inherit meson features_check
 REQUIRED_DISTRO_FEATURES = "opengl"
 
 SRC_URI = "https://wpewebkit.org/releases/${BPN}-${PV}.tar.xz;
-SRC_URI[sha256sum] = 
"9652a99c75fe1c6eab0585b6395f4e104b2427e4d1f42969f1f77df29920d253"
+SRC_URI[sha256sum] = 
"98c291071128326a72918d670a46ef63268c43267928560ac3b84b6aebb2d78d"
 
 # This is a tweak of upstream-version-is-even needed because
 # ipstream directory contains tarballs for other components as well.
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149971): 
https://lists.openembedded.org/g/openembedded-core/message/149971
Mute This Topic: https://lists.openembedded.org/mt/81618848/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] systemd-boot: upgrade 247.4 -> 247.5

2021-03-25 Thread zhengruoqin
Signed-off-by: Zheng Ruoqin 
---
 .../systemd/{systemd-boot_247.4.bb => systemd-boot_247.5.bb}| 0
 meta/recipes-core/systemd/systemd.inc   | 2 +-
 .../recipes-core/systemd/{systemd_247.4.bb => systemd_247.5.bb} | 0
 3 files changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-core/systemd/{systemd-boot_247.4.bb => 
systemd-boot_247.5.bb} (100%)
 rename meta/recipes-core/systemd/{systemd_247.4.bb => systemd_247.5.bb} (100%)

diff --git a/meta/recipes-core/systemd/systemd-boot_247.4.bb 
b/meta/recipes-core/systemd/systemd-boot_247.5.bb
similarity index 100%
rename from meta/recipes-core/systemd/systemd-boot_247.4.bb
rename to meta/recipes-core/systemd/systemd-boot_247.5.bb
diff --git a/meta/recipes-core/systemd/systemd.inc 
b/meta/recipes-core/systemd/systemd.inc
index 098bca98f1..87ee794f05 100644
--- a/meta/recipes-core/systemd/systemd.inc
+++ b/meta/recipes-core/systemd/systemd.inc
@@ -14,7 +14,7 @@ LICENSE = "GPLv2 & LGPLv2.1"
 LIC_FILES_CHKSUM = "file://LICENSE.GPL2;md5=751419260aa954499f7abaabaa882bbe \
 
file://LICENSE.LGPL2.1;md5=4fbd65380cdd255951079008b364516c"
 
-SRCREV = "069525e84a67375e27429cb490e8d28af78e673a"
+SRCREV = "c86f4f5f287258938d0c30354040c3b034b0c066"
 SRCBRANCH = "v247-stable"
 SRC_URI = 
"git://github.com/systemd/systemd-stable.git;protocol=git;branch=${SRCBRANCH}"
 
diff --git a/meta/recipes-core/systemd/systemd_247.4.bb 
b/meta/recipes-core/systemd/systemd_247.5.bb
similarity index 100%
rename from meta/recipes-core/systemd/systemd_247.4.bb
rename to meta/recipes-core/systemd/systemd_247.5.bb
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149970): 
https://lists.openembedded.org/g/openembedded-core/message/149970
Mute This Topic: https://lists.openembedded.org/mt/81618837/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] epiphany: upgrade 3.38.2 -> 3.38.3

2021-03-25 Thread zhengruoqin
Signed-off-by: Zheng Ruoqin 
---
 .../epiphany/{epiphany_3.38.2.bb => epiphany_3.38.3.bb} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-gnome/epiphany/{epiphany_3.38.2.bb => epiphany_3.38.3.bb} 
(93%)

diff --git a/meta/recipes-gnome/epiphany/epiphany_3.38.2.bb 
b/meta/recipes-gnome/epiphany/epiphany_3.38.3.bb
similarity index 93%
rename from meta/recipes-gnome/epiphany/epiphany_3.38.2.bb
rename to meta/recipes-gnome/epiphany/epiphany_3.38.3.bb
index 04f340f133..889de16d00 100644
--- a/meta/recipes-gnome/epiphany/epiphany_3.38.2.bb
+++ b/meta/recipes-gnome/epiphany/epiphany_3.38.3.bb
@@ -19,7 +19,7 @@ SRC_URI = 
"${GNOME_MIRROR}/${GNOMEBN}/${@gnome_verdir("${PV}")}/${GNOMEBN}-${PV}
file://migrator.patch \
file://distributor.patch \
"
-SRC_URI[archive.sha256sum] = 
"8b05f2bcc1e80ecf4a10f6f01b3285087eb4cbdf5741dffb8c0355715ef5116d"
+SRC_URI[archive.sha256sum] = 
"ce62c1fa3e442a9d6131fc884a6bc9a65cf06224c30d649c49624c2ebcb0d275"
 
 # Developer mode enables debugging
 PACKAGECONFIG[developer-mode] = "-Ddeveloper_mode=true,-Ddeveloper_mode=false"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149969): 
https://lists.openembedded.org/g/openembedded-core/message/149969
Mute This Topic: https://lists.openembedded.org/mt/81618834/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] OpenEmbedded Happy Hour March 31 5pm/1700 UTC

2021-03-25 Thread Denys Dmytriyenko
Hi,

Just a reminder about our upcoming OpenEmbedded Happy Hour on March 31 for
Europe/US timezones @ 1700/5pm UTC (1pm ET / 10am PT):

https://www.openembedded.org/wiki/Calendar
https://www.timeanddate.com/worldclock/fixedtime.html?msg=OpenEmbedded+Happy+Hour+March+31=20210331T17

-- 
Regards,
Denys Dmytriyenko 
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149968): 
https://lists.openembedded.org/g/openembedded-core/message/149968
Mute This Topic: https://lists.openembedded.org/mt/81618809/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] make-mod-scripts: Provide the correct objcopy to kernel make

2021-03-25 Thread Nishanth Menon via lists.openembedded.org
When cross-compiling with v5.12-rc3, prepare fails[1] build of vdso at
the objcopy stage since it seems to be using the local host's objcopy
rather than the cross-compile version we want it to use.

This can be trivially reproduced in a localbuild of the kernel
following the build parameters provided in the process[2]

Lets fix this by passing OBJCOPY over to the kernel.

[1] https://pastebin.ubuntu.com/p/pNcQtb93wr/
[2] https://pastebin.ubuntu.com/p/vZGqgh9Sq5/
Signed-off-by: Nishanth Menon 
---

NOTE:
1. This is a different problem and is not resolved with
https://lists.openembedded.org/g/openembedded-core/message/149943

2. Though reproduced during a dunfell build, this should probably
apply else where as well.

3. I have'nt been able to cleanly bisect things, but my best guess was
that this might be a consequence of kernel fixup exposing this.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a5b8ca97fbf8300a5e21c393df25ce6f521e7939

 meta/classes/kernel-arch.bbclass | 3 +++
 meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta/classes/kernel-arch.bbclass b/meta/classes/kernel-arch.bbclass
index 07ec242e63bb..3d25fc7ac531 100644
--- a/meta/classes/kernel-arch.bbclass
+++ b/meta/classes/kernel-arch.bbclass
@@ -60,9 +60,12 @@ TARGET_LD_KERNEL_ARCH ?= ""
 HOST_LD_KERNEL_ARCH ?= "${TARGET_LD_KERNEL_ARCH}"
 TARGET_AR_KERNEL_ARCH ?= ""
 HOST_AR_KERNEL_ARCH ?= "${TARGET_AR_KERNEL_ARCH}"
+TARGET_OBJCOPY_KERNEL_ARCH ?= ""
+HOST_OBJCOPY_KERNEL_ARCH ?= "${TARGET_OBJCOPY_KERNEL_ARCH}"
 
 KERNEL_CC = "${CCACHE}${HOST_PREFIX}gcc ${HOST_CC_KERNEL_ARCH} -fuse-ld=bfd 
${DEBUG_PREFIX_MAP} -fdebug-prefix-map=${STAGING_KERNEL_DIR}=${KERNEL_SRC_PATH}"
 KERNEL_LD = "${CCACHE}${HOST_PREFIX}ld.bfd ${HOST_LD_KERNEL_ARCH}"
 KERNEL_AR = "${CCACHE}${HOST_PREFIX}ar ${HOST_AR_KERNEL_ARCH}"
+KERNEL_OBJCOPY = "${CCACHE}${HOST_PREFIX}objcopy ${HOST_OBJCOPY_KERNEL_ARCH}"
 TOOLCHAIN = "gcc"
 
diff --git a/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb 
b/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
index 87b7d240f51a..2d73e8093c2e 100644
--- a/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
+++ b/meta/recipes-kernel/make-mod-scripts/make-mod-scripts_1.0.bb
@@ -25,7 +25,7 @@ EXTRA_OEMAKE = " HOSTCC="${BUILD_CC} ${BUILD_CFLAGS} 
${BUILD_LDFLAGS}" HOSTCPP="
 do_configure() {
unset CFLAGS CPPFLAGS CXXFLAGS LDFLAGS
for t in prepare scripts_basic scripts; do
-   oe_runmake CC="${KERNEL_CC}" LD="${KERNEL_LD}" 
AR="${KERNEL_AR}" \
+   oe_runmake CC="${KERNEL_CC}" LD="${KERNEL_LD}" 
AR="${KERNEL_AR}" OBJCOPY="${KERNEL_OBJCOPY}"\
-C ${STAGING_KERNEL_DIR} O=${STAGING_KERNEL_BUILDDIR} $t
done
 }
-- 
2.31.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149967): 
https://lists.openembedded.org/g/openembedded-core/message/149967
Mute This Topic: https://lists.openembedded.org/mt/81618199/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] documentation-audit.sh: Fix typo in specifying LICENSE_FLAGS_WHITELIST

2021-03-25 Thread Khem Raj
Signed-off-by: Khem Raj 
---
 scripts/contrib/documentation-audit.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/contrib/documentation-audit.sh 
b/scripts/contrib/documentation-audit.sh
index 1191f57a8e..f436f9bae0 100755
--- a/scripts/contrib/documentation-audit.sh
+++ b/scripts/contrib/documentation-audit.sh
@@ -27,7 +27,7 @@ fi
 
 echo "REMINDER: you need to build for MACHINE=qemux86 or you won't get useful 
results"
 echo "REMINDER: you need to set LICENSE_FLAGS_WHITELIST appropriately in 
local.conf or "
-echo " you'll get false positives.  For example, LICENSE_FLAGS_WHITELIST = 
\"Commercial\""
+echo " you'll get false positives.  For example, LICENSE_FLAGS_WHITELIST = 
\"commercial\""
 
 for pkg in `bitbake -s | awk '{ print \$1 }'`; do
if [[ "$pkg" == "Loading" || "$pkg" == "Loaded" ||
-- 
2.31.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149966): 
https://lists.openembedded.org/g/openembedded-core/message/149966
Mute This Topic: https://lists.openembedded.org/mt/81618162/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] gst-examples,gstreamer1.0-libav: Add option to exclude from world build

2021-03-25 Thread Khem Raj
When LICENSE_FLAGS_WHITELIST does not have commercial in it then the
the dependencies like ffmpeg will become invisible and render these
packages unbuildable, therefore world build will report parse issues

its better to exclude them from world build in such cases

Signed-off-by: Khem Raj 
Cc: Yann Dirson 
---
 meta/recipes-multimedia/gstreamer/gst-examples_1.18.4.bb   | 2 ++
 meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.18.4.bb | 3 +++
 2 files changed, 5 insertions(+)

diff --git a/meta/recipes-multimedia/gstreamer/gst-examples_1.18.4.bb 
b/meta/recipes-multimedia/gstreamer/gst-examples_1.18.4.bb
index 4670ab34db..196258e00f 100644
--- a/meta/recipes-multimedia/gstreamer/gst-examples_1.18.4.bb
+++ b/meta/recipes-multimedia/gstreamer/gst-examples_1.18.4.bb
@@ -33,3 +33,5 @@ RRECOMMENDS_${PN} = "gstreamer1.0-plugins-base-meta \
   ${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", 
"commercial", "gstreamer1.0-libav", "", d)} \
  ${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", 
"commercial", "gstreamer1.0-plugins-ugly-meta", "", d)}"
 RPROVIDES_${PN} += "gst-player gst-player-bin"
+
+EXCLUDE_FROM_WORLD = "${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", 
"commercial", "0", "1", d)}"
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.18.4.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.18.4.bb
index 6a84f92f31..7c39e7da37 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.18.4.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.18.4.bb
@@ -6,6 +6,7 @@ SECTION = "multimedia"
 
 # ffmpeg has comercial license flags so add it as we need ffmpeg as a 
dependency
 LICENSE_FLAGS = "commercial"
+
 LICENSE = "LGPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=6762ed442b3822387a51c92d928ead0d \
 
file://ext/libav/gstav.h;beginline=1;endline=18;md5=a752c35267d8276fd9ca3db6994fca9c
 \
@@ -22,3 +23,5 @@ inherit meson pkgconfig upstream-version-is-even
 
 FILES_${PN} += "${libdir}/gstreamer-1.0/*.so"
 FILES_${PN}-staticdev += "${libdir}/gstreamer-1.0/*.a"
+
+EXCLUDE_FROM_WORLD = "${@bb.utils.contains("LICENSE_FLAGS_WHITELIST", 
"commercial", "0", "1", d)}"
-- 
2.31.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149965): 
https://lists.openembedded.org/g/openembedded-core/message/149965
Mute This Topic: https://lists.openembedded.org/mt/81618084/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 0/1] Enable PR Service support in eSDK

2021-03-25 Thread Mark Hatle
This is the first attempt at enabling the PR service in the eSDK.  It appears
to work for me, but I expect I'm missing corner cases.

Any feedback is appreciatd.

Mark Hatle (1):
  populate_sdk_ext: Add support for PR service

 meta/classes/populate_sdk_ext.bbclass | 25 +
 meta/files/ext-sdk-prepare.py | 17 +
 2 files changed, 42 insertions(+)

-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149964): 
https://lists.openembedded.org/g/openembedded-core/message/149964
Mute This Topic: https://lists.openembedded.org/mt/81617272/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 1/1] populate_sdk_ext: Add support for PR service

2021-03-25 Thread Mark Hatle
From: Mark Hatle 

In the classes/populate_sdk_ext.bbclass the system already copies a number
of configurations, such as the hash equivalency data.  However, the PR
service was being handled.

The new code works by checking if PRSERV_HOST is defined, if it is, use
the existing export functions to write out a conf/prserv.inc file into
the eSDK.  On eSDK install, if a conf/prserv.inc file is present we then
import this file into the system.

This mechanism will work if the PRSERV_HOST is local or remote, as it pulls
the necessary data from the server and then imports it to a local database
on eSDK installation.

Note: the conf/prserv.inc file is not deleted at this time.  It was left
for possible debugging purposes, but removing it is something we could decide
to do in the future.

Signed-off-by: Mark Hatle 
Signed-off-by: Mark Hatle 
---
 meta/classes/populate_sdk_ext.bbclass | 25 +
 meta/files/ext-sdk-prepare.py | 17 +
 2 files changed, 42 insertions(+)

diff --git a/meta/classes/populate_sdk_ext.bbclass 
b/meta/classes/populate_sdk_ext.bbclass
index 14689ec6ac..84232ed9f5 100644
--- a/meta/classes/populate_sdk_ext.bbclass
+++ b/meta/classes/populate_sdk_ext.bbclass
@@ -375,6 +375,10 @@ python copy_buildsystem () {
 # Map gcc-dependent uninative sstate cache for installer usage
 f.write('SSTATE_MIRRORS += " file://universal/(.*) 
file://universal-4.9/\\1 file://universal-4.9/(.*) 
file://universal-4.8/\\1"\n\n')
 
+if d.getVar("PRSERV_HOST"):
+# Override this, we now include PR data, so it should only 
point ot the local database
+f.write('PRSERV_HOST = "localhost:0"\n\n')
+
 # Allow additional config through sdk-extra.conf
 fn = bb.cookerdata.findConfigFile('sdk-extra.conf', d)
 if fn:
@@ -398,6 +402,27 @@ python copy_buildsystem () {
 bb.utils.mkdirhier(os.path.join(baseoutpath, 'cache'))
 shutil.copyfile(builddir + '/cache/bb_unihashes.dat', baseoutpath + 
'/cache/bb_unihashes.dat')
 
+# If PR Service is in use, we need to export this as well
+bb.note('Do we have a pr database?')
+if d.getVar("PRSERV_HOST"):
+bb.note('Writing PR database...')
+# Based on the code in classes/prexport.bbclass
+import oe.prservice
+#dump meta info of tables
+localdata = d.createCopy()
+localdata.setVar('PRSERV_DUMPOPT_COL', "1")
+localdata.setVar('PRSERV_DUMPDIR', os.path.join(baseoutpath, 'conf'))
+localdata.setVar('PRSERV_DUMPFILE', '${PRSERV_DUMPDIR}/prserv.inc')
+
+bb.note('PR Database write to %s' % 
(localdata.getVar('PRSERV_DUMPFILE')))
+
+retval = oe.prservice.prserv_dump_db(localdata)
+if not retval:
+bb.error("prexport_handler: export failed!")
+return
+(metainfo, datainfo) = retval
+oe.prservice.prserv_export_tofile(localdata, metainfo, datainfo, True)
+
 # Use templateconf.cfg file from builddir if exists
 if os.path.exists(builddir + '/conf/templateconf.cfg') and 
use_custom_templateconf == '1':
 shutil.copyfile(builddir + '/conf/templateconf.cfg', baseoutpath + 
'/conf/templateconf.cfg')
diff --git a/meta/files/ext-sdk-prepare.py b/meta/files/ext-sdk-prepare.py
index 163d5e9912..d191e5e19c 100644
--- a/meta/files/ext-sdk-prepare.py
+++ b/meta/files/ext-sdk-prepare.py
@@ -44,6 +44,23 @@ def main():
 sdk_targets = []
 else:
 sdk_targets = ' '.join(sys.argv[2:]).split()
+
+prserv = os.path.join(os.path.dirname(os.path.realpath(__file__)), 'conf', 
'prserv.inc')
+if os.path.isfile(prserv):
+with open(logfile, 'a') as logf:
+logf.write('Importing PR data...\n')
+
+ret = run_command_interruptible('bitbake-prserv-tool import %s' % 
prserv)
+
+lastlog = get_last_consolelog()
+if lastlog:
+with open(lastlog, 'r') as f:
+for line in f:
+logf.write(line)
+if ret:
+print('ERROR: PR data import failed: error log written to %s' 
% logfile)
+return ret
+
 if not sdk_targets:
 # Just do a parse so the cache is primed
 ret = run_command_interruptible('bitbake -p --quiet')
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149963): 
https://lists.openembedded.org/g/openembedded-core/message/149963
Mute This Topic: https://lists.openembedded.org/mt/81617271/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] Autobuilder data collection for intermittent bugs

2021-03-25 Thread Randy MacLeod

On 2021-03-25 6:23 p.m., Richard Purdie wrote:

Hi Sakib,

On Thu, 2021-03-25 at 18:11 -0400, Sakib Sajal wrote:

I have been trying, with help from Randy, Trevor and Steve, to run some
experiments on an internal autobuilder instance to collect host data
during builds for signs of io latency.

Initially i modified yocto-autobuilder-helper/config.json and added the
following 3 lines to existing templates (and created a commit) that were
being used so that the builds would start collecting the data.


+    "BB_HEARTBEAT_EVENT = '5'",
+    "BB_LOG_HOST_STAT_ON_INTERVAL = '1'",
+    "BB_LOG_HOST_STAT_CMDS = 'pwd; ls'"

I also created a template in yocto-autobuilder-helper/config.json

+    "data-collect" : {
+    "BUILDINFO" : true,
+    "BBTARGETS" : "core-image-minimal",
+    "extravars" : [
+    "BB_HEARTBEAT_EVENT = '5'",
+    "BB_LOG_HOST_STAT_ON_INTERVAL = '1'",
+    "BB_LOG_HOST_STAT_CMDS = 'pwd; ls'"
+    ]
+    },

and modified a builder to use the template:

      "beaglebone-alt" : {
-    "MACHINE" : "beaglebone-yocto",
-    "TEMPLATE" : "altcfg-hw"
+    "MACHINE" : "qemux86-64",
+    "TEMPLATE" : "data-collect"
   },

Restarted both the controller and the worker, and started the
beaglebone-alt build. Once the build started, i checked the conf file in
the build directory but those lines were not appended to the local.conf
file and hence no log files.


You shouldn't need to restart the controller, the worker should pull and
use the configuration from the helper branch specified.

I'd note that it writes the config to auto.conf, not local.conf.

It should also give output in the logs about what its doing. For example:

https://autobuilder.yoctoproject.org/typhoon/#/builders/106/builds/2144/steps/10/logs/stdio

is the "Write config" step of beaglebone-alt on the autobuilder where it
clearly says:

Writing /home/pokybuild/yocto-worker/beaglebone-alt/build/build/conf/auto.conf 
with contents:

What do your logs say?


In summary, I have been trying to get the 3 lines from the "extravars"
above to appear in one of the local.conf files. However none of the
builds seems to have been altered.

I am wondering if the changes that I've made are even being picked up by
the controller. Am I missing something?

Is there an easier way to update the conf files?

Initially I'd like collect data on a single node and scale it to all the
nodes. Any ideas/suggestions are welcome!


What you're doing should work. Its hard to comment more without seeing to logs
but check you're looking at auto.conf.


Thanks for the update Sakib.

We're planning to follow Steve's example of getting commit access to
contrib branches of y-a-h and poky and using the YP AB to start custom 
workers/job initially.  We'll also keep working to duplicate the

AB behaviour locally. Trevor has asked Konrad for an additional system
or two since when jobs are running, the web UI is
painfully slow since it's runnig on the same node right now.

The one part that Sakib didn't ask about is how to get all the workers
to do the data collection but that's not urgent since we'll be doing
some smaller scale experiments tomorrow. If you can point out a helpful
example, that'd be good.

Hopefully we'll have a good collection of data showing what processes
are running when the IO latency is > 5 seconds sometime next week.

../Randy



Cheers,

Richard





--
# Randy MacLeod
# Wind River Linux

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149962): 
https://lists.openembedded.org/g/openembedded-core/message/149962
Mute This Topic: https://lists.openembedded.org/mt/81615102/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] master eSDK hash problem with METADATA_REVISION and base-files:do_install

2021-03-25 Thread Mark Hatle
Building an eSDK (all stock config):

MACHINE=qemuarm bitbake core-image-minimal -c populate_sdk_ext


I then install it, and get:

Checking sstate mirror object availability: 100%
|#|
Time: 0:00:00
WARNING: The base-files:do_install sig is computed to be
587ccfa9299e26094ae342f1ce3d782bfaf776c639013428a511770c3828d49a, but the sig is
locked to 0c35f266f1e79b1f34260df1b8d677f7eb74a7f7241155601ed73a014c8e286c in
SIGGEN_LOCKEDSIGS_t-qemuarm
Loading cache: 100%
|##|
Time: 0:00:00
Initialising tasks: 100%
|#|
Time: 0:00:00


Running a bitbake diffsigs on this file, I get:

basehash changed from
cffec52acb67f62fe3812d0d42d52b1081f9150386f43f0426f5469b4a1a6a25 to
332259212c5e9a4ed2a7fe25b8ea77cb54fb6b82f7deed90179e145f1d0cf1b6
Variable METADATA_REVISION value changed from
'3210cabe56d6cf6d51638e6f31f93110e40ab2cd' to ''


So it appears that some of the recent work on METADATA_REVISION has broken hash
computations in the base-files package due to the value being 'unknown' in one,
and set in the other.

--Mark

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149961): 
https://lists.openembedded.org/g/openembedded-core/message/149961
Mute This Topic: https://lists.openembedded.org/mt/81616842/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] populate_sdk_ext: Avoid copying and producing .pyc files

2021-03-25 Thread Mark Hatle
Since pyc cache files are really system specific, no real reason to copy or
generate them during the eSDK build process.  Also generating them has the
possibility of re-using inodes that pseudo may have been tracking, leading
a build failure.

Signed-off-by: Mark Hatle 
---
 meta/classes/populate_sdk_ext.bbclass | 4 +++-
 meta/lib/oe/copy_buildsystem.py   | 6 +++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/meta/classes/populate_sdk_ext.bbclass 
b/meta/classes/populate_sdk_ext.bbclass
index 9112ab6c5e..14689ec6ac 100644
--- a/meta/classes/populate_sdk_ext.bbclass
+++ b/meta/classes/populate_sdk_ext.bbclass
@@ -251,7 +251,9 @@ python copy_buildsystem () {
 
 # Create a layer for new recipes / appends
 bbpath = d.getVar('BBPATH')
-bb.process.run(['devtool', '--bbpath', bbpath, '--basepath', baseoutpath, 
'create-workspace', '--create-only', os.path.join(baseoutpath, 'workspace')])
+env = os.environ.copy()
+env['PYTHONDONTWRITEBYTECODE'] = '1'
+bb.process.run(['devtool', '--bbpath', bbpath, '--basepath', baseoutpath, 
'create-workspace', '--create-only', os.path.join(baseoutpath, 'workspace')], 
env=env)
 
 # Create bblayers.conf
 bb.utils.mkdirhier(baseoutpath + '/conf')
diff --git a/meta/lib/oe/copy_buildsystem.py b/meta/lib/oe/copy_buildsystem.py
index 31a84f5b06..d97bf9d1b9 100644
--- a/meta/lib/oe/copy_buildsystem.py
+++ b/meta/lib/oe/copy_buildsystem.py
@@ -20,7 +20,7 @@ def _smart_copy(src, dest):
 mode = os.stat(src).st_mode
 if stat.S_ISDIR(mode):
 bb.utils.mkdirhier(dest)
-cmd = "tar --exclude='.git' --xattrs --xattrs-include='*' -chf - -C %s 
-p . \
+cmd = "tar --exclude='.git' --exclude='__pycache__' --xattrs 
--xattrs-include='*' -chf - -C %s -p . \
 | tar --xattrs --xattrs-include='*' -xf - -C %s" % (src, dest)
 subprocess.check_output(cmd, shell=True, stderr=subprocess.STDOUT)
 else:
@@ -259,7 +259,7 @@ def create_locked_sstate_cache(lockedsigs, 
input_sstate_cache, output_sstate_cac
 bb.note('Generating sstate-cache...')
 
 nativelsbstring = d.getVar('NATIVELSBSTRING')
-bb.process.run("gen-lockedsig-cache %s %s %s %s %s" % (lockedsigs, 
input_sstate_cache, output_sstate_cache, nativelsbstring, filterfile or ''))
+bb.process.run("PYTHONDONTWRITEBYTECODE=1 gen-lockedsig-cache %s %s %s %s 
%s" % (lockedsigs, input_sstate_cache, output_sstate_cache, nativelsbstring, 
filterfile or ''))
 if fixedlsbstring and nativelsbstring != fixedlsbstring:
 nativedir = output_sstate_cache + '/' + nativelsbstring
 if os.path.isdir(nativedir):
@@ -286,7 +286,7 @@ def check_sstate_task_list(d, targets, filteroutfile, 
cmdprefix='', cwd=None, lo
 logparam = '-l %s' % logfile
 else:
 logparam = ''
-cmd = "%sBB_SETSCENE_ENFORCE=1 PSEUDO_DISABLED=1 oe-check-sstate %s -s -o 
%s %s" % (cmdprefix, targets, filteroutfile, logparam)
+cmd = "%sPYTHONDONTWRITEBYTECODE=1 BB_SETSCENE_ENFORCE=1 PSEUDO_DISABLED=1 
oe-check-sstate %s -s -o %s %s" % (cmdprefix, targets, filteroutfile, logparam)
 env = dict(d.getVar('BB_ORIGENV', False))
 env.pop('BUILDDIR', '')
 env.pop('BBPATH', '')
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149960): 
https://lists.openembedded.org/g/openembedded-core/message/149960
Mute This Topic: https://lists.openembedded.org/mt/81615695/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] Autobuilder data collection for intermittent bugs

2021-03-25 Thread Richard Purdie
Hi Sakib,

On Thu, 2021-03-25 at 18:11 -0400, Sakib Sajal wrote:
> I have been trying, with help from Randy, Trevor and Steve, to run some 
> experiments on an internal autobuilder instance to collect host data 
> during builds for signs of io latency.
> 
> Initially i modified yocto-autobuilder-helper/config.json and added the 
> following 3 lines to existing templates (and created a commit) that were 
> being used so that the builds would start collecting the data.
> 
> 
> +    "BB_HEARTBEAT_EVENT = '5'",
> +    "BB_LOG_HOST_STAT_ON_INTERVAL = '1'",
> +    "BB_LOG_HOST_STAT_CMDS = 'pwd; ls'"
> 
> I also created a template in yocto-autobuilder-helper/config.json
> 
> +    "data-collect" : {
> +    "BUILDINFO" : true,
> +    "BBTARGETS" : "core-image-minimal",
> +    "extravars" : [
> +    "BB_HEARTBEAT_EVENT = '5'",
> +    "BB_LOG_HOST_STAT_ON_INTERVAL = '1'",
> +    "BB_LOG_HOST_STAT_CMDS = 'pwd; ls'"
> +    ]
> +    },
> 
> and modified a builder to use the template:
> 
>      "beaglebone-alt" : {
> -    "MACHINE" : "beaglebone-yocto",
> -    "TEMPLATE" : "altcfg-hw"
> +    "MACHINE" : "qemux86-64",
> +    "TEMPLATE" : "data-collect"
>   },
> 
> Restarted both the controller and the worker, and started the 
> beaglebone-alt build. Once the build started, i checked the conf file in 
> the build directory but those lines were not appended to the local.conf 
> file and hence no log files.

You shouldn't need to restart the controller, the worker should pull and 
use the configuration from the helper branch specified.

I'd note that it writes the config to auto.conf, not local.conf.

It should also give output in the logs about what its doing. For example:

https://autobuilder.yoctoproject.org/typhoon/#/builders/106/builds/2144/steps/10/logs/stdio

is the "Write config" step of beaglebone-alt on the autobuilder where it
clearly says:

Writing /home/pokybuild/yocto-worker/beaglebone-alt/build/build/conf/auto.conf 
with contents:

What do your logs say?

> In summary, I have been trying to get the 3 lines from the "extravars" 
> above to appear in one of the local.conf files. However none of the 
> builds seems to have been altered.
> 
> I am wondering if the changes that I've made are even being picked up by 
> the controller. Am I missing something?
> 
> Is there an easier way to update the conf files?
> 
> Initially I'd like collect data on a single node and scale it to all the 
> nodes. Any ideas/suggestions are welcome!

What you're doing should work. Its hard to comment more without seeing to logs
but check you're looking at auto.conf.

Cheers,

Richard



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149959): 
https://lists.openembedded.org/g/openembedded-core/message/149959
Mute This Topic: https://lists.openembedded.org/mt/81615102/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe-core][PATCH v2 1/2] valgrind: update 3.16.1 -> 3.17.0

2021-03-25 Thread Yi Fan Yu
Notable changes:
* library is now in libexecdir instead of libdir

Added patches:
* Add musl.supp: missing musl.supp in 3.17.0

Dropped backport patches:
* nlcontrolc: found in c79180a3afcf65902e578646c3b716cc749db406
* drd Fedora33: found in 15330adf7c2471fbaa6a0818db07078d81dbff97
* lmw lswi ppc64le: found in 74b74174d572fee4015b8f4e326db3cd949bcdc3

Other dropped patches
* helgrind intercept: found in d2d54dbcc74244adfc0c80b40862edf2b82f53b9
* drd musl fix: found in d2d54dbcc74244adfc0c80b40862edf2b82f53b9

TESTING RESULTS:
qemux86-64:
FAIL: drd/tests/swapcontext

  3.17.0  3.16.1
===
TOTAL:  736726
PASSED: 694688
FAILED:   1  0
SKIPPED: 41 38

Signed-off-by: Yi Fan Yu 
---
Changes for V2:

minor edit to commit msg

fix reproducibility

+# point the expanded @abs_top_builddir@ of the host to PTEST_PATH
+sed -i s:${S}:${PTEST_PATH}:g \
+${D}${PTEST_PATH}/memcheck/tests/linux/debuginfod-check.vgtest

---

 .../valgrind/0001-Add-missing-musl.supp.patch |  72 +++
 ...gtest-hanging-on-newer-glibc-and-or-.patch | 200 --
 .../valgrind/0001-drd-Port-to-Fedora-33.patch |  48 -
 .../valgrind/valgrind/0001-drd-musl-fix.patch |  31 ---
 ...01-helgrind-Intercept-libc-functions.patch |  54 -
 ...ated-PowerPC-insns-aren-t-allowed-on.patch |  62 --
 ...eak_cmsg.stderr.exp-adjust-tmp-paths.patch |   2 +-
 ...ntext-APIs-are-not-available-on-musl.patch |  50 -
 .../valgrind/valgrind/fixed-perl-path.patch   |  20 +-
 .../valgrind/valgrind/run-ptest   |   3 +-
 .../valgrind/valgrind/s390x_vec_op_t.patch|  24 ++-
 ...{valgrind_3.16.1.bb => valgrind_3.17.0.bb} |  25 ++-
 12 files changed, 165 insertions(+), 426 deletions(-)
 create mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-drd-Port-to-Fedora-33.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-drd-musl-fix.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-helgrind-Intercept-libc-functions.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-lmw-lswi-and-related-PowerPC-insns-aren-t-allowed-on.patch
 rename meta/recipes-devtools/valgrind/{valgrind_3.16.1.bb => 
valgrind_3.17.0.bb} (92%)

diff --git 
a/meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch 
b/meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch
new file mode 100644
index 00..2a73f7e81f
--- /dev/null
+++ b/meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch
@@ -0,0 +1,72 @@
+From 61bc8664f93cd980831c9da4a3e8a385b089a0ab Mon Sep 17 00:00:00 2001
+From: Yi Fan Yu 
+Date: Tue, 23 Mar 2021 09:32:22 -0700
+Subject: [PATCH] Add missing musl.supp
+
+3.17 did not ship musl.supp in the tarball.
+
+This is a workaround until next release.
+
+Upstream-Status: Backport [dde556d51f8226a6de564a00bf82536bb7042c54]
+
+Signed-off-by: Yi Fan Yu 
+---
+ musl.supp | 46 ++
+ 1 file changed, 46 insertions(+)
+ create mode 100644 musl.supp
+
+diff --git a/musl.supp b/musl.supp
+new file mode 100644
+index 0..864172a24
+--- /dev/null
 b/musl.supp
+@@ -0,0 +1,46 @@
++# Suppressions for musl libc
++# See: https://www.openwall.com/lists/musl/2017/06/15/4
++
++{
++   musl-dynlink-false-positive1
++   Memcheck:Leak
++   fun:calloc
++   fun:load_direct_deps
++   fun:load_deps
++   fun:load_deps
++   fun:__dls3
++   fun:__dls2b
++   fun:__dls2
++}
++
++{
++   musl-dynlink-false-positive2
++   Memcheck:Leak
++   fun:calloc
++   fun:load_direct_deps
++   fun:load_deps
++   fun:load_deps
++   fun:__dls3
++   fun:__dls2
++}
++
++{
++   musl-dynlink-false-positive3
++   Memcheck:Leak
++   fun:calloc
++   fun:load_library
++   fun:load_preload
++   fun:__dls3
++   fun:__dls2b
++   fun:__dls2
++}
++
++{
++   musl-dynlink-false-positive4
++   Memcheck:Leak
++   fun:calloc
++   fun:load_library
++   fun:load_preload
++   fun:__dls3
++   fun:__dls2
++}
+-- 
+2.17.1
+
diff --git 
a/meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
 
b/meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
deleted file mode 100644
index 98cbcd132c..00
--- 
a/meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
+++ /dev/null
@@ -1,200 +0,0 @@
-From 83c24e31df6932a6d4fced179050c6d8d8c6f3b5 Mon Sep 17 00:00:00 2001
-From: Philippe Waroquiers 
-Date: Sun, 7 Mar 2021 22:29:27 +0100
-Subject: [PATCH] Fix nlcontrolc.vgtest hanging on newer glibc and/or arm64
-
-This test verifies that GDB can interrupt a process with all threads
-blocked in a long select syscall.
-The test used to terminate by having 

[oe-core][PATCH v2 2/2] valgrind: Disable ptest swapcontext.vgtest

2021-03-25 Thread Yi Fan Yu
New test introduced in valgrind 3.17.0.
Test fails on both qemuarm64 and qemux64.

[YOCTO #14324]

Signed-off-by: Yi Fan Yu 
---
 ...orarily-drd-tests-swapcontext.vgtest.patch | 28 +++
 .../valgrind/valgrind_3.17.0.bb   |  1 +
 2 files changed, 29 insertions(+)
 create mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch

diff --git 
a/meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch
 
b/meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch
new file mode 100644
index 00..f217098650
--- /dev/null
+++ 
b/meta/recipes-devtools/valgrind/valgrind/0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch
@@ -0,0 +1,28 @@
+From 0f1814a618eff4233e9e8379a8cb2dededdc5a8b Mon Sep 17 00:00:00 2001
+From: Yi Fan Yu 
+Date: Tue, 23 Mar 2021 11:09:20 -0700
+Subject: [PATCH] Disable temporarily drd/tests/swapcontext.vgtest
+
+New test introduced in valgrind 3.17.0.
+Test fails on both qemuarm64 and qemux64.
+
+Upstream-Status: Pending [Needs more Investigation]
+
+Signed-off-by: Yi Fan Yu 
+---
+ drd/tests/swapcontext.vgtest | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/drd/tests/swapcontext.vgtest b/drd/tests/swapcontext.vgtest
+index 98e3712c4..5492da31d 100644
+--- a/drd/tests/swapcontext.vgtest
 b/drd/tests/swapcontext.vgtest
+@@ -1,4 +1,4 @@
+-prereq: test -e swapcontext && ./supported_libpthread
++prereq: false
+ vgopts: --read-var-info=yes --check-stack-var=yes --show-confl-seg=no 
--num-callers=2
+ prog: swapcontext
+ stderr_filter: filter_stderr
+-- 
+2.17.1
+
diff --git a/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb 
b/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb
index 1e29896596..7a6b766121 100644
--- a/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb
+++ b/meta/recipes-devtools/valgrind/valgrind_3.17.0.bb
@@ -43,6 +43,7 @@ SRC_URI = 
"https://sourceware.org/pub/valgrind/valgrind-${PV}.tar.bz2 \

file://0001-none-tests-fdleak_cmsg.stderr.exp-adjust-tmp-paths.patch \
file://0001-memcheck-tests-Fix-timerfd-syscall-test.patch \
file://0001-Add-missing-musl.supp.patch \
+   file://0001-Disable-temporarily-drd-tests-swapcontext.vgtest.patch \
"
 SRC_URI[md5sum] = "afe11b5572c3121a781433b7c0ab741b"
 SRC_URI[sha256sum] = 
"ad3aec668e813e40f238995f60796d9590eee64a16dff88421430630e69285a2"
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149958): 
https://lists.openembedded.org/g/openembedded-core/message/149958
Mute This Topic: https://lists.openembedded.org/mt/81615169/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] Autobuilder data collection for intermittent bugs

2021-03-25 Thread Sakib Sajal

Hi Richard,

I have been trying, with help from Randy, Trevor and Steve, to run some 
experiments on an internal autobuilder instance to collect host data 
during builds for signs of io latency.


Initially i modified yocto-autobuilder-helper/config.json and added the 
following 3 lines to existing templates (and created a commit) that were 
being used so that the builds would start collecting the data.



+    "BB_HEARTBEAT_EVENT = '5'",
+    "BB_LOG_HOST_STAT_ON_INTERVAL = '1'",
+    "BB_LOG_HOST_STAT_CMDS = 'pwd; ls'"

I also created a template in yocto-autobuilder-helper/config.json

+    "data-collect" : {
+    "BUILDINFO" : true,
+    "BBTARGETS" : "core-image-minimal",
+    "extravars" : [
+    "BB_HEARTBEAT_EVENT = '5'",
+    "BB_LOG_HOST_STAT_ON_INTERVAL = '1'",
+    "BB_LOG_HOST_STAT_CMDS = 'pwd; ls'"
+    ]
+    },

and modified a builder to use the template:

        "beaglebone-alt" : {
-    "MACHINE" : "beaglebone-yocto",
-    "TEMPLATE" : "altcfg-hw"
+    "MACHINE" : "qemux86-64",
+    "TEMPLATE" : "data-collect"
 },

Restarted both the controller and the worker, and started the 
beaglebone-alt build. Once the build started, i checked the conf file in 
the build directory but those lines were not appended to the local.conf 
file and hence no log files.


In summary, I have been trying to get the 3 lines from the "extravars" 
above to appear in one of the local.conf files. However none of the 
builds seems to have been altered.


I am wondering if the changes that I've made are even being picked up by 
the controller. Am I missing something?


Is there an easier way to update the conf files?

Initially I'd like collect data on a single node and scale it to all the 
nodes. Any ideas/suggestions are welcome!



Sincerely,

Sakib Sajal


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149956): 
https://lists.openembedded.org/g/openembedded-core/message/149956
Mute This Topic: https://lists.openembedded.org/mt/81615102/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 2/3] linux-yocto/5.4: update to v5.4.107

2021-03-25 Thread Bruce Ashfield
From: Bruce Ashfield 

Updating linux-yocto/5.4 to the latest korg -stable release that comprises
the following commits:

a65e78863443 Linux 5.4.107
5161cc4350de net: dsa: b53: Support setting learning on port
ebeefdc3d8ee net: dsa: tag_mtk: fix 802.1ad VLAN egress
6c3d86e6ffde crypto: x86/aes-ni-xts - use direct calls to and 4-way stride
ae69c97bb76e crypto: aesni - Use TEST %reg,%reg instead of CMP $0,%reg
eeb0899e0073 crypto: x86 - Regularize glue function prototypes
187ae0463653 fuse: fix live lock in fuse_iget()
28e53acd3065 drm/i915/gvt: Fix vfio_edid issue for BXT/APL
5a7c72ffb412 drm/i915/gvt: Fix port number for BDW on EDID region setup
4ab29329668d drm/i915/gvt: Fix virtual display setup for BXT/APL
e46f72e1f27c drm/i915/gvt: Fix mmio handler break on BXT/APL.
8cd68991b836 drm/i915/gvt: Set SNOOP for PAT3 on BXT/APL to workaround GPU 
BB hang
50f83ffc58ab btrfs: scrub: Don't check free space before marking a block 
group RO
591ea83fd2ce bpf, selftests: Fix up some test_verifier cases for 
unprivileged
4e4c85404a23 bpf: Add sanity check for upper ptr_limit
524471df8fa9 bpf: Simplify alu_limit masking for pointer arithmetic
2da0540739e4 bpf: Fix off-by-one for area size in creating mask to left
ea8fb45eaac1 bpf: Prohibit alu ops for pointer types not defining ptr_limit
010c5bee66bd KVM: arm64: nvhe: Save the SPE context early
0437de26e28d Linux 5.4.106
b802b6ef28d6 xen/events: avoid handling the same event on two cpus at the 
same time
92aefc62f483 xen/events: don't unmask an event channel when an eoi is 
pending
43d0b82bb45c xen/events: reset affinity of 2-level event when tearing it 
down
38563c1ff081 KVM: arm64: Reject VM creation when the default IPA size is 
unsupported
da2e37b55d4c KVM: arm64: Ensure I-cache isolation between vcpus of a same VM
4e2156c0d37b nvme: release namespace head reference on error
eb565f052b3e nvme: unlink head after removing last namespace
4535fb9ec5fd KVM: arm64: Fix exclusive limit for IPA size
e28b19ca2aeb x86/unwind/orc: Disable KASAN checking in the ORC unwinder, 
part 2
c0e0ab60d0b1 binfmt_misc: fix possible deadlock in bm_register_write
106fea9ad246 powerpc/64s: Fix instruction encoding for lis in 
ppc_function_entry()
907f7f2cf0ff sched/membarrier: fix missing local execution of 
ipi_sync_rq_state()
2306580a95b7 zram: fix return value on writeback_store
29e28a134a49 include/linux/sched/mm.h: use rcu_dereference in in_vfork()
99f1960cae4f stop_machine: mark helpers __always_inline
aaf92d0538d2 hrtimer: Update softirq_expires_next correctly after 
__hrtimer_get_next_event()
88c79851b82d arm64: mm: use a 48-bit ID map when possible on 52-bit VA 
builds
73aa6f93e1e9 configfs: fix a use-after-free in __configfs_open_file
babd55002dd4 block: rsxx: fix error return code of rsxx_pci_probe()
41deefab452a NFSv4.2: fix return value of _nfs4_get_security_label()
86954a52d829 NFS: Don't gratuitously clear the inode cache when lookup 
failed
d29f9aa6a8b2 NFS: Don't revalidate the directory permissions on a lookup 
failure
d5a69ed75931 SUNRPC: Set memalloc_nofs_save() for sync tasks
9c9ea7ac18b2 arm64/mm: Fix pfn_valid() for ZONE_DEVICE based memory
19bb2a20710d sh_eth: fix TRSCER mask for R7S72100
c3c1defad2dd staging: comedi: pcl818: Fix endian problem for AI command data
c5916897a6e1 staging: comedi: pcl711: Fix endian problem for AI command data
7d8ec7bef320 staging: comedi: me4000: Fix endian problem for AI command data
e70294943c89 staging: comedi: dmm32at: Fix endian problem for AI command 
data
47a2af64eea3 staging: comedi: das800: Fix endian problem for AI command data
0f2522ec71b6 staging: comedi: das6402: Fix endian problem for AI command 
data
e91490b9edb9 staging: comedi: adv_pci1710: Fix endian problem for AI 
command data
4d6505edee5a staging: comedi: addi_apci_1500: Fix endian problem for 
command sample
f258c1c26f64 staging: comedi: addi_apci_1032: Fix endian problem for COS 
sample
e644fc4ab7bb staging: rtl8192e: Fix possible buffer overflow in 
_rtl92e_wx_set_scan
8f586a59829b staging: rtl8712: Fix possible buffer overflow in 
r8712_sitesurvey_cmd
9fe42273b2c6 staging: ks7010: prevent buffer overflow in ks_wlan_set_scan()
ab42f28d5f34 staging: rtl8188eu: fix potential memory corruption in 
rtw_check_beacon_data()
1a866057e970 staging: rtl8712: unterminated string leads to read overflow
da5abe369b03 staging: rtl8188eu: prevent ->ssid overflow in 
rtw_wx_set_scan()
a311b6a7f099 staging: rtl8192u: fix ->ssid overflow in r8192_wx_set_scan()
e4b52c7cbaaf misc: fastrpc: restrict user apps from sending kernel RPC 
messages
9009b59dfd5f misc/pvpanic: Export module FDT device table
0a58a400a93b usbip: fix vudc usbip_sockfd_store races leading to gpf
8a50dda5243e usbip: fix vhci_hcd attach_store() races 

[OE-core] [PATCH 1/3] linux-yocto/5.10: update to v5.10.25

2021-03-25 Thread Bruce Ashfield
From: Bruce Ashfield 

Updating linux-yocto/5.10 to the latest korg -stable release that comprises
the following commits:

3ba56f490c7a Linux 5.10.25
dd0b7edb7794 net: dsa: b53: Support setting learning on port
0f6cab2350d5 ALSA: usb-audio: Don't avoid stopping the stream at 
disconnection
df8596f57743 Revert "nfsd4: a client's own opens needn't prevent 
delegations"
894ecf0cb505 Revert "nfsd4: remove check_conflicting_opens warning"
d955f13ea212 fuse: fix live lock in fuse_iget()
2d7888b2c4cd RDMA/srp: Fix support for unpopulated and unbalanced NUMA nodes
3672c3ce622e bpf, selftests: Fix up some test_verifier cases for 
unprivileged
1010f17aaa78 bpf: Add sanity check for upper ptr_limit
6a3504bf4006 bpf: Simplify alu_limit masking for pointer arithmetic
ac1b87a18c1f bpf: Fix off-by-one for area size in creating mask to left
c4d37eea1c64 bpf: Prohibit alu ops for pointer types not defining ptr_limit
bf93113d46f8 crypto: x86/aes-ni-xts - use direct calls to and 4-way stride
fcfab1a9aa40 crypto: aesni - Use TEST %reg,%reg instead of CMP $0,%reg
05d125f7524e Linux 5.10.24
1c0899636d44 RDMA/umem: Use ib_dma_max_seg_size instead of 
dma_get_max_seg_size
1dbce9ba2aa9 KVM: arm64: Fix nVHE hyp panic host context restore
f67e5243d0f3 xen/events: avoid handling the same event on two cpus at the 
same time
30cdb862e831 xen/events: don't unmask an event channel when an eoi is 
pending
4c84191cbc3e mm/page_alloc.c: refactor initialization of struct page for 
holes in memory layout
e7afadd0dbe2 KVM: arm64: Ensure I-cache isolation between vcpus of a same VM
518f98e39077 mm/madvise: replace ptrace attach requirement for 
process_madvise
2aaa79f69417 mm/userfaultfd: fix memory corruption due to writeprotect
c3d70b1bf1ce KVM: arm64: Fix exclusive limit for IPA size
ada8817ab674 KVM: arm64: Reject VM creation when the default IPA size is 
unsupported
eeba4e4cc524 KVM: arm64: nvhe: Save the SPE context early
a9779820bb97 KVM: arm64: Avoid corrupting vCPU context register in guest 
exit
17becbfca9fc KVM: arm64: Fix range alignment when walking page tables
a688bf8cf550 KVM: kvmclock: Fix vCPUs > 64 can't be online/hotpluged
4ab5d1b70929 KVM: x86: Ensure deadline timer has truly expired before 
posting its IRQ
e40384fcd600 x86/entry: Fix entry/exit mismatch on failed fast 32-bit 
syscalls
a2bab396cb97 x86/sev-es: Use __copy_from_user_inatomic()
977b9f4190ad x86/sev-es: Correctly track IRQ states in runtime #VC handler
269424432731 x86/entry: Move nmi entry/exit into common code
752fbe0c8ddd x86/sev-es: Check regs->sp is trusted before adjusting #VC IST 
stack
871fd1e3ee8e x86/sev-es: Introduce ip_within_syscall_gap() helper
d327d8632cdb x86/unwind/orc: Disable KASAN checking in the ORC unwinder, 
part 2
5ab9464a2a3c binfmt_misc: fix possible deadlock in bm_register_write
0e4750f69c17 powerpc: Fix missing declaration of [en/dis]able_kernel_vsx()
1f372e89567b powerpc: Fix inverted SET_FULL_REGS bitop
9776812ee861 powerpc/64s: Fix instruction encoding for lis in 
ppc_function_entry()
8571c66401ea efi: stub: omit SetVirtualAddressMap() if marked unsupported 
in RT_PROP table
68b4378d910e sched/membarrier: fix missing local execution of 
ipi_sync_rq_state()
5f2f616343b1 linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
bc7c1b09f7a4 zram: fix return value on writeback_store
3cbe8f9193e6 include/linux/sched/mm.h: use rcu_dereference in in_vfork()
7da7542c04a4 stop_machine: mark helpers __always_inline
2a39eb7b8670 seqlock,lockdep: Fix seqcount_latch_init()
372734dc1897 powerpc/64s/exception: Clean up a missed SRR specifier
df7dbfc24c33 hrtimer: Update softirq_expires_next correctly after 
__hrtimer_get_next_event()
896846b8151d perf/x86/intel: Set PERF_ATTACH_SCHED_CB for large PEBS and LBR
82ad50c112f8 perf/core: Flush PMU internal buffers for per-CPU events
3ebd4bd2eb6f arm64: mm: use a 48-bit ID map when possible on 52-bit VA 
builds
109720342efd configfs: fix a use-after-free in __configfs_open_file
6cf11f3a09a2 nvme-fc: fix racing controller reset and create association
d1d918492e6e block: rsxx: fix error return code of rsxx_pci_probe()
caa86901c863 NFSv4.2: fix return value of _nfs4_get_security_label()
e181960ec51d NFS: Don't gratuitously clear the inode cache when lookup 
failed
dd756d05bee5 NFS: Don't revalidate the directory permissions on a lookup 
failure
faa48b23d0e3 SUNRPC: Set memalloc_nofs_save() for sync tasks
475a4307c14c arm64/mm: Fix pfn_valid() for ZONE_DEVICE based memory
e50ada589497 cpufreq: qcom-hw: Fix return value check in 
qcom_cpufreq_hw_cpu_init()
7dfe37e9ea69 cpufreq: qcom-hw: fix dereferencing freed memory 'data'
75d9be57cf2e sh_eth: fix TRSCER mask for R7S72100
a854bd051465 staging: comedi: pcl818: Fix endian problem for AI command data

[OE-core] [PATCH 0/3] linux-yocto: -stable updates

2021-03-25 Thread Bruce Ashfield
From: Bruce Ashfield 

Richard,

I had some pings about v5.10.25, so I thought I'd send out this round of
-stable udpates while I finish up some other config warning cleanups.

These are routine -stable and -rt bugfix updates, and I don't expect any
issues.

Bruce

The following changes since commit ea455ca8671d3bc2a1097989bfaabe92f3ca37ab:

  build-appliance-image: Update to master head revision (2021-03-23 22:56:44 
+)

are available in the Git repository at:

  git://git.yoctoproject.org/poky-contrib zedd/kernel
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=zedd/kernel

Bruce Ashfield (3):
  linux-yocto/5.10: update to v5.10.25
  linux-yocto/5.4: update to v5.4.107
  linux-yocto-rt/5.10: update to -rt34

 .../linux/linux-yocto-rt_5.10.bb  |  6 ++---
 .../linux/linux-yocto-rt_5.4.bb   |  6 ++---
 .../linux/linux-yocto-tiny_5.10.bb|  8 +++
 .../linux/linux-yocto-tiny_5.4.bb |  8 +++
 meta/recipes-kernel/linux/linux-yocto_5.10.bb | 24 +--
 meta/recipes-kernel/linux/linux-yocto_5.4.bb  | 22 -
 6 files changed, 37 insertions(+), 37 deletions(-)

-- 
2.19.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149952): 
https://lists.openembedded.org/g/openembedded-core/message/149952
Mute This Topic: https://lists.openembedded.org/mt/81612931/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 3/3] linux-yocto-rt/5.10: update to -rt34

2021-03-25 Thread Bruce Ashfield
From: Bruce Ashfield 

Integrating the following commit(s) to linux-yocto/5.10:

be2935bce35f v5.10.21-rt34
8078987238f9 softirq: Update the softirq/tasklet patches
0042f5e5ac7d mm: slub: Don't resize the location tracking cache on 
PREEMPT_RT
69bcb4682eaa v5.10.21-rt33
75e139bb405a v5.10.17-rt32
209e0ad0f61d printk: Update the printk code
f1e0daad5cd4 trace: Add the flags for need_resched_lazy()

Signed-off-by: Bruce Ashfield 
---
 meta/recipes-kernel/linux/linux-yocto-rt_5.10.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.10.bb 
b/meta/recipes-kernel/linux/linux-yocto-rt_5.10.bb
index 840cef88e2..cb34887cda 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.10.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.10.bb
@@ -11,7 +11,7 @@ python () {
 raise bb.parse.SkipRecipe("Set PREFERRED_PROVIDER_virtual/kernel to 
linux-yocto-rt to enable it")
 }
 
-SRCREV_machine ?= "5ae660cafcd609de82c450999487d4e6169f9f1f"
+SRCREV_machine ?= "be2935bce35f9adb6d0e735d42651e81a5094adf"
 SRCREV_meta ?= "031f6c76e488a3563f35258c72ff1de3e25a512e"
 
 SRC_URI = 
"git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \
-- 
2.19.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149955): 
https://lists.openembedded.org/g/openembedded-core/message/149955
Mute This Topic: https://lists.openembedded.org/mt/81612936/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] grub-efi: Fix build failure on RISCV32

2021-03-25 Thread Alistair Francis
On Thu, Mar 25, 2021 at 12:15 AM Khem Raj  wrote:
>
> Signed-off-by: Khem Raj 
> Cc: Alistair Francis 

I'm not sure if this is the right thing to do here.

Have you tested this does GRUB boot with this check removed?

>From memory I don't think the original GRUB port would have been tested on 
>RV32.

Alistair

> ---
>  ...sable-offset-overflow-check-for-rv32.patch | 45 +++
>  meta/recipes-bsp/grub/grub2.inc   |  1 +
>  2 files changed, 46 insertions(+)
>  create mode 100644 
> meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
>
> diff --git 
> a/meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
>  
> b/meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
> new file mode 100644
> index 00..6547bf2261
> --- /dev/null
> +++ 
> b/meta/recipes-bsp/grub/files/0001-Disable-offset-overflow-check-for-rv32.patch
> @@ -0,0 +1,45 @@
> +From ce1cea0563011698e4a70c9a60e7da66b57ab1d6 Mon Sep 17 00:00:00 2001
> +From: Khem Raj 
> +Date: Wed, 24 Mar 2021 20:55:56 -0700
> +Subject: [PATCH] Disable offset overflow check for rv32
> +
> +Fixes
> +/tmp/grub/grub-mkimage: error: target 17c6 not reachable from pc=936.
> +
> +Upstream-Status: Reported [https://savannah.gnu.org/bugs/index.php?60283]
> +Signed-off-by: Khem Raj 
> +---
> + util/grub-mkimagexx.c | 6 --
> + 1 file changed, 4 insertions(+), 2 deletions(-)
> +
> +diff --git a/util/grub-mkimagexx.c b/util/grub-mkimagexx.c
> +index 00f49ccaa..6deb66226 100644
> +--- a/util/grub-mkimagexx.c
>  b/util/grub-mkimagexx.c
> +@@ -1297,9 +1297,10 @@ SUFFIX (relocate_addrs) (Elf_Ehdr *e, struct 
> section_metadata *smd,
> +{
> +  grub_uint32_t hi20, lo12;
> +
> ++#ifdef MKIMAGE_ELF64
> +  if (off != (grub_int32_t)off)
> +grub_util_error ("target %lx not reachable from 
> pc=%lx", (long)sym_addr, (long)((char *)target - (char *)e));
> +-
> ++#endif
> +  hi20 = (off + 0x800) & 0xf000;
> +  lo12 = (off - hi20) & 0xfff;
> +  t32[0] = grub_host_to_target32 ((grub_target_to_host32 
> (t32[0]) & 0xfff) | hi20);
> +@@ -1336,9 +1337,10 @@ SUFFIX (relocate_addrs) (Elf_Ehdr *e, struct 
> section_metadata *smd,
> +{
> +  grub_int32_t hi20;
> +
> ++#ifdef MKIMAGE_ELF64
> +  if (off != (grub_int32_t)off)
> +grub_util_error ("target %lx not reachable from 
> pc=%lx", (long)sym_addr, (long)((char *)target - (char *)e));
> +-
> ++#endif
> +  hi20 = (off + 0x800) & 0xf000;
> +  *t32 = grub_host_to_target32 ((grub_target_to_host32 
> (*t32) & 0xfff) | hi20);
> +}
> +--
> +2.31.0
> +
> diff --git a/meta/recipes-bsp/grub/grub2.inc b/meta/recipes-bsp/grub/grub2.inc
> index bcff676c26..d8a556e06c 100644
> --- a/meta/recipes-bsp/grub/grub2.inc
> +++ b/meta/recipes-bsp/grub/grub2.inc
> @@ -19,6 +19,7 @@ SRC_URI = 
> "https://alpha.gnu.org/gnu/grub/grub-${REALPV}.tar.xz \
> file://grub-module-explicitly-keeps-symbole-.module_license.patch 
> \
> file://0001-grub.d-10_linux.in-add-oe-s-kernel-name.patch \
> file://determinism.patch \
> +   file://0001-Disable-offset-overflow-check-for-rv32.patch \
>  "
>
>  SRC_URI[sha256sum] = 
> "2c87f1f21e2ab50043e6cd9163c08f1b6c3a6171556bf23ff9ed65b074145484"
> --
> 2.31.0
>
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149951): 
https://lists.openembedded.org/g/openembedded-core/message/149951
Mute This Topic: https://lists.openembedded.org/mt/81595170/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH][gatesgarth][dunfell] openssl: update to 1.1.1k to fix CVE-2021-3450 and CVE-2021-3449

2021-03-25 Thread Mikko Rapeli
Only security issues fixed in this release according to
https://www.openssl.org/news/cl111.txt

Signed-off-by: Mikko Rapeli 
---
 .../openssl/{openssl_1.1.1j.bb => openssl_1.1.1k.bb}| 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-connectivity/openssl/{openssl_1.1.1j.bb => 
openssl_1.1.1k.bb} (98%)

diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1j.bb 
b/meta/recipes-connectivity/openssl/openssl_1.1.1k.bb
similarity index 98%
rename from meta/recipes-connectivity/openssl/openssl_1.1.1j.bb
rename to meta/recipes-connectivity/openssl/openssl_1.1.1k.bb
index f054d2fdba..5f281197c9 100644
--- a/meta/recipes-connectivity/openssl/openssl_1.1.1j.bb
+++ b/meta/recipes-connectivity/openssl/openssl_1.1.1k.bb
@@ -23,7 +23,7 @@ SRC_URI_append_class-nativesdk = " \
file://environment.d-openssl.sh \
"
 
-SRC_URI[sha256sum] = 
"aaf2fcb575cdf6491b98ab4829abf78a3dec8402b8b81efc8f23c00d443981bf"
+SRC_URI[sha256sum] = 
"892a0875b9872acd04a9fde79b1f943075d5ea162415de3047c327df33fbaee5"
 
 inherit lib_package multilib_header multilib_script ptest
 MULTILIB_SCRIPTS = "${PN}-bin:${bindir}/c_rehash"
-- 
2.20.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149950): 
https://lists.openembedded.org/g/openembedded-core/message/149950
Mute This Topic: https://lists.openembedded.org/mt/81611322/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH][master] openssl: update to 1.1.1k to fix CVE-2021-3450 and CVE-2021-3449

2021-03-25 Thread Mikko Rapeli
Only security issues fixed in this release according to
https://www.openssl.org/news/cl111.txt

Signed-off-by: Mikko Rapeli 
---
 .../openssl/{openssl_1.1.1j.bb => openssl_1.1.1k.bb}| 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-connectivity/openssl/{openssl_1.1.1j.bb => 
openssl_1.1.1k.bb} (98%)

diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1j.bb 
b/meta/recipes-connectivity/openssl/openssl_1.1.1k.bb
similarity index 98%
rename from meta/recipes-connectivity/openssl/openssl_1.1.1j.bb
rename to meta/recipes-connectivity/openssl/openssl_1.1.1k.bb
index ed6d7e0cd4..e518cb6a02 100644
--- a/meta/recipes-connectivity/openssl/openssl_1.1.1j.bb
+++ b/meta/recipes-connectivity/openssl/openssl_1.1.1k.bb
@@ -28,7 +28,7 @@ SRC_URI_append_riscv32 = " \
file://0004-Fixup-support-for-io_pgetevents_time64-syscall.patch \
"
 
-SRC_URI[sha256sum] = 
"aaf2fcb575cdf6491b98ab4829abf78a3dec8402b8b81efc8f23c00d443981bf"
+SRC_URI[sha256sum] = 
"892a0875b9872acd04a9fde79b1f943075d5ea162415de3047c327df33fbaee5"
 
 inherit lib_package multilib_header multilib_script ptest
 MULTILIB_SCRIPTS = "${PN}-bin:${bindir}/c_rehash"
-- 
2.20.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149949): 
https://lists.openembedded.org/g/openembedded-core/message/149949
Mute This Topic: https://lists.openembedded.org/mt/81611320/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2 1/3] meson: fix native/host confusion in gobject-introspection

2021-03-25 Thread Ross Burton
On Thu, 25 Mar 2021 at 18:15, Alexander Kanavin  wrote:
> I don't think I fully understand this. Is g-i in meson projects already 
> broken for a while (there was a bit of refactor in the latest meson version, 
> which I tested, but perhaps not well enough), or is this necessary for the 
> other patches in your set?

I just sent a mail which sort-of answers this question without seeing it.

Basically, the change is that Meson now knows that it should call
pkg-config-native for native pkgconfig lookups.  Before, it was using
pkg-config.

As far as I can tell there is very little code which hits that
codepath, so we didn't notice.  When setting pkgconfig-native
correctly, this broke.

Ross

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149948): 
https://lists.openembedded.org/g/openembedded-core/message/149948
Mute This Topic: https://lists.openembedded.org/mt/81608826/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH 2/2] meson: use native-file instead of environment variables

2021-03-25 Thread Ross Burton
So v1 failed all over the AB, the root cause being that in master
Meson doesn't know that there's a specific pkg-config binary for all
native builds, but with this series it does.  In master it only uses
pkg-config-native in native *recipes* but now it uses it for all
native lookups in target recipes too.

This means that when the G-I code uses a native pkgconfig lookup to
get target library dependencies, it goes terribly wrong.

Upstream are basically confused as to how "cross" and "G-I" are meant
to work, because they from a glance can't easily, so I expect to be
able to get this patch upstream as it doesn't break what already works
and is a step towards the correct model.

Ross

On Tue, 23 Mar 2021 at 21:28, Ross Burton via lists.openembedded.org
 wrote:
>
> Meson now supports native-files, which are the same as cross files but
> describe the native build.
>
> By writing and using a native file which describes the tools to use, we
> can drop the environment variable overriding.
>
> Signed-off-by: Ross Burton 
> ---
>  meta/classes/meson.bbclass | 49 --
>  1 file changed, 21 insertions(+), 28 deletions(-)
>
> diff --git a/meta/classes/meson.bbclass b/meta/classes/meson.bbclass
> index a7644e70cb..290b39e1e0 100644
> --- a/meta/classes/meson.bbclass
> +++ b/meta/classes/meson.bbclass
> @@ -26,7 +26,8 @@ MESONOPTS = " --prefix ${prefix} \
>--sysconfdir ${sysconfdir} \
>--localstatedir ${localstatedir} \
>--sharedstatedir ${sharedstatedir} \
> -  --wrap-mode nodownload"
> +  --wrap-mode nodownload \
> +  --native-file ${WORKDIR}/native.cross"
>
>  EXTRA_OEMESON_append = " ${PACKAGECONFIG_CONFARGS}"
>
> @@ -121,9 +122,28 @@ system = '${@meson_operating_system('TARGET_OS', d)}'
>  cpu_family = '${@meson_cpu_family('TARGET_ARCH', d)}'
>  cpu = '${TARGET_ARCH}'
>  endian = '${@meson_endian('TARGET', d)}'
> +EOF
> +
> +cat >${WORKDIR}/native.cross < +[binaries]
> +c = ${@meson_array('BUILD_CC', d)}
> +cpp = ${@meson_array('BUILD_CXX', d)}
> +ar = ${@meson_array('BUILD_AR', d)}
> +nm = ${@meson_array('BUILD_NM', d)}
> +strip = ${@meson_array('BUILD_STRIP', d)}
> +readelf = ${@meson_array('BUILD_READELF', d)}
> +pkgconfig = 'pkg-config-native'
> +
> +[properties]
> +needs_exe_wrapper = false
> +c_args = ${@meson_array('BUILD_CFLAGS', d)}
> +c_link_args = ${@meson_array('BUILD_LDFLAGS', d)}
> +cpp_args = ${@meson_array('BUILD_CXXFLAGS', d)}
> +cpp_link_args = ${@meson_array('BUILD_LDFLAGS', d)}
>  EOF
>  }
>
> +# Tell externalsrc that changes to this file require a reconfigure
>  CONFIGURE_FILES = "meson.build"
>
>  meson_do_configure() {
> @@ -140,33 +160,6 @@ meson_do_configure() {
>  fi
>  }
>
> -override_native_tools() {
> -# Set these so that meson uses the native tools for its build sanity 
> tests,
> -# which require executables to be runnable. The cross file will still
> -# override these for the target build.
> -export CC="${BUILD_CC}"
> -export CXX="${BUILD_CXX}"
> -export LD="${BUILD_LD}"
> -export AR="${BUILD_AR}"
> -export STRIP="${BUILD_STRIP}"
> -# These contain *target* flags but will be used as *native* flags.  The
> -# correct native flags will be passed via -Dc_args and so on, unset them 
> so
> -# they don't interfere with tools invoked by Meson (such as g-ir-scanner)
> -unset CPPFLAGS CFLAGS CXXFLAGS LDFLAGS
> -}
> -
> -meson_do_configure_prepend_class-target() {
> -override_native_tools
> -}
> -
> -meson_do_configure_prepend_class-nativesdk() {
> -override_native_tools
> -}
> -
> -meson_do_configure_prepend_class-native() {
> -export PKG_CONFIG="pkg-config-native"
> -}
> -
>  python meson_do_qa_configure() {
>  import re
>  warn_re = re.compile(r"^WARNING: Cross property (.+) is using default 
> value (.+)$", re.MULTILINE)
> --
> 2.25.1
>
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149947): 
https://lists.openembedded.org/g/openembedded-core/message/149947
Mute This Topic: https://lists.openembedded.org/mt/81562048/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH V2] linux-firmware: Package RSI 911x WiFi firmware

2021-03-25 Thread Marek Vasut

On 3/12/21 9:57 PM, Marek Vasut wrote:

The RSI 911x WiFi firmware is already part of the linux-firmware
repository, package it to make it easily available.

Signed-off-by: Marek Vasut 
Cc: Richard Purdie 
Cc: Steve Sakoman 
---
V2: Rebase on current oe-core/master


Bump ?

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149946): 
https://lists.openembedded.org/g/openembedded-core/message/149946
Mute This Topic: https://lists.openembedded.org/mt/81290193/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2 1/3] meson: fix native/host confusion in gobject-introspection

2021-03-25 Thread Alexander Kanavin
I don't think I fully understand this. Is g-i in meson projects already
broken for a while (there was a bit of refactor in the latest meson
version, which I tested, but perhaps not well enough), or is this necessary
for the other patches in your set?

Alex

On Thu, 25 Mar 2021 at 18:39, Ross Burton  wrote:

> When building G-I we want to use *native* binaries (as they need to be
> executed) but the *cross* libraries, as otherwise when using the correct
> pkg-config binary in native lookups Meson will end up linking native and
> cross libraries together.
>
> Signed-off-by: Ross Burton 
> ---
>  meta/recipes-devtools/meson/meson.inc |  1 +
>  .../meson/meson/gi-target-dep.patch   | 41 +++
>  2 files changed, 42 insertions(+)
>  create mode 100644 meta/recipes-devtools/meson/meson/gi-target-dep.patch
>
> diff --git a/meta/recipes-devtools/meson/meson.inc
> b/meta/recipes-devtools/meson/meson.inc
> index fc3ca85fae..e186dede92 100644
> --- a/meta/recipes-devtools/meson/meson.inc
> +++ b/meta/recipes-devtools/meson/meson.inc
> @@ -15,6 +15,7 @@ SRC_URI = "
> https://github.com/mesonbuild/meson/releases/download/${PV}/meson-${P
> file://cross-prop-default.patch \
>
> file://0001-modules-python.py-do-not-substitute-python-s-install.patch \
>
> file://0001-minstall-Correctly-set-uid-gid-of-installed-files.patch \
> +   file://gi-target-dep.patch \
> "
>  SRC_URI[sha256sum] =
> "72e1c782ba9bda204f4a1ed57f98d027d7b6eb9414c723eebbd6ec7f1955c8a6"
>
> diff --git a/meta/recipes-devtools/meson/meson/gi-target-dep.patch
> b/meta/recipes-devtools/meson/meson/gi-target-dep.patch
> new file mode 100644
> index 00..25b30ca27a
> --- /dev/null
> +++ b/meta/recipes-devtools/meson/meson/gi-target-dep.patch
> @@ -0,0 +1,41 @@
> +When building gobject-introspection we want the *native* binaries (as
> they need
> +to be executed) but *host* gobject-introspection libraries, as otherwise
> the
> +native pkg-config can be used and the build will try to link native and
> host
> +binaries together.
> +
> +Upstream-Status: Pending
> +Signed-off-by: Ross Burton 
> +
> +diff --git a/mesonbuild/modules/gnome.py b/mesonbuild/modules/gnome.py
> +index f9660838e..2eefc3fc7 100644
> +--- a/mesonbuild/modules/gnome.py
>  b/mesonbuild/modules/gnome.py
> +@@ -84,8 +84,8 @@ class GnomeModule(ExtensionModule):
> +  mlog.bold('
> https://github.com/mesonbuild/meson/issues/1387'),
> +  once=True)
> +
> +-def _get_native_dep(self, state, depname, required=True):
> +-kwargs = {'native': True, 'required': required}
> ++def _get_dep(self, state, depname, native=False, required=True):
> ++kwargs = {'native': native, 'required': required}
> + holder = self.interpreter.func_dependency(state.current_node,
> [depname], kwargs)
> + return holder.held_object
> +
> +@@ -101,7 +101,7 @@ class GnomeModule(ExtensionModule):
> + return ExternalProgram.from_entry(name, prog)
> +
> + # Check if pkgconfig has a variable
> +-dep = self._get_native_dep(state, depname, required=False)
> ++dep = self._get_dep(state, depname, native=True, required=False)
> + if dep.found() and dep.type_name == 'pkgconfig':
> + value = dep.get_pkgconfig_variable(varname, {})
> + if value:
> +@@ -481,7 +481,7 @@ class GnomeModule(ExtensionModule):
> +
> + def _get_gir_dep(self, state):
> + if not self.gir_dep:
> +-self.gir_dep = self._get_native_dep(state,
> 'gobject-introspection-1.0')
> ++self.gir_dep = self._get_dep(state,
> 'gobject-introspection-1.0')
> + self.giscanner = self._get_native_binary(state,
> 'g-ir-scanner', 'gobject-introspection-1.0', 'g_ir_scanner')
> + self.gicompiler = self._get_native_binary(state,
> 'g-ir-compiler', 'gobject-introspection-1.0', 'g_ir_compiler')
> + return self.gir_dep, self.giscanner, self.gicompiler
> --
> 2.25.1
>
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149945): 
https://lists.openembedded.org/g/openembedded-core/message/149945
Mute This Topic: https://lists.openembedded.org/mt/81608826/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v2 2/3] bitbake.conf: ensure BUILD_* tools match target tools

2021-03-25 Thread Ross Burton
Add a few more tools to the BUILD_* list, to match the target tool list.

Signed-off-by: Ross Burton 
---
 meta/conf/bitbake.conf | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index ecd4d1638e..4c472156e9 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -541,7 +541,10 @@ export BUILD_AR = "${BUILD_PREFIX}ar"
 export BUILD_AS = "${BUILD_PREFIX}as ${BUILD_AS_ARCH}"
 export BUILD_RANLIB = "${BUILD_PREFIX}ranlib"
 export BUILD_STRIP = "${BUILD_PREFIX}strip"
+BUILD_OBJCOPY = "${BUILD_PREFIX}objcopy"
+BUILD_OBJDUMP = "${BUILD_PREFIX}objdump"
 export BUILD_NM = "${BUILD_PREFIX}nm"
+BUILD_READELF = "${BUILD_PREFIX}readelf"
 
 export MAKE = "make"
 EXTRA_OEMAKE = ""
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149943): 
https://lists.openembedded.org/g/openembedded-core/message/149943
Mute This Topic: https://lists.openembedded.org/mt/81608827/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v2 1/3] meson: fix native/host confusion in gobject-introspection

2021-03-25 Thread Ross Burton
When building G-I we want to use *native* binaries (as they need to be
executed) but the *cross* libraries, as otherwise when using the correct
pkg-config binary in native lookups Meson will end up linking native and
cross libraries together.

Signed-off-by: Ross Burton 
---
 meta/recipes-devtools/meson/meson.inc |  1 +
 .../meson/meson/gi-target-dep.patch   | 41 +++
 2 files changed, 42 insertions(+)
 create mode 100644 meta/recipes-devtools/meson/meson/gi-target-dep.patch

diff --git a/meta/recipes-devtools/meson/meson.inc 
b/meta/recipes-devtools/meson/meson.inc
index fc3ca85fae..e186dede92 100644
--- a/meta/recipes-devtools/meson/meson.inc
+++ b/meta/recipes-devtools/meson/meson.inc
@@ -15,6 +15,7 @@ SRC_URI = 
"https://github.com/mesonbuild/meson/releases/download/${PV}/meson-${P
file://cross-prop-default.patch \

file://0001-modules-python.py-do-not-substitute-python-s-install.patch \
file://0001-minstall-Correctly-set-uid-gid-of-installed-files.patch 
\
+   file://gi-target-dep.patch \
"
 SRC_URI[sha256sum] = 
"72e1c782ba9bda204f4a1ed57f98d027d7b6eb9414c723eebbd6ec7f1955c8a6"
 
diff --git a/meta/recipes-devtools/meson/meson/gi-target-dep.patch 
b/meta/recipes-devtools/meson/meson/gi-target-dep.patch
new file mode 100644
index 00..25b30ca27a
--- /dev/null
+++ b/meta/recipes-devtools/meson/meson/gi-target-dep.patch
@@ -0,0 +1,41 @@
+When building gobject-introspection we want the *native* binaries (as they need
+to be executed) but *host* gobject-introspection libraries, as otherwise the
+native pkg-config can be used and the build will try to link native and host
+binaries together.
+
+Upstream-Status: Pending
+Signed-off-by: Ross Burton 
+
+diff --git a/mesonbuild/modules/gnome.py b/mesonbuild/modules/gnome.py
+index f9660838e..2eefc3fc7 100644
+--- a/mesonbuild/modules/gnome.py
 b/mesonbuild/modules/gnome.py
+@@ -84,8 +84,8 @@ class GnomeModule(ExtensionModule):
+  
mlog.bold('https://github.com/mesonbuild/meson/issues/1387'),
+  once=True)
+ 
+-def _get_native_dep(self, state, depname, required=True):
+-kwargs = {'native': True, 'required': required}
++def _get_dep(self, state, depname, native=False, required=True):
++kwargs = {'native': native, 'required': required}
+ holder = self.interpreter.func_dependency(state.current_node, 
[depname], kwargs)
+ return holder.held_object
+ 
+@@ -101,7 +101,7 @@ class GnomeModule(ExtensionModule):
+ return ExternalProgram.from_entry(name, prog)
+ 
+ # Check if pkgconfig has a variable
+-dep = self._get_native_dep(state, depname, required=False)
++dep = self._get_dep(state, depname, native=True, required=False)
+ if dep.found() and dep.type_name == 'pkgconfig':
+ value = dep.get_pkgconfig_variable(varname, {})
+ if value:
+@@ -481,7 +481,7 @@ class GnomeModule(ExtensionModule):
+ 
+ def _get_gir_dep(self, state):
+ if not self.gir_dep:
+-self.gir_dep = self._get_native_dep(state, 
'gobject-introspection-1.0')
++self.gir_dep = self._get_dep(state, 'gobject-introspection-1.0')
+ self.giscanner = self._get_native_binary(state, 'g-ir-scanner', 
'gobject-introspection-1.0', 'g_ir_scanner')
+ self.gicompiler = self._get_native_binary(state, 'g-ir-compiler', 
'gobject-introspection-1.0', 'g_ir_compiler')
+ return self.gir_dep, self.giscanner, self.gicompiler
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149942): 
https://lists.openembedded.org/g/openembedded-core/message/149942
Mute This Topic: https://lists.openembedded.org/mt/81608826/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH v2 3/3] meson: use native-file instead of environment variables

2021-03-25 Thread Ross Burton
Meson now supports native-files, which are the same as cross files but
describe the native build.

By writing and using a native file which describes the tools to use, we
can drop the environment variable overriding.

Signed-off-by: Ross Burton 
---
 meta/classes/meson.bbclass | 54 +-
 1 file changed, 24 insertions(+), 30 deletions(-)

diff --git a/meta/classes/meson.bbclass b/meta/classes/meson.bbclass
index a7644e70cb..bf9b02e06e 100644
--- a/meta/classes/meson.bbclass
+++ b/meta/classes/meson.bbclass
@@ -26,7 +26,8 @@ MESONOPTS = " --prefix ${prefix} \
   --sysconfdir ${sysconfdir} \
   --localstatedir ${localstatedir} \
   --sharedstatedir ${sharedstatedir} \
-  --wrap-mode nodownload"
+  --wrap-mode nodownload \
+  --native-file ${WORKDIR}/meson.native"
 
 EXTRA_OEMESON_append = " ${PACKAGECONFIG_CONFARGS}"
 
@@ -102,12 +103,14 @@ cups-config = 'cups-config'
 g-ir-scanner = '${STAGING_BINDIR}/g-ir-scanner-wrapper'
 g-ir-compiler = '${STAGING_BINDIR}/g-ir-compiler-wrapper'
 
-[properties]
-needs_exe_wrapper = true
+[built-in options]
 c_args = ${@meson_array('CFLAGS', d)}
 c_link_args = ${@meson_array('LDFLAGS', d)}
 cpp_args = ${@meson_array('CXXFLAGS', d)}
 cpp_link_args = ${@meson_array('LDFLAGS', d)}
+
+[properties]
+needs_exe_wrapper = true
 gtkdoc_exe_wrapper = '${B}/gtkdoc-qemuwrapper'
 
 [host_machine]
@@ -121,9 +124,27 @@ system = '${@meson_operating_system('TARGET_OS', d)}'
 cpu_family = '${@meson_cpu_family('TARGET_ARCH', d)}'
 cpu = '${TARGET_ARCH}'
 endian = '${@meson_endian('TARGET', d)}'
+EOF
+
+cat >${WORKDIR}/meson.native <
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149944): 
https://lists.openembedded.org/g/openembedded-core/message/149944
Mute This Topic: https://lists.openembedded.org/mt/81608828/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Richard Purdie
On Thu, 2021-03-25 at 16:19 +, Peter Kjellerstedt wrote:
> > -Original Message-
> > From: Richard Purdie 
> > Sent: den 25 mars 2021 15:27
> > To: Peter Kjellerstedt ; Oleksiy Obitotskyi -
> > X (oobitots - GLOBALLOGIC INC at Cisco) ; Luca Bocassi
> > ; openembedded-core@lists.openembedded.org
> > Cc: bluelightn...@bluelightning.org; Khem Raj 
> > Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> > recipe to allow bootstrapping
> > 
> > On Thu, 2021-03-25 at 14:22 +, Peter Kjellerstedt wrote:
> > > > -Original Message-
> > > > From: Richard Purdie 
> > > > Sent: den 25 mars 2021 10:34
> > > > To: Oleksiy Obitotskyi -X (oobitots - GLOBALLOGIC INC at Cisco)
> > > > ; Luca Bocassi ;
> > > > openembedded-core@lists.openembedded.org
> > > > Cc: bluelightn...@bluelightning.org; Peter Kjellerstedt
> > > > ; Khem Raj 
> > > > Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> > > > recipe to allow bootstrapping
> > > > 
> > > > On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots -
> > > > GLOBALLOGIC INC at Cisco) wrote:
> > > > > Could you look into this warning.
> > > > > 
> > > > > WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-
> > dev
> > > > rdepends on util-linux-libuuid-dev, but it isn't a build dependency?
> > > > [build-deps]
> > > > > 
> > > > > 
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226
> > > > 
> > > > That failure was my fault when testing some fixes.
> > > > 
> > > > I've sent out a patch which renames util-linux-uuid to util-linux-
> > libuuid
> > > > and sorts out the license issue Peter reported.
> > > 
> > > I don't mind the recipe being renamed and cleaned up, but I would prefer
> > > to see my entire patch for the license parts being either integrated
> > before
> > > this or squashed into it, whichever you prefer. It does not make sense
> > to
> > > use the same LIC_FILES_CHKSUM for util-linux-libuuid as for util-linux,
> > > and setting the other LICENSE variables in util-linux.inc no longer
> > makes
> > > sense as they are only relevant for util-linux.
> > 
> > I'm torn on that. Code with the other licenses is present, just not used
> > in the final output and I personally suspect that having one 
> > LIC_FILES_CHKSUM
> > is going to be easier to maintain in the future rather than two separate
> > ones.
> 
> I actually checked all the files that go into -dev and -src before suggesting 
> this change, and all files are either marked as public domain or use a 
> BSD-3-Clause license.

There is a difference between what ends up in ${S} and what ends up in the 
binary packages. LICENSE clearly governs the latter. Its the scope of 
LIC_FILES_CHECKSUM which there are differences of opinion on.

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149941): 
https://lists.openembedded.org/g/openembedded-core/message/149941
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Peter Kjellerstedt
> -Original Message-
> From: Richard Purdie 
> Sent: den 25 mars 2021 15:27
> To: Peter Kjellerstedt ; Oleksiy Obitotskyi -
> X (oobitots - GLOBALLOGIC INC at Cisco) ; Luca Bocassi
> ; openembedded-core@lists.openembedded.org
> Cc: bluelightn...@bluelightning.org; Khem Raj 
> Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> recipe to allow bootstrapping
> 
> On Thu, 2021-03-25 at 14:22 +, Peter Kjellerstedt wrote:
> > > -Original Message-
> > > From: Richard Purdie 
> > > Sent: den 25 mars 2021 10:34
> > > To: Oleksiy Obitotskyi -X (oobitots - GLOBALLOGIC INC at Cisco)
> > > ; Luca Bocassi ;
> > > openembedded-core@lists.openembedded.org
> > > Cc: bluelightn...@bluelightning.org; Peter Kjellerstedt
> > > ; Khem Raj 
> > > Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> > > recipe to allow bootstrapping
> > >
> > > On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots -
> > > GLOBALLOGIC INC at Cisco) wrote:
> > > > Could you look into this warning.
> > > >
> > > > WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-
> dev
> > > rdepends on util-linux-libuuid-dev, but it isn't a build dependency?
> > > [build-deps]
> > > >
> > > >
> https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226
> > >
> > > That failure was my fault when testing some fixes.
> > >
> > > I've sent out a patch which renames util-linux-uuid to util-linux-
> libuuid
> > > and sorts out the license issue Peter reported.
> >
> > I don't mind the recipe being renamed and cleaned up, but I would prefer
> > to see my entire patch for the license parts being either integrated
> before
> > this or squashed into it, whichever you prefer. It does not make sense
> to
> > use the same LIC_FILES_CHKSUM for util-linux-libuuid as for util-linux,
> > and setting the other LICENSE variables in util-linux.inc no longer
> makes
> > sense as they are only relevant for util-linux.
> 
> I'm torn on that. Code with the other licenses is present, just not used
> in the final output and I personally suspect that having one LIC_FILES_CHKSUM
> is going to be easier to maintain in the future rather than two separate
> ones.

I actually checked all the files that go into -dev and -src before suggesting 
this change, and all files are either marked as public domain or use a 
BSD-3-Clause license.

> Cheers,
> 
> Richard

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149940): 
https://lists.openembedded.org/g/openembedded-core/message/149940
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] image,populate_sdk_base: move 'func' flag setting for sdk command vars

2021-03-25 Thread Christopher Larson
Setting the 'func' flag on the commands variables ensures that they are parsed
as shell, and therefore that the referenced commands contents are included in
checksums. Doing this only in image.bbclass means that this is missing in
recipes that are not images, but which inherit populate_sdk or populate_sdk_base
directly, so move it to the latter.

[YOCTO #13998]

Signed-off-by: Christopher Larson 
---
 meta/classes/image.bbclass | 2 +-
 meta/classes/populate_sdk_base.bbclass | 7 +++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index b558653635..c36c7a9695 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -112,7 +112,7 @@ def rootfs_command_variables(d):
 
'IMAGE_PREPROCESS_COMMAND','RPM_PREPROCESS_COMMANDS','RPM_POSTPROCESS_COMMANDS','DEB_PREPROCESS_COMMANDS','DEB_POSTPROCESS_COMMANDS']
 
 python () {
-variables = rootfs_command_variables(d) + sdk_command_variables(d)
+variables = rootfs_command_variables(d)
 for var in variables:
 if d.getVar(var, False):
 d.setVarFlag(var, 'func', '1')
diff --git a/meta/classes/populate_sdk_base.bbclass 
b/meta/classes/populate_sdk_base.bbclass
index c8a7084d33..33ba3fc3c1 100644
--- a/meta/classes/populate_sdk_base.bbclass
+++ b/meta/classes/populate_sdk_base.bbclass
@@ -329,6 +329,13 @@ def sdk_variables(d):
 
 do_populate_sdk[vardeps] += "${@sdk_variables(d)}"
 
+python () {
+variables = sdk_command_variables(d)
+for var in variables:
+if d.getVar(var, False):
+d.setVarFlag(var, 'func', '1')
+}
+
 do_populate_sdk[file-checksums] += "${TOOLCHAIN_SHAR_REL_TMPL}:True \
 ${TOOLCHAIN_SHAR_EXT_TMPL}:True"
 
-- 
2.28.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149939): 
https://lists.openembedded.org/g/openembedded-core/message/149939
Mute This Topic: https://lists.openembedded.org/mt/81606656/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] any *compelling* reasons to whitelist env vars for an OE build?

2021-03-25 Thread Christopher Larson
Just a quick note that you *can* script config file alterations without
having to alter files that may be under source control, that's what
auto.conf is for :)

On Thu, Mar 25, 2021 at 7:51 AM Robert P. J. Day 
wrote:

> On Thu, 25 Mar 2021, Mike Looijmans wrote:
>
> > I for one use a whitelist env vars daily, both hobby and work.
> >
> > We tend to build for multiple machines, so in general we'd have
> > something like this running on the build server:
> >
> > for machine in mach1 mach2 mach3 mach4 ... mach40
> > do
> >MACHINE=$machine bitbake image1 image2 image3
> > done
> >
> > Being able to pass the MACHINE through the environment is a big win. We
> could
> > alter a config file in the shell script, but that would change a file
> that
> > we'd want under version control, and I really don't like it when builds
> change
> > files under version control.
> >
> > (The "40" machines is not exaggerated, I'm really involved in projects
> that
> > build for that many targets)
> >
> > Come to think of it, MACHINE is the one and only environment we ever
> pass to
> > bitbake.
> >
> > For release/test/production/debug/... etcetera I tend to use a different
> > image, so you'd see:
> >
> > MACHINE=mach1 bitbake interesting-image interesting-image-dev
>
>   you and mark are right, i was being a bit too draconian -- being
> able to select MACHINE and DISTRO on the bitbake invocation line are
> obvious benefits.
>
>   just to refresh my memory, what is it that identifies the env vars
> that are by default passed through without any need for extra
> whitelisting? is that BB_ENV_EXTRAWHITE? i recall that contains
> MACHINE and DISTRO already, so that might be all i care about.
>
> rday
>
> 
>
>

-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149938): 
https://lists.openembedded.org/g/openembedded-core/message/149938
Mute This Topic: https://lists.openembedded.org/mt/81603196/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Khem Raj
On Thu, Mar 25, 2021 at 8:45 AM Luca Boccassi  wrote:
>
> On Thu, 2021-03-25 at 14:27 +, Richard Purdie wrote:
> > On Thu, 2021-03-25 at 14:22 +, Peter Kjellerstedt wrote:
> > > > -Original Message-
> > > > From: Richard Purdie 
> > > > Sent: den 25 mars 2021 10:34
> > > > To: Oleksiy Obitotskyi -X (oobitots - GLOBALLOGIC INC at Cisco)
> > > > ; Luca Bocassi ;
> > > > openembedded-core@lists.openembedded.org
> > > > Cc: bluelightn...@bluelightning.org; Peter Kjellerstedt
> > > > ; Khem Raj 
> > > > Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> > > > recipe to allow bootstrapping
> > > >
> > > > On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots -
> > > > GLOBALLOGIC INC at Cisco) wrote:
> > > > > Could you look into this warning.
> > > > >
> > > > > WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-dev
> > > > rdepends on util-linux-libuuid-dev, but it isn't a build dependency?
> > > > [build-deps]
> > > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226
> > > >
> > > > That failure was my fault when testing some fixes.
> > > >
> > > > I've sent out a patch which renames util-linux-uuid to 
> > > > util-linux-libuuid
> > > > and sorts out the license issue Peter reported.
> > >
> > > I don't mind the recipe being renamed and cleaned up, but I would prefer
> > > to see my entire patch for the license parts being either integrated 
> > > before
> > > this or squashed into it, whichever you prefer. It does not make sense to
> > > use the same LIC_FILES_CHKSUM for util-linux-libuuid as for util-linux,
> > > and setting the other LICENSE variables in util-linux.inc no longer makes
> > > sense as they are only relevant for util-linux.
> >
> > I'm torn on that. Code with the other licenses is present, just not used
> > in the final output and I personally suspect that having one 
> > LIC_FILES_CHKSUM
> > is going to be easier to maintain in the future rather than two separate 
> > ones.
>
> FWIW, in Debian the license definitions always cover the _sources_, not
> the built binaries. IOW: even if you don't build and distribute a
> subtool, the license metadata must cover it. I'd think this would be
> even more important for Yocto since you exclusively distribute sources,
> not binaries.

ideally, it would be good to have both expressed distinctly since we also have
tooling for end users to create manifests for whats strictly on target
images. we perhaps
need source license which is perhaps at recipe level and then target
license which is at package level.

>
> --
> Kind regards,
> Luca Boccassi
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149937): 
https://lists.openembedded.org/g/openembedded-core/message/149937
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Luca Bocassi
On Thu, 2021-03-25 at 14:27 +, Richard Purdie wrote:
> On Thu, 2021-03-25 at 14:22 +, Peter Kjellerstedt wrote:
> > > -Original Message-
> > > From: Richard Purdie 
> > > Sent: den 25 mars 2021 10:34
> > > To: Oleksiy Obitotskyi -X (oobitots - GLOBALLOGIC INC at Cisco)
> > > ; Luca Bocassi ;
> > > openembedded-core@lists.openembedded.org
> > > Cc: bluelightn...@bluelightning.org; Peter Kjellerstedt
> > > ; Khem Raj 
> > > Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> > > recipe to allow bootstrapping
> > > 
> > > On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots -
> > > GLOBALLOGIC INC at Cisco) wrote:
> > > > Could you look into this warning.
> > > > 
> > > > WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-dev
> > > rdepends on util-linux-libuuid-dev, but it isn't a build dependency?
> > > [build-deps]
> > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226
> > > 
> > > That failure was my fault when testing some fixes.
> > > 
> > > I've sent out a patch which renames util-linux-uuid to util-linux-libuuid
> > > and sorts out the license issue Peter reported.
> > 
> > I don't mind the recipe being renamed and cleaned up, but I would prefer 
> > to see my entire patch for the license parts being either integrated before 
> > this or squashed into it, whichever you prefer. It does not make sense to 
> > use the same LIC_FILES_CHKSUM for util-linux-libuuid as for util-linux, 
> > and setting the other LICENSE variables in util-linux.inc no longer makes 
> > sense as they are only relevant for util-linux.
> 
> I'm torn on that. Code with the other licenses is present, just not used
> in the final output and I personally suspect that having one LIC_FILES_CHKSUM 
> is going to be easier to maintain in the future rather than two separate ones.

FWIW, in Debian the license definitions always cover the _sources_, not
the built binaries. IOW: even if you don't build and distribute a
subtool, the license metadata must cover it. I'd think this would be
even more important for Yocto since you exclusively distribute sources,
not binaries.

-- 
Kind regards,
Luca Boccassi


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149936): 
https://lists.openembedded.org/g/openembedded-core/message/149936
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [meta-oe][PATCH] bpftool: remove recipe from blacklist

2021-03-25 Thread Martin Jansa
Needs to go to openembedded-devel ML and add bpftool to
meta-oe/recipes-core/packagegroups/packagegroup-meta-oe.bb

On Thu, Mar 25, 2021 at 4:26 PM Denys Zagorui -X (dzagorui - GLOBALLOGIC
INC@Cisco) via lists.openembedded.org  wrote:

> Have not seen build issues for 5.4 and 5.10
> kernels so allow to build it
>
> Signed-off-by: Denys Zagorui 
> ---
>  meta-oe/recipes-kernel/bpftool/bpftool.bb | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/meta-oe/recipes-kernel/bpftool/bpftool.bb
> b/meta-oe/recipes-kernel/bpftool/bpftool.bb
> index 1758430bc..e58bdd493 100644
> --- a/meta-oe/recipes-kernel/bpftool/bpftool.bb
> +++ b/meta-oe/recipes-kernel/bpftool/bpftool.bb
> @@ -32,4 +32,3 @@ python do_package_prepend() {
>  }
>
>  B = "${WORKDIR}/${BPN}-${PV}"
> -PNBLACKLIST[bpftool] ?= "Needs forward porting to kernel 5.2+"
> --
> 2.26.2.Cisco
>
>
> 
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149935): 
https://lists.openembedded.org/g/openembedded-core/message/149935
Mute This Topic: https://lists.openembedded.org/mt/81605266/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [meta-oe][PATCH] bpftool: remove recipe from blacklist

2021-03-25 Thread Denys Zagorui -X (dzagorui - GLOBALLOGIC INC@Cisco) via lists.openembedded.org
Have not seen build issues for 5.4 and 5.10
kernels so allow to build it

Signed-off-by: Denys Zagorui 
---
 meta-oe/recipes-kernel/bpftool/bpftool.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta-oe/recipes-kernel/bpftool/bpftool.bb 
b/meta-oe/recipes-kernel/bpftool/bpftool.bb
index 1758430bc..e58bdd493 100644
--- a/meta-oe/recipes-kernel/bpftool/bpftool.bb
+++ b/meta-oe/recipes-kernel/bpftool/bpftool.bb
@@ -32,4 +32,3 @@ python do_package_prepend() {
 }
 
 B = "${WORKDIR}/${BPN}-${PV}"
-PNBLACKLIST[bpftool] ?= "Needs forward porting to kernel 5.2+"
-- 
2.26.2.Cisco


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149934): 
https://lists.openembedded.org/g/openembedded-core/message/149934
Mute This Topic: https://lists.openembedded.org/mt/81605266/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe-core][PATCH 1/2] valgrind: update 3.16.1 -> 3.17.0

2021-03-25 Thread Oleksiy Obitotskyy via lists.openembedded.org
Here is build and link to the diff-html:

https://autobuilder.yoctoproject.org/typhoon/#/builders/116/builds/126
https://autobuilder.yocto.io/pub/repro-fail/oe-reproducible-20210324-x8y0w43u/packages/diff-html/

Regards,
Oleksiy


From: openembedded-core@lists.openembedded.org 
 on behalf of Yi Fan Yu 

Sent: Tuesday, March 23, 2021 22:50
To: openembedded-core@lists.openembedded.org
Subject: [oe-core][PATCH 1/2] valgrind: update 3.16.1 -> 3.17.0

Notable changes:
* libdir is now libexecdir

Added patches:
Add musl.supp: missing musl.supp in 3.17.0

Dropped backport patches:
* nlcontrolc: found in c79180a3afcf65902e578646c3b716cc749db406
* drd Fedora33: found in 15330adf7c2471fbaa6a0818db07078d81dbff97

Other dropped patches
* helgrind intercept: found in d2d54dbcc74244adfc0c80b40862edf2b82f53b9
* drd musl fix: found in d2d54dbcc74244adfc0c80b40862edf2b82f53b9

TESTING RESULTS:
qemux86-64:
FAIL: drd/tests/swapcontext

  3.17.0  3.16.1
===
TOTAL:  736726
PASSED: 694688
FAILED:   1  0
SKIPPED: 41 38

Signed-off-by: Yi Fan Yu 
---
 .../valgrind/0001-Add-missing-musl.supp.patch |  72 +++
 ...gtest-hanging-on-newer-glibc-and-or-.patch | 200 --
 .../valgrind/0001-drd-Port-to-Fedora-33.patch |  48 -
 .../valgrind/valgrind/0001-drd-musl-fix.patch |  31 ---
 ...01-helgrind-Intercept-libc-functions.patch |  54 -
 ...ated-PowerPC-insns-aren-t-allowed-on.patch |  62 --
 ...eak_cmsg.stderr.exp-adjust-tmp-paths.patch |   2 +-
 ...ntext-APIs-are-not-available-on-musl.patch |  50 -
 .../valgrind/valgrind/fixed-perl-path.patch   |  20 +-
 .../valgrind/valgrind/run-ptest   |   3 +-
 .../valgrind/valgrind/s390x_vec_op_t.patch|  24 ++-
 ...{valgrind_3.16.1.bb => valgrind_3.17.0.bb} |  20 +-
 12 files changed, 160 insertions(+), 426 deletions(-)
 create mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-drd-Port-to-Fedora-33.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-drd-musl-fix.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-helgrind-Intercept-libc-functions.patch
 delete mode 100644 
meta/recipes-devtools/valgrind/valgrind/0001-lmw-lswi-and-related-PowerPC-insns-aren-t-allowed-on.patch
 rename meta/recipes-devtools/valgrind/{valgrind_3.16.1.bb => 
valgrind_3.17.0.bb} (93%)

diff --git 
a/meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch 
b/meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch
new file mode 100644
index 00..2a73f7e81f
--- /dev/null
+++ b/meta/recipes-devtools/valgrind/valgrind/0001-Add-missing-musl.supp.patch
@@ -0,0 +1,72 @@
+From 61bc8664f93cd980831c9da4a3e8a385b089a0ab Mon Sep 17 00:00:00 2001
+From: Yi Fan Yu 
+Date: Tue, 23 Mar 2021 09:32:22 -0700
+Subject: [PATCH] Add missing musl.supp
+
+3.17 did not ship musl.supp in the tarball.
+
+This is a workaround until next release.
+
+Upstream-Status: Backport [dde556d51f8226a6de564a00bf82536bb7042c54]
+
+Signed-off-by: Yi Fan Yu 
+---
+ musl.supp | 46 ++
+ 1 file changed, 46 insertions(+)
+ create mode 100644 musl.supp
+
+diff --git a/musl.supp b/musl.supp
+new file mode 100644
+index 0..864172a24
+--- /dev/null
 b/musl.supp
+@@ -0,0 +1,46 @@
++# Suppressions for musl libc
++# See: https://www.openwall.com/lists/musl/2017/06/15/4
++
++{
++   musl-dynlink-false-positive1
++   Memcheck:Leak
++   fun:calloc
++   fun:load_direct_deps
++   fun:load_deps
++   fun:load_deps
++   fun:__dls3
++   fun:__dls2b
++   fun:__dls2
++}
++
++{
++   musl-dynlink-false-positive2
++   Memcheck:Leak
++   fun:calloc
++   fun:load_direct_deps
++   fun:load_deps
++   fun:load_deps
++   fun:__dls3
++   fun:__dls2
++}
++
++{
++   musl-dynlink-false-positive3
++   Memcheck:Leak
++   fun:calloc
++   fun:load_library
++   fun:load_preload
++   fun:__dls3
++   fun:__dls2b
++   fun:__dls2
++}
++
++{
++   musl-dynlink-false-positive4
++   Memcheck:Leak
++   fun:calloc
++   fun:load_library
++   fun:load_preload
++   fun:__dls3
++   fun:__dls2
++}
+--
+2.17.1
+
diff --git 
a/meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
 
b/meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
deleted file mode 100644
index 98cbcd132c..00
--- 
a/meta/recipes-devtools/valgrind/valgrind/0001-Fix-nlcontrolc.vgtest-hanging-on-newer-glibc-and-or-.patch
+++ /dev/null
@@ -1,200 +0,0 @@
-From 83c24e31df6932a6d4fced179050c6d8d8c6f3b5 Mon Sep 17 00:00:00 2001
-From: Philippe Waroquiers 
-Date: Sun, 7 Mar 2021 22:29:27 +0100
-Subject: [PATCH] Fix nlcontrolc.vgtest hanging on newer glibc and/or arm64
-

Re: [OE-core] any *compelling* reasons to whitelist env vars for an OE build?

2021-03-25 Thread Robert P. J. Day
On Thu, 25 Mar 2021, Mike Looijmans wrote:

> I for one use a whitelist env vars daily, both hobby and work.
>
> We tend to build for multiple machines, so in general we'd have
> something like this running on the build server:
>
> for machine in mach1 mach2 mach3 mach4 ... mach40
> do
>MACHINE=$machine bitbake image1 image2 image3
> done
>
> Being able to pass the MACHINE through the environment is a big win. We could
> alter a config file in the shell script, but that would change a file that
> we'd want under version control, and I really don't like it when builds change
> files under version control.
>
> (The "40" machines is not exaggerated, I'm really involved in projects that
> build for that many targets)
>
> Come to think of it, MACHINE is the one and only environment we ever pass to
> bitbake.
>
> For release/test/production/debug/... etcetera I tend to use a different
> image, so you'd see:
>
> MACHINE=mach1 bitbake interesting-image interesting-image-dev

  you and mark are right, i was being a bit too draconian -- being
able to select MACHINE and DISTRO on the bitbake invocation line are
obvious benefits.

  just to refresh my memory, what is it that identifies the env vars
that are by default passed through without any need for extra
whitelisting? is that BB_ENV_EXTRAWHITE? i recall that contains
MACHINE and DISTRO already, so that might be all i care about.

rday

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149932): 
https://lists.openembedded.org/g/openembedded-core/message/149932
Mute This Topic: https://lists.openembedded.org/mt/81603196/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] any *compelling* reasons to whitelist env vars for an OE build?

2021-03-25 Thread Mike Looijmans

I for one use a whitelist env vars daily, both hobby and work.

We tend to build for multiple machines, so in general we'd have something like 
this running on the build server:


for machine in mach1 mach2 mach3 mach4 ... mach40
do
   MACHINE=$machine bitbake image1 image2 image3
done

Being able to pass the MACHINE through the environment is a big win. We could 
alter a config file in the shell script, but that would change a file that 
we'd want under version control, and I really don't like it when builds change 
files under version control.


(The "40" machines is not exaggerated, I'm really involved in projects that 
build for that many targets)


Come to think of it, MACHINE is the one and only environment we ever pass to 
bitbake.


For release/test/production/debug/... etcetera I tend to use a different 
image, so you'd see:


MACHINE=mach1 bitbake interesting-image interesting-image-dev

So "interesting-image" would be the product, while the "dev" version would 
apply debug tweaks that make the rootfs writeable and enable the serial 
console for login and things like that.





Met vriendelijke groet / kind regards,

Mike Looijmans
System Expert


TOPIC Embedded Products B.V.
Materiaalweg 4, 5681 RJ Best
The Netherlands

T: +31 (0) 499 33 69 69
E: mike.looijm...@topicproducts.com
W: www.topic.nl

Please consider the environment before printing this e-mail
On 25-03-2021 15:08, Robert P. J. Day via lists.openembedded.org wrote:


   kind of a philosophical question, but i'm having a discussion with
some new colleagues about how to customize an OE (actually, wind river
linux) build, and these colleagues have, until now, used (whitelisted)
environment variables to pass info to the build, that info being used
to specify things like a development versus manufacturing build, and
so on.

   i suggested that i didn't think there was any really persuasive
reasons to use environment variables this way, as there are more than
enough configuration files to customize a build. i mentioned things
like:

   * machine config files
   * distro config files
   * defining packagegroup files
   * defining image files

and on and on and on.

   my point was that there are more than enough ways to support all the
customization you might need, that taking advantage of shell
environment variables strikes me as unnecessarily messy and, for that
matter, dangerous if you forget so set the environment.

   thoughts? am i out of line in advising them to use what OE offers
them, and not extract stuff from the environment?

rday








-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149931): 
https://lists.openembedded.org/g/openembedded-core/message/149931
Mute This Topic: https://lists.openembedded.org/mt/81603196/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Richard Purdie
On Thu, 2021-03-25 at 14:22 +, Peter Kjellerstedt wrote:
> > -Original Message-
> > From: Richard Purdie 
> > Sent: den 25 mars 2021 10:34
> > To: Oleksiy Obitotskyi -X (oobitots - GLOBALLOGIC INC at Cisco)
> > ; Luca Bocassi ;
> > openembedded-core@lists.openembedded.org
> > Cc: bluelightn...@bluelightning.org; Peter Kjellerstedt
> > ; Khem Raj 
> > Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> > recipe to allow bootstrapping
> > 
> > On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots -
> > GLOBALLOGIC INC at Cisco) wrote:
> > > Could you look into this warning.
> > > 
> > > WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-dev
> > rdepends on util-linux-libuuid-dev, but it isn't a build dependency?
> > [build-deps]
> > > 
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226
> > 
> > That failure was my fault when testing some fixes.
> > 
> > I've sent out a patch which renames util-linux-uuid to util-linux-libuuid
> > and sorts out the license issue Peter reported.
> 
> I don't mind the recipe being renamed and cleaned up, but I would prefer 
> to see my entire patch for the license parts being either integrated before 
> this or squashed into it, whichever you prefer. It does not make sense to 
> use the same LIC_FILES_CHKSUM for util-linux-libuuid as for util-linux, 
> and setting the other LICENSE variables in util-linux.inc no longer makes 
> sense as they are only relevant for util-linux.

I'm torn on that. Code with the other licenses is present, just not used
in the final output and I personally suspect that having one LIC_FILES_CHKSUM 
is going to be easier to maintain in the future rather than two separate ones.

Cheers,

Richard




-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149930): 
https://lists.openembedded.org/g/openembedded-core/message/149930
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Peter Kjellerstedt
> -Original Message-
> From: Richard Purdie 
> Sent: den 25 mars 2021 10:34
> To: Oleksiy Obitotskyi -X (oobitots - GLOBALLOGIC INC at Cisco)
> ; Luca Bocassi ;
> openembedded-core@lists.openembedded.org
> Cc: bluelightn...@bluelightning.org; Peter Kjellerstedt
> ; Khem Raj 
> Subject: Re: [OE-core] [PATCH v11] util-linux: split uuid in separate
> recipe to allow bootstrapping
> 
> On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots -
> GLOBALLOGIC INC at Cisco) wrote:
> > Could you look into this warning.
> >
> > WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-dev
> rdepends on util-linux-libuuid-dev, but it isn't a build dependency?
> [build-deps]
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226
> 
> That failure was my fault when testing some fixes.
> 
> I've sent out a patch which renames util-linux-uuid to util-linux-libuuid
> and sorts out the license issue Peter reported.

I don't mind the recipe being renamed and cleaned up, but I would prefer 
to see my entire patch for the license parts being either integrated before 
this or squashed into it, whichever you prefer. It does not make sense to 
use the same LIC_FILES_CHKSUM for util-linux-libuuid as for util-linux, 
and setting the other LICENSE variables in util-linux.inc no longer makes 
sense as they are only relevant for util-linux.

> I'm optimistic this fixes the various issues people have been having
> and simplifies the recipe a bit as an added bonus.
> 
> The patch has already tested cleanly on the autobuilder (the above warning
> was from an earlier aborted build before I opted to rename the recipe).
> 
> Khem: Thanks for the related meta-oe tweak, I think the rename whilst
> slightly painful now is the best way forward overall.
> 
> Cheers,
> 
> Richard

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149929): 
https://lists.openembedded.org/g/openembedded-core/message/149929
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] any *compelling* reasons to whitelist env vars for an OE build?

2021-03-25 Thread Mark Hatle


On 3/25/21 9:08 AM, Robert P. J. Day wrote:
> 
>   kind of a philosophical question, but i'm having a discussion with
> some new colleagues about how to customize an OE (actually, wind river
> linux) build, and these colleagues have, until now, used (whitelisted)
> environment variables to pass info to the build, that info being used
> to specify things like a development versus manufacturing build, and
> so on.
> 
>   i suggested that i didn't think there was any really persuasive
> reasons to use environment variables this way, as there are more than
> enough configuration files to customize a build. i mentioned things
> like:
> 
>   * machine config files
>   * distro config files
>   * defining packagegroup files
>   * defining image files
> 
> and on and on and on.
> 
>   my point was that there are more than enough ways to support all the
> customization you might need, that taking advantage of shell
> environment variables strikes me as unnecessarily messy and, for that
> matter, dangerous if you forget so set the environment.
> 
>   thoughts? am i out of line in advising them to use what OE offers
> them, and not extract stuff from the environment?

I _always_ do builds like:

MACHINE=zynqmp-generic DISTRO=petalinux bitbake core-image-minimal

I don't want to have to update any configuration files to pass basic values into
the system.  This is also used heavily on my build systems so we can have a
common configuration for all builds, but built across a wide breadth of 
components.

I also occasionally use BB_ENV_EXTRAWHITE to add additional variables I might
want to override.  For instance, sometimes I want TMPDIR to go to a different
location, instead of having to modify local.conf (and then remember to put it
back the way it was) I can just add to the EXTRAWHITE, and call passing in 
TMPDIR.

--Mark

> rday
> 
> 
> 
> 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149928): 
https://lists.openembedded.org/g/openembedded-core/message/149928
Mute This Topic: https://lists.openembedded.org/mt/81603196/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] any *compelling* reasons to whitelist env vars for an OE build?

2021-03-25 Thread Robert P. J. Day

  kind of a philosophical question, but i'm having a discussion with
some new colleagues about how to customize an OE (actually, wind river
linux) build, and these colleagues have, until now, used (whitelisted)
environment variables to pass info to the build, that info being used
to specify things like a development versus manufacturing build, and
so on.

  i suggested that i didn't think there was any really persuasive
reasons to use environment variables this way, as there are more than
enough configuration files to customize a build. i mentioned things
like:

  * machine config files
  * distro config files
  * defining packagegroup files
  * defining image files

and on and on and on.

  my point was that there are more than enough ways to support all the
customization you might need, that taking advantage of shell
environment variables strikes me as unnecessarily messy and, for that
matter, dangerous if you forget so set the environment.

  thoughts? am i out of line in advising them to use what OE offers
them, and not extract stuff from the environment?

rday

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149927): 
https://lists.openembedded.org/g/openembedded-core/message/149927
Mute This Topic: https://lists.openembedded.org/mt/81603196/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] [RFC] ffmpeg: make LICENSE_FLAGS fine-grained

2021-03-25 Thread Yann Dirson
Hi Alex,

Le mar. 23 mars 2021 à 17:42, Alexander Kanavin
 a écrit :
>
> Is it possible to further split the NONCOMMERCIAL_PACKAGECONFIGS into ones 
> that are enabled and disabled, and use the former in PACKAGECONFIG itself?

You mean something like this ?

# PACKAGECONFIG features that do not pull codecs
NONCOMMERCIAL_DEFAULT_PACKAGECONFIGS = " \
 avdevice avfilter avformat swresample swscale postproc avresample \
 alsa bzlib drm gpl lzma pic pthreads shared zlib \
 ${@bb.utils.contains('AVAILTUNES', 'mips32r2', 'mips32r2', '', d)} \
 ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'xv xcb', '', d)} \
"
NONCOMMERCIAL_NONDEFAULT_PACKAGECONFIGS = " \
 altivec bzlib gsm jack openssl sdl2 \
"

PACKAGECONFIG ??= "avcodec theora \
   ${NONCOMMERCIAL_DEFAULT_PACKAGECONFIGS}"

I admit I'm not completely sure about every feature's status.

I tend to think that if ffmpeg relies on a lib providing an
implementation, and there
are patent-related risks, it should that lib setting a LICENSE_FLAG.
Eg. x264 and others
already sets the "commercial" flag, so I'm tempted to not add a
duplicate flag in
ffmpeg (in the same spirit as my patches against gst-libav and mpv).
Would it be
good enough ?

>
>
> Alex
>
> On Tue, 23 Mar 2021 at 17:38, Yann Dirson  wrote:
>>
>> From: Yann Dirson 
>>
>> The rationale here is that if the user can only whitelist "commercial"
>> to use any part of ffmpeg, even it the list of features is carefully
>> reviewed when switching the whitelisting on, there was nothing to
>> guard from inadvertently activating a new feature that would not have
>> been reviewed.
>>
>> This patch adds one LICENSE_FLAGS value for each feature, except for
>> those that bring no codec, trying to be on the same level of legal
>> safety - but then I may miss something.
>>
>> I tried to leave out of the safe NONCOMMERCIAL_PACKAGECONFIGS list
>> anything that brings a codec, notably libavcodec.  I also did not look
>> at non-default features yet.
>>
>> There may still be a problem if any feature in ffmpeg gets activated
>> by default upstream and not registed as a PACKAGECONFIG feature.  At
>> least any of those that depend on another lib would not be enabled,
>> that could be seen as a sufficient safeguard.
>> ---
>>  meta/recipes-multimedia/ffmpeg/ffmpeg_4.3.2.bb | 12 +++-
>>  1 file changed, 11 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-multimedia/ffmpeg/ffmpeg_4.3.2.bb 
>> b/meta/recipes-multimedia/ffmpeg/ffmpeg_4.3.2.bb
>> index 08be38ca50..3a36c95151 100644
>> --- a/meta/recipes-multimedia/ffmpeg/ffmpeg_4.3.2.bb
>> +++ b/meta/recipes-multimedia/ffmpeg/ffmpeg_4.3.2.bb
>> @@ -16,7 +16,17 @@ LICENSE_libavutil = 
>> "${@bb.utils.contains('PACKAGECONFIG', 'gpl', 'GPLv2+', 'LGP
>>  LICENSE_libpostproc = "GPLv2+"
>>  LICENSE_libswresample = "${@bb.utils.contains('PACKAGECONFIG', 'gpl', 
>> 'GPLv2+', 'LGPLv2.1+', d)}"
>>  LICENSE_libswscale = "${@bb.utils.contains('PACKAGECONFIG', 'gpl', 
>> 'GPLv2+', 'LGPLv2.1+', d)}"
>> -LICENSE_FLAGS = "commercial"
>> +
>> +# PACKAGECONFIG features that do not pull codecs
>> +NONCOMMERCIAL_PACKAGECONFIGS = " \
>> + alsa bzlib drm gpl lzma zlib xcb xv \
>> + avdevice avfilter avformat swresample swscale postproc avresample \
>> +"
>> +# An ffmpeg feature not in NONCOMMERCIAL_PACKAGECONFIGS should be 
>> explicitly whitelisted.
>> +# See https://ffmpeg.org/legal.html
>> +LICENSE_FLAGS = "${@' '.join('commercial_' + cfg \
>> + for cfg in '${PACKAGECONFIG}'.split() \
>> + if cfg not in 
>> '${NONCOMMERCIAL_PACKAGECONFIGS}'.split())}"
>>
>>  LIC_FILES_CHKSUM = 
>> "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>>  
>> file://COPYING.GPLv3;md5=d32239bcb673463ab874e80d47fae504 \
>> --
>> 2.30.2
>>
>>
>> 
>>


-- 
Yann Dirson 
Blade / Shadow -- http://shadow.tech

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149926): 
https://lists.openembedded.org/g/openembedded-core/message/149926
Mute This Topic: https://lists.openembedded.org/mt/81555357/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2] rootfs.py: mask run-postinsts systemd service when unneeded

2021-03-25 Thread Awais Belal
[Edited Message Follows]

Hi Oleksiy,

I've submitted 
https://lists.openembedded.org/g/openembedded-core/topic/81600237 which should 
supersede this I did not send it as v3 because it is a different solution. I've 
also ran 'bitbake -c testimage core-image-sato' with the new solution and it is 
passing.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149925): 
https://lists.openembedded.org/g/openembedded-core/message/149925
Mute This Topic: https://lists.openembedded.org/mt/81522036/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v2] rootfs.py: mask run-postinsts systemd service when unneeded

2021-03-25 Thread Awais Belal
Hi Oleksiy,

I've submitted 
https://lists.openembedded.org/g/openembedded-core/topic/81600237 which should 
supersede this I did not send it as v3 because it is a different solution.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149925): 
https://lists.openembedded.org/g/openembedded-core/message/149925
Mute This Topic: https://lists.openembedded.org/mt/81522036/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] rootfs.py: uninstall the run-postinsts package if not needed

2021-03-25 Thread Awais Belal
The run-postinsts package runs post installation scripts
on target if packages request delayed post installations. When
no delayed post installations are found the sysV style scripts
are disabled for the package and hence it did not run on sysV
based systems. However, the package provides systemd service
as well which still ran on systems based on systemd even when
no post installations were found.
Rather than disabling/masking scripts for different initialization
managers we now simply remove/uninstall the run-postinsts package
when no post installations are found to be delayed till runtime.
This is also more aligned with the function (_uninstall_unneeded)
this functionality is triggered through.

Signed-off-by: Awais Belal 
---
 meta/lib/oe/rootfs.py | 8 +++-
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py
index 249c685dcf..5f81023040 100644
--- a/meta/lib/oe/rootfs.py
+++ b/meta/lib/oe/rootfs.py
@@ -250,13 +250,11 @@ class Rootfs(object, metaclass=ABCMeta):
 
 
 def _uninstall_unneeded(self):
-# Remove unneeded init script symlinks
+# Remove the run-postinsts package if no delayed postinsts are found
 delayed_postinsts = self._get_delayed_postinsts()
 if delayed_postinsts is None:
-if 
os.path.exists(self.d.expand("${IMAGE_ROOTFS}${sysconfdir}/init.d/run-postinsts")):
-self._exec_shell_cmd(["update-rc.d", "-f", "-r",
-  self.d.getVar('IMAGE_ROOTFS'),
-  "run-postinsts", "remove"])
+if 
os.path.exists(self.d.expand("${IMAGE_ROOTFS}${sysconfdir}/init.d/run-postinsts"))
 or 
os.path.exists(self.d.expand("${IMAGE_ROOTFS}${systemd_unitdir}/system/run-postinsts.service")):
+self.pm.remove(["run-postinsts"])
 
 image_rorfs = bb.utils.contains("IMAGE_FEATURES", "read-only-rootfs",
 True, False, self.d)
-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149924): 
https://lists.openembedded.org/g/openembedded-core/message/149924
Mute This Topic: https://lists.openembedded.org/mt/81600237/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] openssh: Wait for local-fs before generating host keys

2021-03-25 Thread Mattias Hansson
Currently, the sshdgenkeys.service wait for the host keys default paths
/var and /run. The service runs a script which picks up the host key
paths from the sshd_config which means that the paths are configurable
and can potentially be placed outside /var and /run.

This patch makes sshdgenkeys.service wait for /etc/fstab to be mounted
and which should include alternative host keys paths.

The requirement for /var and /run is left for backwards compatibility.

Signed-off-by: Mattias Hansson 
---
 meta/recipes-connectivity/openssh/openssh/sshdgenkeys.service | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-connectivity/openssh/openssh/sshdgenkeys.service 
b/meta/recipes-connectivity/openssh/openssh/sshdgenkeys.service
index fd81793d51..44831c9955 100644
--- a/meta/recipes-connectivity/openssh/openssh/sshdgenkeys.service
+++ b/meta/recipes-connectivity/openssh/openssh/sshdgenkeys.service
@@ -1,6 +1,8 @@
 [Unit]
 Description=OpenSSH Key Generation
+Requires=local-fs.target
 RequiresMountsFor=/var /run
+After=local-fs.target
 
 [Service]
 ExecStart=@LIBEXECDIR@/sshd_check_keys
-- 
2.31.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149923): 
https://lists.openembedded.org/g/openembedded-core/message/149923
Mute This Topic: https://lists.openembedded.org/mt/81598424/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Luca Bocassi
On Thu, 2021-03-25 at 09:34 +, Richard Purdie wrote:
> On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots - 
> GLOBALLOGIC INC at Cisco) wrote:
> > Could you look into this warning.
> > 
> > WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-dev 
> > rdepends on util-linux-libuuid-dev, but it isn't a build dependency? 
> > [build-deps]
> > 
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226
> 
> That failure was my fault when testing some fixes.
> 
> I've sent out a patch which renames util-linux-uuid to util-linux-libuuid
> and sorts out the license issue Peter reported.
> 
> I'm optimistic this fixes the various issues people have been having
> and simplifies the recipe a bit as an added bonus.
> 
> The patch has already tested cleanly on the autobuilder (the above warning
> was from an earlier aborted build before I opted to rename the recipe).
> 
> Khem: Thanks for the related meta-oe tweak, I think the rename whilst 
> slightly painful now is the best way forward overall.
> 
> Cheers,
> 
> Richard

Looks good to me, thanks!

-- 
Kind regards,
Luca Boccassi


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149922): 
https://lists.openembedded.org/g/openembedded-core/message/149922
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Richard Purdie
On Thu, 2021-03-25 at 09:17 +, Oleksiy Obitotskyi -X (oobitots - 
GLOBALLOGIC INC at Cisco) wrote:
> Could you look into this warning.
> 
> WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-dev 
> rdepends on util-linux-libuuid-dev, but it isn't a build dependency? 
> [build-deps]
> 
> https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226

That failure was my fault when testing some fixes.

I've sent out a patch which renames util-linux-uuid to util-linux-libuuid
and sorts out the license issue Peter reported.

I'm optimistic this fixes the various issues people have been having
and simplifies the recipe a bit as an added bonus.

The patch has already tested cleanly on the autobuilder (the above warning
was from an earlier aborted build before I opted to rename the recipe).

Khem: Thanks for the related meta-oe tweak, I think the rename whilst 
slightly painful now is the best way forward overall.

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149921): 
https://lists.openembedded.org/g/openembedded-core/message/149921
Mute This Topic: https://lists.openembedded.org/mt/81254724/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH] util-linux-libuuid: Simplify recipe and rename from util-linux-uuid

2021-03-25 Thread Richard Purdie
Rename the recipe from util-linux-uuid to util-linux-libuuid which means
we can drop the custom PACKAGES and FILES defintions which simplifies
things. Also move the LICENSE setting to the libuuid recipe so that
it is correctly applied to the right packages.

This means the standard definitions from bitbake.conf are used, avoiding
errors from situations where users have customised settings causing
failures.

Signed-off-by: Richard Purdie 
---
 meta/conf/distro/include/maintainers.inc| 2 +-
 ...il-linux-uuid_2.36.2.bb => util-linux-libuuid_2.36.2.bb} | 6 +-
 meta/recipes-core/util-linux/util-linux.inc | 1 -
 meta/recipes-core/util-linux/util-linux_2.36.2.bb   | 6 +++---
 4 files changed, 5 insertions(+), 10 deletions(-)
 rename meta/recipes-core/util-linux/{util-linux-uuid_2.36.2.bb => 
util-linux-libuuid_2.36.2.bb} (56%)

diff --git a/meta/conf/distro/include/maintainers.inc 
b/meta/conf/distro/include/maintainers.inc
index e1cdf40073c..62763318576 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -722,7 +722,7 @@ RECIPE_MAINTAINER_pn-update-rc.d = "Ross Burton 
"
 RECIPE_MAINTAINER_pn-usbinit = "Alexander Kanavin "
 RECIPE_MAINTAINER_pn-usbutils = "Alexander Kanavin "
 RECIPE_MAINTAINER_pn-util-linux = "Chen Qi "
-RECIPE_MAINTAINER_pn-util-linux-uuid = "Chen Qi "
+RECIPE_MAINTAINER_pn-util-linux-libuuid = "Chen Qi "
 RECIPE_MAINTAINER_pn-util-macros = "Armin Kuster "
 RECIPE_MAINTAINER_pn-v86d = "Alexander Kanavin "
 RECIPE_MAINTAINER_pn-vala = "Alexander Kanavin "
diff --git a/meta/recipes-core/util-linux/util-linux-uuid_2.36.2.bb 
b/meta/recipes-core/util-linux/util-linux-libuuid_2.36.2.bb
similarity index 56%
rename from meta/recipes-core/util-linux/util-linux-uuid_2.36.2.bb
rename to meta/recipes-core/util-linux/util-linux-libuuid_2.36.2.bb
index 65e4d23b7e9..9612c491cd9 100644
--- a/meta/recipes-core/util-linux/util-linux-uuid_2.36.2.bb
+++ b/meta/recipes-core/util-linux/util-linux-libuuid_2.36.2.bb
@@ -7,11 +7,7 @@ inherit autotools gettext pkgconfig
 
 S = "${WORKDIR}/util-linux-${PV}"
 EXTRA_OECONF += "--disable-all-programs --enable-libuuid"
-PACKAGES = "util-linux-libuuid util-linux-libuuid-dev 
util-linux-libuuid-staticdev util-linux-libuuid-dbg"
-FILES_util-linux-libuuid = "${libdir}/libuuid.so.*"
-FILES_util-linux-libuuid-dev = "${libdir}/libuuid.so ${includedir} 
${libdir}/pkgconfig"
-FILES_util-linux-libuuid-staticdev = "${libdir}/libuuid.a"
-FILES_util-linux-libuuid-dbg = "/usr/src ${libdir}/.debug"
+LICENSE = "BSD-3-Clause"
 
 do_install_append() {
rm -rf ${D}${datadir} ${D}${bindir} ${D}${base_bindir} ${D}${sbindir} 
${D}${base_sbindir} ${D}${exec_prefix}/sbin
diff --git a/meta/recipes-core/util-linux/util-linux.inc 
b/meta/recipes-core/util-linux/util-linux.inc
index 4031c586c91..cbf6102ee0a 100644
--- a/meta/recipes-core/util-linux/util-linux.inc
+++ b/meta/recipes-core/util-linux/util-linux.inc
@@ -11,7 +11,6 @@ LICENSE_${PN}-libblkid = "LGPLv2.1+"
 LICENSE_${PN}-libfdisk = "LGPLv2.1+"
 LICENSE_${PN}-libmount = "LGPLv2.1+"
 LICENSE_${PN}-libsmartcols = "LGPLv2.1+"
-LICENSE_${PN}-libuuid = "BSD-3-Clause"
 
 LIC_FILES_CHKSUM = 
"file://README.licensing;md5=0fd5c050c6187d2bf0a4492b7f4e33da \
 file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
diff --git a/meta/recipes-core/util-linux/util-linux_2.36.2.bb 
b/meta/recipes-core/util-linux/util-linux_2.36.2.bb
index 50a70cd4522..c79cf782d15 100644
--- a/meta/recipes-core/util-linux/util-linux_2.36.2.bb
+++ b/meta/recipes-core/util-linux/util-linux_2.36.2.bb
@@ -2,7 +2,7 @@ require util-linux.inc
 
 #gtk-doc is not enabled as it requires xmlto which requires util-linux
 inherit autotools gettext manpages pkgconfig systemd update-alternatives 
python3-dir bash-completion ptest
-DEPENDS = "libcap-ng ncurses virtual/crypt zlib util-linux-uuid"
+DEPENDS = "libcap-ng ncurses virtual/crypt zlib util-linux-libuuid"
 
 PACKAGES =+ "${PN}-swaponoff"
 PACKAGES += "${@bb.utils.contains('PACKAGECONFIG', 'pylibmount', 
'${PN}-pylibmount', '', d)}"
@@ -49,7 +49,7 @@ python util_linux_binpackages () {
 # we must execute before update-alternatives PACKAGE_PREPROCESS_FUNCS
 PACKAGE_PREPROCESS_FUNCS =+ "util_linux_binpackages "
 
-# skip libuuid as it will be packaged by the util-linux-uuid recipe
+# skip libuuid as it will be packaged by the util-linux-libuuid recipe
 python util_linux_libpackages() {
 do_split_packages(d, root=d.getVar('UTIL_LINUX_LIBDIR'), 
file_regex=r'^lib(?!uuid)(.*)\.so\..*$',
   output_pattern='${PN}-lib%s',
@@ -201,7 +201,7 @@ do_install_append_class-native () {
rm -f ${D}${base_bindir}/kill
 }
 
-# dm-verity support introduces a circular build dependency, so util-linux-uuid 
is split out for target builds
+# dm-verity support introduces a circular build dependency, so 
util-linux-libuuid is split out for target builds
 # Need to build libuuid for uuidgen, but then delete 

Re: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to allow bootstrapping

2021-03-25 Thread Oleksiy Obitotskyy via lists.openembedded.org
Could you look into this warning.

WARNING: util-linux-2.36.2-r0 do_package_qa: QA Issue: util-linux-dev rdepends 
on util-linux-libuuid-dev, but it isn't a build dependency? [build-deps]

https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3226

Regards,
Oleksiy


From: openembedded-core@lists.openembedded.org 
 on behalf of Luca Bocassi 

Sent: Thursday, March 11, 2021 17:09
To: openembedded-core@lists.openembedded.org
Cc: richard.pur...@linuxfoundation.org; bluelightn...@bluelightning.org
Subject: [OE-core] [PATCH v11] util-linux: split uuid in separate recipe to 
allow bootstrapping

From: Luca Boccassi 

Recently util-linux gained an (optional) build dependency on libcryptsetup.
But libcryptsetup build-depends on util-linux for blkid (optional, can be 
disabled)
and uuid (mandatory).
Split out util-linux-uuid in a different recipe to break the cycle.

https://github.com/karelzak/util-linux/pull/898

Signed-off-by: Luca Boccassi 
---
v1: util-linux 2.35 is not out yet, but I'd like to get the preparatory work
underway as I'm not sure if this is the best approach or if there are
alternatives. Suggestions and comments very welcome. Thanks!
v2: changed packages names to reflect old ones (eg: libuuid1 -> 
util-linux-libuuid)
and leave uuid build enable in main recipe to allow for uuidgen build to 
happen,
as it does not have its own autoconf switch. Delete the library manualy from
the main recipe after build instead, and add dependency.
Might help to break loop python3 -> util-linux -> libselinux -> python3, as 
it's
only libuuid that is needed, see 
https://lists.yoctoproject.org/g/yocto/message/47570
v3: rebased and refactored to have a common util-linux.inc file
v4: added RDEPENDS on util-linux-libuuid on various packages to fix QA warnings
v5: remove RDEPENDS and instead have util-linux RDEPEND on util-linux-uuid.
Removed PACKAGES_remove and instead filter out libuuid via the package 
generation
regex.
Rebased on util-linux 2.36.2.
v6: install libuuid.a in libuuid-dev
change rdepends on uuid to use binary package rather than source
add rdepends on uuid-dev to libuuid-dev
remove rprovides of libuuid-dev from uuid-dev
v7: do not use '_append' for RDEPENDS, as it is not supported by BBCLASSEXTEND, 
use
simply '+='
v8: added missing libuuid-staticdev package
remove libx32/libuuid* too
v9: set RECIPE_MAINTAINER_pn-util-linux-uuid, same value as 
RECIPE_MAINTAINER_pn-util-linux
v10: removed DEBIAN_NOAUTONAME
 remove lib64/libuuid* too
v11: use ${D}${base_libdir}/libuuid* instead of manually specifying 
/lib|/libx32|/lib64

 meta/conf/distro/include/maintainers.inc  |  1 +
 .../util-linux/util-linux-uuid_2.36.2.bb  | 20 +++
 meta/recipes-core/util-linux/util-linux.inc   | 41 +
 .../util-linux/util-linux_2.36.2.bb   | 57 +--
 4 files changed, 77 insertions(+), 42 deletions(-)
 create mode 100644 meta/recipes-core/util-linux/util-linux-uuid_2.36.2.bb
 create mode 100644 meta/recipes-core/util-linux/util-linux.inc

diff --git a/meta/conf/distro/include/maintainers.inc 
b/meta/conf/distro/include/maintainers.inc
index 3d8e3d5de0..c6a97bc280 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -720,6 +720,7 @@ RECIPE_MAINTAINER_pn-update-rc.d = "Ross Burton 
"
 RECIPE_MAINTAINER_pn-usbinit = "Alexander Kanavin "
 RECIPE_MAINTAINER_pn-usbutils = "Alexander Kanavin "
 RECIPE_MAINTAINER_pn-util-linux = "Chen Qi "
+RECIPE_MAINTAINER_pn-util-linux-uuid = "Chen Qi "
 RECIPE_MAINTAINER_pn-util-macros = "Armin Kuster "
 RECIPE_MAINTAINER_pn-v86d = "Alexander Kanavin "
 RECIPE_MAINTAINER_pn-vala = "Alexander Kanavin "
diff --git a/meta/recipes-core/util-linux/util-linux-uuid_2.36.2.bb 
b/meta/recipes-core/util-linux/util-linux-uuid_2.36.2.bb
new file mode 100644
index 00..65e4d23b7e
--- /dev/null
+++ b/meta/recipes-core/util-linux/util-linux-uuid_2.36.2.bb
@@ -0,0 +1,20 @@
+# To allow util-linux to optionally build-depend on cryptsetup, libuuid is
+# split out of the main recipe, as it's needed by cryptsetup
+
+require util-linux.inc
+
+inherit autotools gettext pkgconfig
+
+S = "${WORKDIR}/util-linux-${PV}"
+EXTRA_OECONF += "--disable-all-programs --enable-libuuid"
+PACKAGES = "util-linux-libuuid util-linux-libuuid-dev 
util-linux-libuuid-staticdev util-linux-libuuid-dbg"
+FILES_util-linux-libuuid = "${libdir}/libuuid.so.*"
+FILES_util-linux-libuuid-dev = "${libdir}/libuuid.so ${includedir} 
${libdir}/pkgconfig"
+FILES_util-linux-libuuid-staticdev = "${libdir}/libuuid.a"
+FILES_util-linux-libuuid-dbg = "/usr/src ${libdir}/.debug"
+
+do_install_append() {
+   rm -rf ${D}${datadir} ${D}${bindir} ${D}${base_bindir} ${D}${sbindir} 
${D}${base_sbindir} ${D}${exec_prefix}/sbin
+}
+
+BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-core/util-linux/util-linux.inc 

Re: [OE-core] [PATCH] libassuan: upgrade 2.5.4 -> 2.5.5

2021-03-25 Thread Alexander Kanavin
On Thu, 25 Mar 2021 at 00:47, wangmy  wrote:

> -@@ -158,12 +88,8 @@ dnl
> - AC_DEFUN([AM_PATH_LIBASSUAN],
> - [ _AM_PATH_LIBASSUAN_COMMON($1)
> -   if test $ok = yes; then
> --LIBASSUAN_CFLAGS=`$LIBASSUAN_CONFIG --cflags`
> --LIBASSUAN_LIBS=`$LIBASSUAN_CONFIG --libs`
> - ifelse([$2], , :, [$2])
> -   else
> --LIBASSUAN_CFLAGS=""
> --LIBASSUAN_LIBS=""
> - ifelse([$3], , :, [$3])
> -   fi
> -   AC_SUBST(LIBASSUAN_CFLAGS)
>
>
How was the patch refreshed? This chunk should not have been removed,
unless upstream has fixed the code, and even then needs to be separately
explained.

Please do use 'devtool upgrade'.

Alex

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149918): 
https://lists.openembedded.org/g/openembedded-core/message/149918
Mute This Topic: https://lists.openembedded.org/mt/81590941/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-