All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
@ 2019-08-07 10:29 Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 2/7] stress-ng: add a patch to remove unneeded bash dependency Alexander Kanavin
                   ` (6 more replies)
  0 siblings, 7 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-07 10:29 UTC (permalink / raw)
  To: openembedded-core

Recursive RDEPENDS resolution requires that all of the dependent
recipes' packaging has completed. There is no mechanism to ensure that
and therefore races were observed.

This change effectively requires recipes to list their runtime file
dependencies explicitly rather than have them pulled indirectly.
This may require a bit of fixing in layers, but should result
in a better definition of runtime file dependencies.

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 meta/classes/insane.bbclass | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 35c4fdb4913..9b886d13805 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -722,25 +722,7 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, taskdeps, packages, d):
                             filerdepends[subkey] = key[13:]
 
             if filerdepends:
-                next = rdepends
                 done = rdepends[:]
-                # Find all the rdepends on the dependency chain
-                while next:
-                    new = []
-                    for rdep in next:
-                        rdep_data = oe.packagedata.read_subpkgdata(rdep, d)
-                        sub_rdeps = rdep_data.get("RDEPENDS_" + rdep)
-                        if not sub_rdeps:
-                            continue
-                        for sub_rdep in bb.utils.explode_deps(sub_rdeps):
-                            if sub_rdep in done:
-                                continue
-                            if oe.packagedata.has_subpkgdata(sub_rdep, d):
-                                # It's a new rdep
-                                done.append(sub_rdep)
-                                new.append(sub_rdep)
-                    next = new
-
                 # Add the rprovides of itself
                 if pkg not in done:
                     done.insert(0, pkg)
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [PATCH 2/7] stress-ng: add a patch to remove unneeded bash dependency
  2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
@ 2019-08-07 10:29 ` Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 3/7] elfutils: use PRIVATE_LIBS for the ptest package Alexander Kanavin
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-07 10:29 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 ...tion-remove-the-shebang-at-the-start.patch | 23 +++++++++++++++++++
 .../stress-ng/stress-ng_0.10.00.bb            |  1 +
 2 files changed, 24 insertions(+)
 create mode 100644 meta/recipes-extended/stress-ng/stress-ng/0001-bash-completion-remove-the-shebang-at-the-start.patch

diff --git a/meta/recipes-extended/stress-ng/stress-ng/0001-bash-completion-remove-the-shebang-at-the-start.patch b/meta/recipes-extended/stress-ng/stress-ng/0001-bash-completion-remove-the-shebang-at-the-start.patch
new file mode 100644
index 00000000000..66d99dd885f
--- /dev/null
+++ b/meta/recipes-extended/stress-ng/stress-ng/0001-bash-completion-remove-the-shebang-at-the-start.patch
@@ -0,0 +1,23 @@
+From 042147675c7c2ea7dd65b2597f2e350376a710aa Mon Sep 17 00:00:00 2001
+From: Alexander Kanavin <alex.kanavin@gmail.com>
+Date: Tue, 6 Aug 2019 17:28:56 +0200
+Subject: [PATCH] bash-completion: remove the shebang at the start
+
+bash completion files do not need to specify that.
+
+Upstream-Status: Pending
+Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
+---
+ bash-completion/stress-ng | 2 --
+ 1 file changed, 2 deletions(-)
+
+diff --git a/bash-completion/stress-ng b/bash-completion/stress-ng
+index 8b1421c..7f195be 100755
+--- a/bash-completion/stress-ng
++++ b/bash-completion/stress-ng
+@@ -1,5 +1,3 @@
+-#!/bin/bash
+-#
+ # stress-ng tab completion for bash.
+ #
+ # Copyright (C) 2019 Canonical
diff --git a/meta/recipes-extended/stress-ng/stress-ng_0.10.00.bb b/meta/recipes-extended/stress-ng/stress-ng_0.10.00.bb
index f7faf97e681..e800040c371 100644
--- a/meta/recipes-extended/stress-ng/stress-ng_0.10.00.bb
+++ b/meta/recipes-extended/stress-ng/stress-ng_0.10.00.bb
@@ -7,6 +7,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
 
 SRC_URI = "https://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.xz \
            file://0001-Do-not-preserve-ownership-when-installing-example-jo.patch \
+           file://0001-bash-completion-remove-the-shebang-at-the-start.patch \
            "
 SRC_URI[md5sum] = "46aa41d37690324ceab4febfcc549018"
 SRC_URI[sha256sum] = "d09dd2a1aea549e478995bf9be90b38906a4cdf33ea7b245ef9d46aa5213c074"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [PATCH 3/7] elfutils: use PRIVATE_LIBS for the ptest package
  2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 2/7] stress-ng: add a patch to remove unneeded bash dependency Alexander Kanavin
@ 2019-08-07 10:29 ` Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 4/7] apt: add a missing perl runtime dependency Alexander Kanavin
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-07 10:29 UTC (permalink / raw)
  To: openembedded-core

EXCLUDE_PACKAGES_FROM_SHLIBS is too broad: it suppresses both generation
of required and provided shlibs. We need to suppress only the provided shlibs
(to avoid clashes with the main package providing the same shlibs),
and run the required shlib dependencies generator as usual.

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 meta/recipes-devtools/elfutils/elfutils_0.176.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/elfutils/elfutils_0.176.bb b/meta/recipes-devtools/elfutils/elfutils_0.176.bb
index 75acf59d26e..d98457e3f9f 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.176.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.176.bb
@@ -142,4 +142,4 @@ INHIBIT_PACKAGE_STRIP_FILES = "\
     ${PKGD}${PTEST_PATH}/backends/libebl_x86_64.so \
 "
 
-EXCLUDE_PACKAGES_FROM_SHLIBS = "${PN}-ptest"
+PRIVATE_LIBS_${PN}-ptest = "libdw.so.1 libelf.so.1"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [PATCH 4/7] apt: add a missing perl runtime dependency
  2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 2/7] stress-ng: add a patch to remove unneeded bash dependency Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 3/7] elfutils: use PRIVATE_LIBS for the ptest package Alexander Kanavin
@ 2019-08-07 10:29 ` Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 5/7] attr: " Alexander Kanavin
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-07 10:29 UTC (permalink / raw)
  To: openembedded-core

/usr/lib/dpkg/methods/apt/setup is a perl script.

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 meta/recipes-devtools/apt/apt-package.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-devtools/apt/apt-package.inc b/meta/recipes-devtools/apt/apt-package.inc
index da91451d7f6..922f739d0d0 100644
--- a/meta/recipes-devtools/apt/apt-package.inc
+++ b/meta/recipes-devtools/apt/apt-package.inc
@@ -35,6 +35,7 @@ FILES_${PN} = "${bindir}/apt-cdrom ${bindir}/apt-get \
 	       ${libdir}/dpkg \
 	       ${systemd_unitdir}/system \
            "
+RDEPENDS_${PN} += "perl"
 FILES_${PN}-utils = "${bindir}/apt-sortpkgs ${bindir}/apt-extracttemplates"
 FILES_${PN}-doc = "${@get_files_apt_doc(d, bb, d.getVar('apt-manpages'))} \
 		   ${docdir}/apt"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [PATCH 5/7] attr: add a missing perl runtime dependency
  2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
                   ` (2 preceding siblings ...)
  2019-08-07 10:29 ` [PATCH 4/7] apt: add a missing perl runtime dependency Alexander Kanavin
@ 2019-08-07 10:29 ` Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 6/7] ofono: correct the python3 " Alexander Kanavin
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-07 10:29 UTC (permalink / raw)
  To: openembedded-core

/usr/lib/attr/ptest/test/sort-getfattr-output is a perl script.

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 meta/recipes-support/attr/attr.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/attr/attr.inc b/meta/recipes-support/attr/attr.inc
index 1a29eb339e7..3503d66b9e1 100644
--- a/meta/recipes-support/attr/attr.inc
+++ b/meta/recipes-support/attr/attr.inc
@@ -41,6 +41,6 @@ do_install_ptest() {
 	sed -e 's|; @echo|; echo|' -i ${D}${PTEST_PATH}/test/Makefile
 }
 
-RDEPENDS_${PN}-ptest = "attr coreutils perl-module-filehandle perl-module-getopt-std perl-module-posix make"
+RDEPENDS_${PN}-ptest = "attr coreutils perl-module-filehandle perl-module-getopt-std perl-module-posix make perl"
 
 BBCLASSEXTEND = "native nativesdk"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [PATCH 6/7] ofono: correct the python3 runtime dependency
  2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
                   ` (3 preceding siblings ...)
  2019-08-07 10:29 ` [PATCH 5/7] attr: " Alexander Kanavin
@ 2019-08-07 10:29 ` Alexander Kanavin
  2019-08-07 10:29 ` [PATCH 7/7] bluez5: " Alexander Kanavin
  2019-08-07 22:08 ` [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Richard Purdie
  6 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-07 10:29 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 meta/recipes-connectivity/ofono/ofono.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/ofono/ofono.inc b/meta/recipes-connectivity/ofono/ofono.inc
index 03d0fd53a61..bdbb0b5bc61 100644
--- a/meta/recipes-connectivity/ofono/ofono.inc
+++ b/meta/recipes-connectivity/ofono/ofono.inc
@@ -35,5 +35,5 @@ RRECOMMENDS_${PN} += "kernel-module-tun mobile-broadband-provider-info"
 
 FILES_${PN} += "${systemd_unitdir}"
 FILES_${PN}-tests = "${libdir}/${BPN}/test"
-RDEPENDS_${PN}-tests = "python3 python3-dbus"
+RDEPENDS_${PN}-tests = "python3-core python3-dbus"
 RDEPENDS_${PN}-tests += "${@bb.utils.contains('GI_DATA_ENABLED', 'True', 'python3-pygobject', '', d)}"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 18+ messages in thread

* [PATCH 7/7] bluez5: correct the python3 runtime dependency
  2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
                   ` (4 preceding siblings ...)
  2019-08-07 10:29 ` [PATCH 6/7] ofono: correct the python3 " Alexander Kanavin
@ 2019-08-07 10:29 ` Alexander Kanavin
  2019-08-07 22:08 ` [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Richard Purdie
  6 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-07 10:29 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 meta/recipes-connectivity/bluez5/bluez5.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc
index 28aaeea418f..bffd174c0a0 100644
--- a/meta/recipes-connectivity/bluez5/bluez5.inc
+++ b/meta/recipes-connectivity/bluez5/bluez5.inc
@@ -139,7 +139,7 @@ def get_noinst_tools_paths (d, bb, tools):
 
 FILES_${PN}-noinst-tools = "${@get_noinst_tools_paths(d, bb, d.getVar('NOINST_TOOLS'))}"
 
-RDEPENDS_${PN}-testtools += "python3 python3-dbus"
+RDEPENDS_${PN}-testtools += "python3-core python3-dbus"
 RDEPENDS_${PN}-testtools += "${@bb.utils.contains('GI_DATA_ENABLED', 'True', 'python3-pygobject', '', d)}"
 
 SYSTEMD_SERVICE_${PN} = "${@bb.utils.contains('PACKAGECONFIG', 'systemd', 'bluetooth.service', '', d)}"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
                   ` (5 preceding siblings ...)
  2019-08-07 10:29 ` [PATCH 7/7] bluez5: " Alexander Kanavin
@ 2019-08-07 22:08 ` Richard Purdie
  2019-08-08 15:44   ` Alexander Kanavin
  6 siblings, 1 reply; 18+ messages in thread
From: Richard Purdie @ 2019-08-07 22:08 UTC (permalink / raw)
  To: Alexander Kanavin, openembedded-core

On Wed, 2019-08-07 at 12:29 +0200, Alexander Kanavin wrote:
> Recursive RDEPENDS resolution requires that all of the dependent
> recipes' packaging has completed. There is no mechanism to ensure
> that
> and therefore races were observed.
> 
> This change effectively requires recipes to list their runtime file
> dependencies explicitly rather than have them pulled indirectly.
> This may require a bit of fixing in layers, but should result
> in a better definition of runtime file dependencies.
> 
> Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
> ---
>  meta/classes/insane.bbclass | 18 ------------------
>  1 file changed, 18 deletions(-)

You might want to have a look at 
https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/621
and maybe other pieces of that build yet to finish, not sure...

Cheers,

Richard



^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-07 22:08 ` [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Richard Purdie
@ 2019-08-08 15:44   ` Alexander Kanavin
  2019-08-08 19:29     ` Khem Raj
  0 siblings, 1 reply; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-08 15:44 UTC (permalink / raw)
  To: Richard Purdie; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 424 bytes --]

On Thu, 8 Aug 2019 at 00:08, Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> You might want to have a look at
> https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/621
> and maybe other pieces of that build yet to finish, not sure...
>

I ran bitbake world to check for newly exposed missing dependencies, but
gcc-cross-canadian wasn't included in that. Will fix that too now.

Alex

[-- Attachment #2: Type: text/html, Size: 863 bytes --]

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-08 15:44   ` Alexander Kanavin
@ 2019-08-08 19:29     ` Khem Raj
  2019-08-09  1:34       ` Khem Raj
  0 siblings, 1 reply; 18+ messages in thread
From: Khem Raj @ 2019-08-08 19:29 UTC (permalink / raw)
  To: Alexander Kanavin; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 944 bytes --]

On Thu, Aug 8, 2019 at 8:44 AM Alexander Kanavin <alex.kanavin@gmail.com>
wrote:

> On Thu, 8 Aug 2019 at 00:08, Richard Purdie <
> richard.purdie@linuxfoundation.org> wrote:
>
>> You might want to have a look at
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/621
>> and maybe other pieces of that build yet to finish, not sure...
>>
>
> I ran bitbake world to check for newly exposed missing dependencies, but
> gcc-cross-canadian wasn't included in that. Will fix that too now.
>

There are several packages in meta-openembedded which are showing these
errors one that I see is net kit-rusers seems to be failing for wrong
reason

I will send full list once I am close to computer

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

[-- Attachment #2: Type: text/html, Size: 2132 bytes --]

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-08 19:29     ` Khem Raj
@ 2019-08-09  1:34       ` Khem Raj
  2019-08-09  9:45         ` Alexander Kanavin
  0 siblings, 1 reply; 18+ messages in thread
From: Khem Raj @ 2019-08-09  1:34 UTC (permalink / raw)
  To: Alexander Kanavin; +Cc: OE-core

On Thu, Aug 8, 2019 at 12:29 PM Khem Raj <raj.khem@gmail.com> wrote:
>
>
>
> On Thu, Aug 8, 2019 at 8:44 AM Alexander Kanavin <alex.kanavin@gmail.com> wrote:
>>
>> On Thu, 8 Aug 2019 at 00:08, Richard Purdie <richard.purdie@linuxfoundation.org> wrote:
>>>
>>> You might want to have a look at
>>> https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/621
>>> and maybe other pieces of that build yet to finish, not sure...
>>
>>
>> I ran bitbake world to check for newly exposed missing dependencies, but gcc-cross-canadian wasn't included in that. Will fix that too now.
>
>
> There are several packages in meta-openembedded which are showing these errors one that I see is net kit-rusers seems to be failing for wrong reason
>
> I will send full list once I am close to computer

here is full list

https://errors.yoctoproject.org/Errors/Build/86513/
all do_package_qa are due to this change.




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


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-09  1:34       ` Khem Raj
@ 2019-08-09  9:45         ` Alexander Kanavin
  2019-08-09 15:19           ` Khem Raj
  2019-08-12 19:00           ` Khem Raj
  0 siblings, 2 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-09  9:45 UTC (permalink / raw)
  To: Khem Raj; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 1427 bytes --]

On Fri, 9 Aug 2019 at 03:34, Khem Raj <raj.khem@gmail.com> wrote:

> > There are several packages in meta-openembedded which are showing these
> errors one that I see is net kit-rusers seems to be failing for wrong reason
> >
> > I will send full list once I am close to computer
>
> here is full list
>
> https://errors.yoctoproject.org/Errors/Build/86513/
> all do_package_qa are due to this change.
>

There are plenty of recipes in that list that come from oe-core (not
meta-oe), and do not show errors on the AB, e.g. glib-2.0:

ERROR: QA Issue: /usr/lib/libgio-2.0.so.0.6000.6 contained in package
glib-2.0 requires libz.so.1(ZLIB_1.2.2)(64bit), but no providers found
in RDEPENDS_glib-2.0? [file-rdeps]
ERROR: QA Issue: /usr/lib/libgio-2.0.so.0.6000.6 contained in package
glib-2.0 requires libz.so.1()(64bit), but no providers found in
RDEPENDS_glib-2.0? [file-rdeps]
ERROR: QA Issue: /usr/lib/libglib-2.0.so.0.6000.6 contained in package
glib-2.0 requires libpcre.so.1()(64bit), but no providers found in
RDEPENDS_glib-2.0? [file-rdeps]
ERROR: QA Issue: /usr/lib/libgobject-2.0.so.0.6000.6 contained in
package glib-2.0 requires libffi.so.6()(64bit), but no providers found
in RDEPENDS_glib-2.0? [file-rdeps]

Can I see the configuration where this happened? Might be some other
setting that breaks shlib resolution (that was previously unnoticed due to
recursive qa RDEPENDS).

Alex

[-- Attachment #2: Type: text/html, Size: 1878 bytes --]

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-09  9:45         ` Alexander Kanavin
@ 2019-08-09 15:19           ` Khem Raj
  2019-08-09 16:36             ` Alexander Kanavin
  2019-08-12 19:00           ` Khem Raj
  1 sibling, 1 reply; 18+ messages in thread
From: Khem Raj @ 2019-08-09 15:19 UTC (permalink / raw)
  To: Alexander Kanavin; +Cc: OE-core

On Fri, Aug 9, 2019 at 2:45 AM Alexander Kanavin <alex.kanavin@gmail.com> wrote:
>
> On Fri, 9 Aug 2019 at 03:34, Khem Raj <raj.khem@gmail.com> wrote:
>>
>> > There are several packages in meta-openembedded which are showing these errors one that I see is net kit-rusers seems to be failing for wrong reason
>> >
>> > I will send full list once I am close to computer
>>
>> here is full list
>>
>> https://errors.yoctoproject.org/Errors/Build/86513/
>> all do_package_qa are due to this change.
>
>
> There are plenty of recipes in that list that come from oe-core (not meta-oe), and do not show errors on the AB, e.g. glib-2.0:
>
> ERROR: QA Issue: /usr/lib/libgio-2.0.so.0.6000.6 contained in package glib-2.0 requires libz.so.1(ZLIB_1.2.2)(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
> ERROR: QA Issue: /usr/lib/libgio-2.0.so.0.6000.6 contained in package glib-2.0 requires libz.so.1()(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
> ERROR: QA Issue: /usr/lib/libglib-2.0.so.0.6000.6 contained in package glib-2.0 requires libpcre.so.1()(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
> ERROR: QA Issue: /usr/lib/libgobject-2.0.so.0.6000.6 contained in package glib-2.0 requires libffi.so.6()(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
>
> Can I see the configuration where this happened? Might be some other setting that breaks shlib resolution (that was previously unnoticed due to recursive qa RDEPENDS).
>

a running musl world build did not show this issue but that was
rebased tree without this fix. I will have to do a new build for glibc
and see if it was my build system injected behavior. For this patch, I
would suggest it's tested across multiple layers before it's installed
to oe-core.

> Alex
>


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-09 15:19           ` Khem Raj
@ 2019-08-09 16:36             ` Alexander Kanavin
  0 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-09 16:36 UTC (permalink / raw)
  To: Khem Raj; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 575 bytes --]

On Fri, 9 Aug 2019 at 17:19, Khem Raj <raj.khem@gmail.com> wrote:

> a running musl world build did not show this issue but that was
> rebased tree without this fix. I will have to do a new build for glibc
> and see if it was my build system injected behavior. For this patch, I
> would suggest it's tested across multiple layers before it's installed
> to oe-core.
>

We should certainly at least fix the meta-oe issues, if you can produce a
list of affected recipes I would  appreciate :)
(don't really want to start a build of entire meta-oe set here)

Alex

[-- Attachment #2: Type: text/html, Size: 906 bytes --]

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-09  9:45         ` Alexander Kanavin
  2019-08-09 15:19           ` Khem Raj
@ 2019-08-12 19:00           ` Khem Raj
  2019-08-12 19:50             ` Alexander Kanavin
  1 sibling, 1 reply; 18+ messages in thread
From: Khem Raj @ 2019-08-12 19:00 UTC (permalink / raw)
  To: Alexander Kanavin; +Cc: OE-core

On Fri, Aug 9, 2019 at 2:45 AM Alexander Kanavin <alex.kanavin@gmail.com> wrote:
>
> On Fri, 9 Aug 2019 at 03:34, Khem Raj <raj.khem@gmail.com> wrote:
>>
>> > There are several packages in meta-openembedded which are showing these errors one that I see is net kit-rusers seems to be failing for wrong reason
>> >
>> > I will send full list once I am close to computer
>>
>> here is full list
>>
>> https://errors.yoctoproject.org/Errors/Build/86513/
>> all do_package_qa are due to this change.
>
>
> There are plenty of recipes in that list that come from oe-core (not meta-oe), and do not show errors on the AB, e.g. glib-2.0:
>
> ERROR: QA Issue: /usr/lib/libgio-2.0.so.0.6000.6 contained in package glib-2.0 requires libz.so.1(ZLIB_1.2.2)(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
> ERROR: QA Issue: /usr/lib/libgio-2.0.so.0.6000.6 contained in package glib-2.0 requires libz.so.1()(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
> ERROR: QA Issue: /usr/lib/libglib-2.0.so.0.6000.6 contained in package glib-2.0 requires libpcre.so.1()(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
> ERROR: QA Issue: /usr/lib/libgobject-2.0.so.0.6000.6 contained in package glib-2.0 requires libffi.so.6()(64bit), but no providers found in RDEPENDS_glib-2.0? [file-rdeps]
>
> Can I see the configuration where this happened? Might be some other setting that breaks shlib resolution (that was previously unnoticed due to recursive qa RDEPENDS).

well, this seems to go away once I disable hash equivalency

#INHERIT += "reproducible_build"
#BB_HASHSERVE = "localhost:0"
#BB_SIGNATURE_HANDLER = "OEEquivHash"

Richard, another issue I saw was slowness in world builds when I had
this enabled
I was on master-next for both core and bitbake. It was spending a lot
of time in ensuring
the build is not needed so much that the build took almost same time
than the build with
out it where the build without it will build everything.


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-12 19:00           ` Khem Raj
@ 2019-08-12 19:50             ` Alexander Kanavin
  2019-08-12 20:50               ` richard.purdie
  0 siblings, 1 reply; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-12 19:50 UTC (permalink / raw)
  To: Khem Raj; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 669 bytes --]

On Mon, 12 Aug 2019 at 21:01, Khem Raj <raj.khem@gmail.com> wrote:

>
> Richard, another issue I saw was slowness in world builds when I had
> this enabled
> I was on master-next for both core and bitbake. It was spending a lot
> of time in ensuring
> the build is not needed so much that the build took almost same time
> than the build with
> out it where the build without it will build everything.
>

world builds seem to have regressed (preparation time wise) even without
having hash equivalency. Just oe-core is bearable, but adding meta-oe
layers makes it spent many minutes after initializing tasks but before any
tasks actually begin.

Alex

[-- Attachment #2: Type: text/html, Size: 981 bytes --]

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-12 19:50             ` Alexander Kanavin
@ 2019-08-12 20:50               ` richard.purdie
  2019-08-13  9:48                 ` Alexander Kanavin
  0 siblings, 1 reply; 18+ messages in thread
From: richard.purdie @ 2019-08-12 20:50 UTC (permalink / raw)
  To: Alexander Kanavin, Khem Raj; +Cc: OE-core

On Mon, 2019-08-12 at 21:50 +0200, Alexander Kanavin wrote:
> On Mon, 12 Aug 2019 at 21:01, Khem Raj <raj.khem@gmail.com> wrote:
> > Richard, another issue I saw was slowness in world builds when I
> > had this enabled I was on master-next for both core and bitbake. It
> > was spending a lot of time in ensuring the build is not needed so
> > much that the build took almost same time than the build with
> > out it where the build without it will build everything.

Were you using the patch in meta-yocto master-next or your own?

There is definitely a server connection problem with the hashserver
right now but if you used the setting in meta-yocto master-next you'd
probably only ever see timeouts which would be unbearably slow.

> 
> world builds seem to have regressed (preparation time wise) even
> without having hash equivalency. Just oe-core is bearable, but adding
> meta-oe layers makes it spent many minutes after initializing tasks
> but before any tasks actually begin.

Is that with master or current master-next?

I'm curious if master-next is better or worse...

To be honest I'm worrying more about correctness right now.

Cheers,

Richard



^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively
  2019-08-12 20:50               ` richard.purdie
@ 2019-08-13  9:48                 ` Alexander Kanavin
  0 siblings, 0 replies; 18+ messages in thread
From: Alexander Kanavin @ 2019-08-13  9:48 UTC (permalink / raw)
  To: Richard Purdie; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 705 bytes --]

On Mon, 12 Aug 2019 at 22:50, <richard.purdie@linuxfoundation.org> wrote:

>
> > world builds seem to have regressed (preparation time wise) even
> > without having hash equivalency. Just oe-core is bearable, but adding
> > meta-oe layers makes it spent many minutes after initializing tasks
> > but before any tasks actually begin.
>
> Is that with master or current master-next?
>
> I'm curious if master-next is better or worse...
>
> To be honest I'm worrying more about correctness right now.
>

Certainly. I didn't want to raise this now as you have plenty on your
plate, but as Khem did, I had to chime in.

This was with master@9983b07fffd19082abded7c3f15cc77d306dd69c.

Alex

[-- Attachment #2: Type: text/html, Size: 1130 bytes --]

^ permalink raw reply	[flat|nested] 18+ messages in thread

end of thread, other threads:[~2019-08-13  9:48 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-07 10:29 [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Alexander Kanavin
2019-08-07 10:29 ` [PATCH 2/7] stress-ng: add a patch to remove unneeded bash dependency Alexander Kanavin
2019-08-07 10:29 ` [PATCH 3/7] elfutils: use PRIVATE_LIBS for the ptest package Alexander Kanavin
2019-08-07 10:29 ` [PATCH 4/7] apt: add a missing perl runtime dependency Alexander Kanavin
2019-08-07 10:29 ` [PATCH 5/7] attr: " Alexander Kanavin
2019-08-07 10:29 ` [PATCH 6/7] ofono: correct the python3 " Alexander Kanavin
2019-08-07 10:29 ` [PATCH 7/7] bluez5: " Alexander Kanavin
2019-08-07 22:08 ` [PATCH 1/7] insane.bbclass: in file-rdeps do not look into RDEPENDS recursively Richard Purdie
2019-08-08 15:44   ` Alexander Kanavin
2019-08-08 19:29     ` Khem Raj
2019-08-09  1:34       ` Khem Raj
2019-08-09  9:45         ` Alexander Kanavin
2019-08-09 15:19           ` Khem Raj
2019-08-09 16:36             ` Alexander Kanavin
2019-08-12 19:00           ` Khem Raj
2019-08-12 19:50             ` Alexander Kanavin
2019-08-12 20:50               ` richard.purdie
2019-08-13  9:48                 ` Alexander Kanavin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.