Re: [OE-core] [PATCH 0/4] kernel/yocto: conslidated pull request

[Anuj Mittal]

On Wed, 2020-08-05 at 12:10 +0100, Richard Purdie wrote:
> On Wed, 2020-08-05 at 12:06 +0100, Richard Purdie via
> lists.openembedded.org wrote:
> > On Fri, 2020-07-17 at 08:56 -0400, bruce.ashfi...@gmail.com wrote:
> > > Here's my latest set of patches.
> > > 
> > > For 5.4: a -stable update, and -rt bug fix
> > > 
> > > For the build: a tweak to the config generation (as reported
> > > and discussed on the mailing list)
> > > 
> > > For 5.8 (aka the next reference kernel): a fix for devsrc and
> > > on target kernel building. With this tweak, I have all of the
> > > refrences booting and passing most tests on 5.8, so we are in
> > > good shape on that front. I'll be doing libc-headers testing
> > > next.
> > > 
> > > I ran these through the AB and local testing, no issues popped
> > > up, except a warning on qemuarm-alt in my last run. It was
> > > about a console login and didn't look related to the change,
> > > but I'm keeping my eye on it.
> > 
> > Thanks, I ran this combined with Khem's binutils change through the
> > autobuilder and your changes are likely fine but we need more work
> > to
> > get binutils in. Basically:
> > 
> > * meta-arm needs to add the kernel patch for perf
> > * meta-intel needs to add the kernel patch for perf
> > * the hardware BSPs need to add the kernel patch for perf
> > 
> > I've added a few maintainers/people to the address list to try and
> > sort
> > that out. Who is going to handle the hardware platforms?
> 
> Sorry, the url for the failing builds:
> 
> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1231
> 
> The kernel tweak needed for binutils 2.35:
> 
> http://git.yoctoproject.org/cgit/cgit.cgi/linux-yocto/commit/?h=v5.4/standard/base&id=1fe4ee7f680c3f69f620d68a5d2f7341e8db693b
> 

Thanks Richard. I have included the patch in meta-intel.

Thanks,

Anuj
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141263): 
https://lists.openembedded.org/g/openembedded-core/message/141263
Mute This Topic: https://lists.openembedded.org/mt/75611071/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] xf86-video-intel: Fix build with -fno-common

[Khem Raj]

Signed-off-by: Khem Raj 
---
v2: Use the backported patch

 ...plicate-definition-of-I810PatternROP.patch | 27 +++
 .../xorg-driver/xf86-video-intel_git.bb   |  1 +
 2 files changed, 28 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Avoid-duplicate-definition-of-I810PatternROP.patch

diff --git 
a/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Avoid-duplicate-definition-of-I810PatternROP.patch
 
b/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Avoid-duplicate-definition-of-I810PatternROP.patch
new file mode 100644
index 00..765d9ec090
--- /dev/null
+++ 
b/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Avoid-duplicate-definition-of-I810PatternROP.patch
@@ -0,0 +1,27 @@
+From c2d730cf79eb3e4bea41f5ed8a8a21092ced8b03 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Hanno=20B=C3=B6ck?= 
+Date: Sun, 2 Feb 2020 14:18:39 +0100
+Subject: [PATCH] i810: Avoid duplicate definition of I810PatternROP
+
+Upstream-Status: Backport 
[https://gitlab.freedesktop.org/xorg/driver/xf86-video-intel/-/commit/652d93cbbdc159c0883f1b626ea48e28bac63ae3]
+Signed-off-by: Chris Wilson 
+Signed-off-by: Khem Raj 
+---
+ src/legacy/i810/i810.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/legacy/i810/i810.h b/src/legacy/i810/i810.h
+index 347188c9..19be049c 100644
+--- a/src/legacy/i810/i810.h
 b/src/legacy/i810/i810.h
+@@ -322,6 +322,6 @@ extern void I810InitMC(ScreenPtr pScreen);
+ extern const OptionInfoRec *I810AvailableOptions(int chipid, int busid);
+ 
+ extern const int I810CopyROP[16];
+-const int I810PatternROP[16];
++extern const int I810PatternROP[16];
+ 
+ #endif /* _I810_H_ */
+-- 
+2.28.0
+
diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb 
b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
index 7ab2236237..161371b118 100644
--- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
+++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
@@ -15,6 +15,7 @@ S = "${WORKDIR}/git"
 
 SRC_URI = "git://anongit.freedesktop.org/xorg/driver/xf86-video-intel \
file://0001-Sync-i915_pciids-upto-8717c6b7414f.patch \
+   file://0001-i810-Avoid-duplicate-definition-of-I810PatternROP.patch 
\
 "
 
 UPSTREAM_CHECK_GITTAGREGEX = "(?P\d+(\.\d+)+)"
-- 
2.28.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141262): 
https://lists.openembedded.org/g/openembedded-core/message/141262
Mute This Topic: https://lists.openembedded.org/mt/76022456/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] xf86-video-intel: Fix build with -fno-common

[Khem Raj]

On Wed, Aug 5, 2020 at 6:38 PM Mittal, Anuj  wrote:

> On Wed, 2020-08-05 at 12:08 -0700, Khem Raj wrote:
> > Signed-off-by: Khem Raj 
> > ---
> >  ...0001-i810-Make-I810PatternROP-extern.patch | 29
> > +++
> >  .../xorg-driver/xf86-video-intel_git.bb   |  1 +
> >  2 files changed, 30 insertions(+)
> >  create mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-
> > intel/0001-i810-Make-I810PatternROP-extern.patch
> >
> > diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-
> > i810-Make-I810PatternROP-extern.patch b/meta/recipes-graphics/xorg-
> > driver/xf86-video-intel/0001-i810-Make-I810PatternROP-extern.patch
> > new file mode 100644
> > index 00..610efd65d9
> > --- /dev/null
> > +++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-
> > Make-I810PatternROP-extern.patch
> > @@ -0,0 +1,29 @@
> > +From c9a9791576025e46f4411342c0e29ffcc79869ca Mon Sep 17 00:00:00
> > 2001
> > +From: Khem Raj 
> > +Date: Wed, 5 Aug 2020 11:51:54 -0700
> > +Subject: [PATCH] i810: Make I810PatternROP extern
> > +
> > +Its also defined in i810_accel.c
> > +Fixes build with -fno-common
> > +
> > +Upstream-Status: Pending
>
> Should we take this exact same change so we can mark this as a
> backport?
>
>
> https://gitlab.freedesktop.org/xorg/driver/xf86-video-intel/-/commit/652d93cbbdc159c0883f1b626ea48e28bac63ae3


I think that’s better will just add the backport bits and send v2


>
> Thanks,
>
> Anuj
>
> > +Signed-off-by: Khem Raj 
> > +---
> > + src/legacy/i810/i810.h | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/src/legacy/i810/i810.h b/src/legacy/i810/i810.h
> > +index 347188c9..19be049c 100644
> > +--- a/src/legacy/i810/i810.h
> >  b/src/legacy/i810/i810.h
> > +@@ -322,6 +322,6 @@ extern void I810InitMC(ScreenPtr pScreen);
> > + extern const OptionInfoRec *I810AvailableOptions(int chipid, int
> > busid);
> > +
> > + extern const int I810CopyROP[16];
> > +-const int I810PatternROP[16];
> > ++extern const int I810PatternROP[16];
> > +
> > + #endif /* _I810_H_ */
> > +--
> > +2.28.0
> > +
> > diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-
> > intel_git.bb b/meta/recipes-graphics/xorg-driver/xf86-video-
> > intel_git.bb
> > index 7ab2236237..3c97b8c4f4 100644
> > --- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
> > +++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
> > @@ -15,6 +15,7 @@ S = "${WORKDIR}/git"
> >
> >  SRC_URI = "git://anongit.freedesktop.org/xorg/driver/xf86-video-
> > intel \
> > file://0001-Sync-i915_pciids-upto-8717c6b7414f.patch \
> > +   file://0001-i810-Make-I810PatternROP-extern.patch \
> >  "
> >
> >  UPSTREAM_CHECK_GITTAGREGEX = "(?P\d+(\.\d+)+)"
> > 
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141261): 
https://lists.openembedded.org/g/openembedded-core/message/141261
Mute This Topic: https://lists.openembedded.org/mt/76014141/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] xf86-video-intel: Fix build with -fno-common

[Anuj Mittal]

On Wed, 2020-08-05 at 12:08 -0700, Khem Raj wrote:
> Signed-off-by: Khem Raj 
> ---
>  ...0001-i810-Make-I810PatternROP-extern.patch | 29
> +++
>  .../xorg-driver/xf86-video-intel_git.bb   |  1 +
>  2 files changed, 30 insertions(+)
>  create mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-
> intel/0001-i810-Make-I810PatternROP-extern.patch
> 
> diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-
> i810-Make-I810PatternROP-extern.patch b/meta/recipes-graphics/xorg-
> driver/xf86-video-intel/0001-i810-Make-I810PatternROP-extern.patch
> new file mode 100644
> index 00..610efd65d9
> --- /dev/null
> +++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-
> Make-I810PatternROP-extern.patch
> @@ -0,0 +1,29 @@
> +From c9a9791576025e46f4411342c0e29ffcc79869ca Mon Sep 17 00:00:00
> 2001
> +From: Khem Raj 
> +Date: Wed, 5 Aug 2020 11:51:54 -0700
> +Subject: [PATCH] i810: Make I810PatternROP extern
> +
> +Its also defined in i810_accel.c
> +Fixes build with -fno-common
> +
> +Upstream-Status: Pending

Should we take this exact same change so we can mark this as a
backport?

https://gitlab.freedesktop.org/xorg/driver/xf86-video-intel/-/commit/652d93cbbdc159c0883f1b626ea48e28bac63ae3

Thanks,

Anuj

> +Signed-off-by: Khem Raj 
> +---
> + src/legacy/i810/i810.h | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/src/legacy/i810/i810.h b/src/legacy/i810/i810.h
> +index 347188c9..19be049c 100644
> +--- a/src/legacy/i810/i810.h
>  b/src/legacy/i810/i810.h
> +@@ -322,6 +322,6 @@ extern void I810InitMC(ScreenPtr pScreen);
> + extern const OptionInfoRec *I810AvailableOptions(int chipid, int
> busid);
> + 
> + extern const int I810CopyROP[16];
> +-const int I810PatternROP[16];
> ++extern const int I810PatternROP[16];
> + 
> + #endif /* _I810_H_ */
> +-- 
> +2.28.0
> +
> diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-
> intel_git.bb b/meta/recipes-graphics/xorg-driver/xf86-video-
> intel_git.bb
> index 7ab2236237..3c97b8c4f4 100644
> --- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
> +++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
> @@ -15,6 +15,7 @@ S = "${WORKDIR}/git"
>  
>  SRC_URI = "git://anongit.freedesktop.org/xorg/driver/xf86-video-
> intel \
> file://0001-Sync-i915_pciids-upto-8717c6b7414f.patch \
> +   file://0001-i810-Make-I810PatternROP-extern.patch \
>  "
>  
>  UPSTREAM_CHECK_GITTAGREGEX = "(?P\d+(\.\d+)+)"
> 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141260): 
https://lists.openembedded.org/g/openembedded-core/message/141260
Mute This Topic: https://lists.openembedded.org/mt/76014141/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] ✗ patchtest: failure for ltp: make copyFrom scp command non-fatal (rev4)

[Matthew]

Richard Purdie  writes:

> On Wed, 2020-08-05 at 15:23 -0400, Matthew wrote:
>> My poky master is the latest commit (5e1bc3ef5a * net-tools: fix
>> upstream version check) and my patch applies perfectly. Why is
>> Patchwork complaining?
>
> FWIW I just tried to apply the patch and it doesn't apply for me :(.
>
> (Stripping trailing CRs from patch; use --binary to disable.)
> patching file meta/lib/oeqa/core/target/ssh.py
> (Stripping trailing CRs from patch; use --binary to disable.)
> patching file meta/lib/oeqa/runtime/cases/ltp.py
> Hunk #1 FAILED at 78.
> 1 out of 1 hunk FAILED -- saving rejects to file 
> meta/lib/oeqa/runtime/cases/ltp.py.rej
>
> The difference is trailing whitespace which is present on the line:
>
> remote_src = "/opt/ltp/results/%s" % ltp_group
>
> but has been stripped in your patch.

Arghh, I configured automatic remove trailing whitespace on save, that must be 
the issue.

Patch with the whitespace sent.

>
> Cheers,
>
> Richard


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141259): 
https://lists.openembedded.org/g/openembedded-core/message/141259
Mute This Topic: https://lists.openembedded.org/mt/76013995/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCHv5] ltp: make copyFrom scp command non-fatal

[Matthew]

[YOCTO #13802]

Make the scp failure non-fatal so the ltp tests continue to run and
the rest of the logs will be available to see afterwards.

Signed-off-by: Mingde (Matthew) Zeng 
---
 meta/lib/oeqa/core/target/ssh.py   | 7 +--
 meta/lib/oeqa/runtime/cases/ltp.py | 5 +++--
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 090b40a814..aefb576805 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
 scpCmd = self.scp + [localSrc, remotePath]
 return self._run(scpCmd, ignore_status=False)

-def copyFrom(self, remoteSrc, localDst):
+def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
 """
 Copy file from target.
 """
 remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
 scpCmd = self.scp + [remotePath, localDst]
-return self._run(scpCmd, ignore_status=False)
+(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
+if warn_on_failure and status:
+self.logger.warning("Copy returned non-zero exit status %d:\n%s" % 
(status, output))
+return (status, output)

 def copyDirTo(self, localSrc, remoteDst):
 """
diff --git a/meta/lib/oeqa/runtime/cases/ltp.py 
b/meta/lib/oeqa/runtime/cases/ltp.py
index 6dc5ef22ad..a66d5d13d7 100644
--- a/meta/lib/oeqa/runtime/cases/ltp.py
+++ b/meta/lib/oeqa/runtime/cases/ltp.py
@@ -78,9 +78,10 @@ class LtpTest(LtpTestBase):
 # copy nice log from DUT
 dst = os.path.join(self.ltptest_log_dir, "%s" %  ltp_group )
 remote_src = "/opt/ltp/results/%s" % ltp_group 
-(status, output) = self.target.copyFrom(remote_src, dst)
+(status, output) = self.target.copyFrom(remote_src, dst, True)
 msg = 'File could not be copied. Output: %s' % output
-self.assertEqual(status, 0, msg=msg)
+if status:
+self.target.logger.warning(msg)

 parser = LtpParser()
 results, sections  = parser.parse(dst)
--
2.27.0
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141258): 
https://lists.openembedded.org/g/openembedded-core/message/141258
Mute This Topic: https://lists.openembedded.org/mt/76015435/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] ✗ patchtest: failure for ltp: make copyFrom scp command non-fatal (rev4)

[Richard Purdie]

On Wed, 2020-08-05 at 15:23 -0400, Matthew wrote:
> My poky master is the latest commit (5e1bc3ef5a * net-tools: fix
> upstream version check) and my patch applies perfectly. Why is
> Patchwork complaining?

FWIW I just tried to apply the patch and it doesn't apply for me :(.

(Stripping trailing CRs from patch; use --binary to disable.)
patching file meta/lib/oeqa/core/target/ssh.py
(Stripping trailing CRs from patch; use --binary to disable.)
patching file meta/lib/oeqa/runtime/cases/ltp.py
Hunk #1 FAILED at 78.
1 out of 1 hunk FAILED -- saving rejects to file 
meta/lib/oeqa/runtime/cases/ltp.py.rej

The difference is trailing whitespace which is present on the line:

remote_src = "/opt/ltp/results/%s" % ltp_group 

but has been stripped in your patch.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141257): 
https://lists.openembedded.org/g/openembedded-core/message/141257
Mute This Topic: https://lists.openembedded.org/mt/76013995/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] ✗ patchtest: failure for ltp: make copyFrom scp command non-fatal (rev4)

[Matthew]

My poky master is the latest commit (5e1bc3ef5a * net-tools: fix upstream 
version check) and my patch applies perfectly. Why is Patchwork complaining?

Patchwork  writes:

> == Series Details ==
>
> Series: ltp: make copyFrom scp command non-fatal (rev4)
> Revision: 4
> URL   : https://patchwork.openembedded.org/series/25475/
> State : failure
>
> == Summary ==
>
>
> Thank you for submitting this patch series to OpenEmbedded Core. This is
> an automated response. Several tests have been executed on the proposed
> series by patchtest resulting in the following failures:
>
>
>
> * Issue Series does not apply on top of target branch 
> [test_series_merge_on_head]
>   Suggested fixRebase your series on top of targeted branch
>   Targeted branch  master (currently at 10221e578f)
>
>
>
> If you believe any of these test results are incorrect, please reply to the
> mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
> Otherwise we would appreciate you correcting the issues and submitting a new
> version of the patchset if applicable. Please ensure you add/increment the
> version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
> [PATCH v3] -> ...).
>
> ---
> Guidelines: 
> https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
> Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
> Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141256): 
https://lists.openembedded.org/g/openembedded-core/message/141256
Mute This Topic: https://lists.openembedded.org/mt/76013995/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/2] xf86-video-intel: Fix build with -fno-common

[Khem Raj]

Signed-off-by: Khem Raj 
---
 ...0001-i810-Make-I810PatternROP-extern.patch | 29 +++
 .../xorg-driver/xf86-video-intel_git.bb   |  1 +
 2 files changed, 30 insertions(+)
 create mode 100644 
meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Make-I810PatternROP-extern.patch

diff --git 
a/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Make-I810PatternROP-extern.patch
 
b/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Make-I810PatternROP-extern.patch
new file mode 100644
index 00..610efd65d9
--- /dev/null
+++ 
b/meta/recipes-graphics/xorg-driver/xf86-video-intel/0001-i810-Make-I810PatternROP-extern.patch
@@ -0,0 +1,29 @@
+From c9a9791576025e46f4411342c0e29ffcc79869ca Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Wed, 5 Aug 2020 11:51:54 -0700
+Subject: [PATCH] i810: Make I810PatternROP extern
+
+Its also defined in i810_accel.c
+Fixes build with -fno-common
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj 
+---
+ src/legacy/i810/i810.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/legacy/i810/i810.h b/src/legacy/i810/i810.h
+index 347188c9..19be049c 100644
+--- a/src/legacy/i810/i810.h
 b/src/legacy/i810/i810.h
+@@ -322,6 +322,6 @@ extern void I810InitMC(ScreenPtr pScreen);
+ extern const OptionInfoRec *I810AvailableOptions(int chipid, int busid);
+ 
+ extern const int I810CopyROP[16];
+-const int I810PatternROP[16];
++extern const int I810PatternROP[16];
+ 
+ #endif /* _I810_H_ */
+-- 
+2.28.0
+
diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb 
b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
index 7ab2236237..3c97b8c4f4 100644
--- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
+++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
@@ -15,6 +15,7 @@ S = "${WORKDIR}/git"
 
 SRC_URI = "git://anongit.freedesktop.org/xorg/driver/xf86-video-intel \
file://0001-Sync-i915_pciids-upto-8717c6b7414f.patch \
+   file://0001-i810-Make-I810PatternROP-extern.patch \
 "
 
 UPSTREAM_CHECK_GITTAGREGEX = "(?P\d+(\.\d+)+)"
-- 
2.28.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141254): 
https://lists.openembedded.org/g/openembedded-core/message/141254
Mute This Topic: https://lists.openembedded.org/mt/76014141/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/2] libacpi: Fix build with -fno-common

[Khem Raj]

Signed-off-by: Khem Raj 
---
 ...1-libacpi-Fix-build-witth-fno-commom.patch | 68 +++
 meta/recipes-bsp/libacpi/libacpi_0.2.bb   |  4 +-
 2 files changed, 71 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-bsp/libacpi/files/0001-libacpi-Fix-build-witth-fno-commom.patch

diff --git 
a/meta/recipes-bsp/libacpi/files/0001-libacpi-Fix-build-witth-fno-commom.patch 
b/meta/recipes-bsp/libacpi/files/0001-libacpi-Fix-build-witth-fno-commom.patch
new file mode 100644
index 00..32808fb92a
--- /dev/null
+++ 
b/meta/recipes-bsp/libacpi/files/0001-libacpi-Fix-build-witth-fno-commom.patch
@@ -0,0 +1,68 @@
+From 9839c169f513e65fc711646257b3e8588cce623c Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Wed, 5 Aug 2020 12:06:01 -0700
+Subject: [PATCH] libacpi: Fix build witth -fno-commom
+
+Upstream-Status: Pending
+
+Signed-off-by: Khem Raj 
+---
+ libacpi.c | 16 
+ libacpi.h |  6 +++---
+ 2 files changed, 19 insertions(+), 3 deletions(-)
+
+diff --git a/libacpi.c b/libacpi.c
+index 4e34725..9344ea4 100644
+--- a/libacpi.c
 b/libacpi.c
+@@ -14,6 +14,22 @@
+ #include "libacpi.h"
+ #include "list.h"
+ 
++/**
++ * Array for existing batteries, loop until
++ * globals->battery_count
++ */
++battery_t batteries[MAX_ITEMS];
++/**
++ * Array for existing thermal zones, loop until
++ * globals->thermal_count
++ */
++thermal_t thermals[MAX_ITEMS];
++/**
++ * Array for existing fans, loop until
++ * globals->fan_count
++ */
++fan_t fans[MAX_ITEMS];
++
+ static int read_acpi_battinfo(const int num);
+ static int read_acpi_battalarm(const int num);
+ static int read_acpi_battstate(const int num);
+diff --git a/libacpi.h b/libacpi.h
+index 9334b79..5242d3d 100644
+--- a/libacpi.h
 b/libacpi.h
+@@ -183,17 +183,17 @@ typedef struct {
+  * Array for existing batteries, loop until
+  * globals->battery_count
+  */
+-battery_t batteries[MAX_ITEMS];
++extern battery_t batteries[MAX_ITEMS];
+ /**
+  * Array for existing thermal zones, loop until
+  * globals->thermal_count
+  */
+-thermal_t thermals[MAX_ITEMS];
++extern thermal_t thermals[MAX_ITEMS];
+ /**
+  * Array for existing fans, loop until
+  * globals->fan_count
+  */
+-fan_t fans[MAX_ITEMS];
++extern fan_t fans[MAX_ITEMS];
+ /**
+  * Finds existing batteries and fills the
+  * corresponding batteries structures with the paths
+-- 
+2.28.0
+
diff --git a/meta/recipes-bsp/libacpi/libacpi_0.2.bb 
b/meta/recipes-bsp/libacpi/libacpi_0.2.bb
index 5e7f7fa8e0..fa9e3d4894 100644
--- a/meta/recipes-bsp/libacpi/libacpi_0.2.bb
+++ b/meta/recipes-bsp/libacpi/libacpi_0.2.bb
@@ -11,7 +11,9 @@ SRC_URI = "http://www.ngolde.de/download/libacpi-${PV}.tar.gz 
\
   file://makefile-fix.patch \
   file://libacpi_fix_for_x32.patch \
   file://use_correct_strip_in_cross_environment.patch \
-  file://ldflags.patch "
+  file://ldflags.patch \
+   file://0001-libacpi-Fix-build-witth-fno-commom.patch \
+   "
 
 SRC_URI[md5sum] = "05b53dd7bead66dda35fec502b91066c"
 SRC_URI[sha256sum] = 
"13086e31d428b9c125954d48ac497b754bbbce2ef34ea29ecd903e82e25bad29"
-- 
2.28.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141255): 
https://lists.openembedded.org/g/openembedded-core/message/141255
Mute This Topic: https://lists.openembedded.org/mt/76014143/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] systemtap: update to 4.3 latest

[Bruce Ashfield]

From: Bruce Ashfield 

Updating systemtap to the latest of the 4.3 release branch. Without
this update we cannot pass on-target sanity tests with a 5.8
kernel.

Changelog is as follows:

82b8e1a07 (HEAD -> master, origin/master, origin/HEAD) Make dtrace 
generated code work with LTO (take 2)
03a6c471a Fix --enable-http build errors by always using MHD_Result
87344e948 PR26307: rhel6 porting tweak redux
24810e5ac PR26307: rhel6 porting tweak
327956981 java/HelperSDT.c: correct 32-bit pointer cast warnings
2f3580f40 PR26307: adapt to kernel module_sect_attr changes in 5.8+
84bc64ff1 systemtap.spec: let -testsuite subrpm require elfutils-debuginfod
27b087e7f PR26249: "%p" -> "0x%lx" pointer formatting in *conversions.stp 
error messages
5e1ef9d7f PR25568 / RHBZ1857749: sdt_buildid.exp test case
e90530877 PR25568 / RHBZ1857749: buildid/uprobes/inode rework, task_finder 
etc. side
40dbe72af PR26234: stapbpf should warn about other unsupported 
derived_probe types
82f358beb testsuite: shrink output
a9a0131eb rhbz1857749: uprobes-inode regression in sdt semaphore setting
ce0fa621e configury: make systemtap buildable with gcc -flto
e5a63d9c0 Make dtrace generated code work with LTO
e8070f260 Revert "Make dtrace generated code work with LTO"
b26a86bfb systemtap.spec: comment syntax tweak
8212024da Make dtrace generated code work with LTO
601778117 PR26234: stapbpf should warn about unsupported 
utrace_derived_probes
c7d4f550e testsuite: block listing_mode_sanity large tests on small machine
046fa017d RHBZ1847676 cont'd: one more uprobes-inode/onthefly concurrency 
control
a8351c3dc runtime: startup dmesg %p decloaking
65d6fbd21 run-stap.in: forget about bundled elfutils build mode
57c76ab3f PR25549: format tweak
143974310 PR25549: statement probe visibility for openmp / lto binaries
c6831f14e testuite: More @cast() fallout
a4d7b3797 README: modernize
f1a9bb064 Tapset and testsuite updates against @cast() change 00ee19ff03
4ccdfe453 RHBZ1847676 cont'd: more uprobes-inode/onthefly concurrency 
controls
487d6b6e8 tapset: *user_string* formatting directive tweaks
b266d0bf8 PR26142: Adapt to linux/vermagic.h file hiding ... but with 
proper include order
79000b42e PR26142: Adapt to linux/vermagic.h file hiding ... but not on 
rhel6
37066e2c3 PR26142: adapt to linux mmap_sem api transition
6af5df835 PR26142: Adapt to linux/vermagic.h file hiding
403e92779 PR26181: Use explicit @cast() within get_ip_from_client()
4bb36e7fb Exclude some __NR_compat_* defines for newer aarch64 kernels
a948c291c Use explicit @cast() operators pointing to kernel for tapsets
469d20e8b PR26131: garbled data might appear in staprun data channel output
36430614d Use kernel.trace("sched:sched_process_fork") for kprocess.create 
when possible
b2d18cb3a Use explicit @cast() operators for semop-watch.stp example.
3d922919d Use explicit @cast() operators for periodic.stp
9a2cd427a PR24758: increase stack size for BPF userspace interpreter
3040d4e8d Use explicit @cast() operators for stapgames/pingpong.stp tapset.
9eb37102d Use explicit @cast() operators for pfiles.stp and 
ioctl_handler.stp
7a28529f6 Remove the unneeded test_support check the lwtools meta info
717b7 Use explicit @cast() operators to fslatency-nd.stp and 
fsslower-nd.stp
2b2b6a622 Fix sizeof.stp to explicitly use kernel debuginfo if one not 
specified
db91f0291 bpf.exp: tentative fix for bigmap1.stp hang on RHEL8
11c39a737 RHBZ1847676: uprobes-inode tweaks redux
0a281a96d Make sizeof.stp runnable with the bpf backend.
5b8f6c174 remove comment
fb59e8c75 PR26123: fixed memory leaks in vma map for kernels without 
CONFIG_UTRACE.
cc7b62017 Add overcommit.stp to examples that can run in bpf.
ac8f2b7c9 configury: post-release version bump
7a7016a12 Avoid exceeding space constraints for the BPF environment in 
mmfilepage.stp
c488859d6 Avoid exceeding space constraints for BPF environment in 
mmwriteback.stp

Signed-off-by: Bruce Ashfield 
---
 meta/recipes-kernel/systemtap/systemtap_git.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/systemtap/systemtap_git.inc 
b/meta/recipes-kernel/systemtap/systemtap_git.inc
index 4ec0703f2b..56fa51d61e 100644
--- a/meta/recipes-kernel/systemtap/systemtap_git.inc
+++ b/meta/recipes-kernel/systemtap/systemtap_git.inc
@@ -1,6 +1,6 @@
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
-SRCREV = "c9c23c987d819d07c6b96b54f8e03188fecd9e46"
+SRCREV = "82b8e1a07a31bf37ed05d6ebc5162b054c0be9fd"
 PV = "4.3"
 
 SRC_URI = "git://sourceware.org/git/systemtap.git \
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141253): 
https://lists.openembedded.org/g/openembedded-core/message/141253
Mute This Topic

[OE-core] ✗ patchtest: failure for ltp: make copyFrom scp command non-fatal (rev4)

[Patchwork]

== Series Details ==

Series: ltp: make copyFrom scp command non-fatal (rev4)
Revision: 4
URL   : https://patchwork.openembedded.org/series/25475/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Series does not apply on top of target branch 
[test_series_merge_on_head] 
  Suggested fixRebase your series on top of targeted branch
  Targeted branch  master (currently at 10221e578f)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141252): 
https://lists.openembedded.org/g/openembedded-core/message/141252
Mute This Topic: https://lists.openembedded.org/mt/76013995/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCHv4] ltp: make copyFrom scp command non-fatal

[Matthew]

[YOCTO #13802]

Make the scp failure non-fatal so the ltp tests continue to run and
the rest of the logs will be available to see afterwards.

Signed-off-by: Mingde (Matthew) Zeng 
---
 meta/lib/oeqa/core/target/ssh.py   | 7 +--
 meta/lib/oeqa/runtime/cases/ltp.py | 5 +++--
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 090b40a814..aefb576805 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
 scpCmd = self.scp + [localSrc, remotePath]
 return self._run(scpCmd, ignore_status=False)

-def copyFrom(self, remoteSrc, localDst):
+def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
 """
 Copy file from target.
 """
 remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
 scpCmd = self.scp + [remotePath, localDst]
-return self._run(scpCmd, ignore_status=False)
+(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
+if warn_on_failure and status:
+self.logger.warning("Copy returned non-zero exit status %d:\n%s" % 
(status, output))
+return (status, output)

 def copyDirTo(self, localSrc, remoteDst):
 """
diff --git a/meta/lib/oeqa/runtime/cases/ltp.py 
b/meta/lib/oeqa/runtime/cases/ltp.py
index 6dc5ef22ad..a66d5d13d7 100644
--- a/meta/lib/oeqa/runtime/cases/ltp.py
+++ b/meta/lib/oeqa/runtime/cases/ltp.py
@@ -78,9 +78,10 @@ class LtpTest(LtpTestBase):
 # copy nice log from DUT
 dst = os.path.join(self.ltptest_log_dir, "%s" %  ltp_group )
 remote_src = "/opt/ltp/results/%s" % ltp_group
-(status, output) = self.target.copyFrom(remote_src, dst)
+(status, output) = self.target.copyFrom(remote_src, dst, True)
 msg = 'File could not be copied. Output: %s' % output
-self.assertEqual(status, 0, msg=msg)
+if status:
+self.target.logger.warning(msg)

 parser = LtpParser()
 results, sections  = parser.parse(dst)
--
2.27.0
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141251): 
https://lists.openembedded.org/g/openembedded-core/message/141251
Mute This Topic: https://lists.openembedded.org/mt/76013802/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] [PATCHv3] ltp: make copyFrom scp command non-fatal

[Matthew]

Richard Purdie  writes:

> On Wed, 2020-08-05 at 14:33 -0400, Mingde (Matthew) Zeng wrote:
>> Richard Purdie  writes:
>>
>> > On Wed, 2020-08-05 at 14:10 -0400, Matthew wrote:
>> > > [YOCTO #13802]
>> > >
>> > > Make the scp failure non-fatal so the ltp tests continue to run and
>> > > the rest of the logs will be available to see afterwards.
>> > >
>> > > Signed-off-by: Mingde (Matthew) Zeng 
>> > > ---
>> > >  meta/lib/oeqa/core/target/ssh.py   | 7 +--
>> > >  meta/lib/oeqa/runtime/cases/ltp.py | 2 +-
>> > >  2 files changed, 6 insertions(+), 3 deletions(-)
>> > >
>> > > diff --git a/meta/lib/oeqa/core/target/ssh.py 
>> > > b/meta/lib/oeqa/core/target/ssh.py
>> > > index 090b40a814..aefb576805 100644
>> > > --- a/meta/lib/oeqa/core/target/ssh.py
>> > > +++ b/meta/lib/oeqa/core/target/ssh.py
>> > > @@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
>> > >  scpCmd = self.scp + [localSrc, remotePath]
>> > >  return self._run(scpCmd, ignore_status=False)
>> > >
>> > > -def copyFrom(self, remoteSrc, localDst):
>> > > +def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
>> > >  """
>> > >  Copy file from target.
>> > >  """
>> > >  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>> > >  scpCmd = self.scp + [remotePath, localDst]
>> > > -return self._run(scpCmd, ignore_status=False)
>> > > +(status, output) = self._run(scpCmd, 
>> > > ignore_status=warn_on_failure)
>> > > +if warn_on_failure and status:
>> > > +self.logger.warning("Copy returned non-zero exit status 
>> > > %d:\n%s" % (status, output))
>> > > +return (status, output)
>> > >
>> > >  def copyDirTo(self, localSrc, remoteDst):
>> > >  """
>> > > diff --git a/meta/lib/oeqa/runtime/cases/ltp.py 
>> > > b/meta/lib/oeqa/runtime/cases/ltp.py
>> > > index 6dc5ef22ad..da529ce482 100644
>> > > --- a/meta/lib/oeqa/runtime/cases/ltp.py
>> > > +++ b/meta/lib/oeqa/runtime/cases/ltp.py
>> > > @@ -78,7 +78,7 @@ class LtpTest(LtpTestBase):
>> > >  # copy nice log from DUT
>> > >  dst = os.path.join(self.ltptest_log_dir, "%s" %  ltp_group )
>> > >  remote_src = "/opt/ltp/results/%s" % ltp_group
>> > > -(status, output) = self.target.copyFrom(remote_src, dst)
>> > > +(status, output) = self.target.copyFrom(remote_src, dst, 
>> > > True)
>> > >  msg = 'File could not be copied. Output: %s' % output
>> > >  self.assertEqual(status, 0, msg=msg)
>> >
>> > The idea of this change is so "the ltp tests continue to run" yet if I
>> > understand that code, a copy failure will throw the assertEqual and
>> > hence stop the test so we don't get all the other logging information?
>>
>> When warn_on_failure=True, so is ignore_status. Then 'status and not
>> ignore_status' evaluates to false, and 'raise AssertionError("Command
>> '%s' returned non-zero exit ...)' will not run.
>
> Sure, but status will be non-zero and the code says:
>
> (status, output) = self.target.copyFrom(remote_src, dst, True)
> msg = 'File could not be copied. Output: %s' % output
> self.assertEqual(status, 0, msg=msg)
>
> so
>
> self.assertEqual(status, 0, msg=msg)
>
> will fail if copyFrom() fails?

Right, I'll comment that line out as well.

Also, I received the Patchwork email complaining a conflict, however I 
successfully applied my patch after git pulling a couple times, do you know 
what's wrong?

>
> Cheers,
>
> Richard


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141250): 
https://lists.openembedded.org/g/openembedded-core/message/141250
Mute This Topic: https://lists.openembedded.org/mt/76013023/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] [PATCHv3] ltp: make copyFrom scp command non-fatal

[Richard Purdie]

On Wed, 2020-08-05 at 14:33 -0400, Mingde (Matthew) Zeng wrote:
> Richard Purdie  writes:
> 
> > On Wed, 2020-08-05 at 14:10 -0400, Matthew wrote:
> > > [YOCTO #13802]
> > > 
> > > Make the scp failure non-fatal so the ltp tests continue to run and
> > > the rest of the logs will be available to see afterwards.
> > > 
> > > Signed-off-by: Mingde (Matthew) Zeng 
> > > ---
> > >  meta/lib/oeqa/core/target/ssh.py   | 7 +--
> > >  meta/lib/oeqa/runtime/cases/ltp.py | 2 +-
> > >  2 files changed, 6 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/meta/lib/oeqa/core/target/ssh.py 
> > > b/meta/lib/oeqa/core/target/ssh.py
> > > index 090b40a814..aefb576805 100644
> > > --- a/meta/lib/oeqa/core/target/ssh.py
> > > +++ b/meta/lib/oeqa/core/target/ssh.py
> > > @@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
> > >  scpCmd = self.scp + [localSrc, remotePath]
> > >  return self._run(scpCmd, ignore_status=False)
> > > 
> > > -def copyFrom(self, remoteSrc, localDst):
> > > +def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
> > >  """
> > >  Copy file from target.
> > >  """
> > >  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
> > >  scpCmd = self.scp + [remotePath, localDst]
> > > -return self._run(scpCmd, ignore_status=False)
> > > +(status, output) = self._run(scpCmd, 
> > > ignore_status=warn_on_failure)
> > > +if warn_on_failure and status:
> > > +self.logger.warning("Copy returned non-zero exit status 
> > > %d:\n%s" % (status, output))
> > > +return (status, output)
> > > 
> > >  def copyDirTo(self, localSrc, remoteDst):
> > >  """
> > > diff --git a/meta/lib/oeqa/runtime/cases/ltp.py 
> > > b/meta/lib/oeqa/runtime/cases/ltp.py
> > > index 6dc5ef22ad..da529ce482 100644
> > > --- a/meta/lib/oeqa/runtime/cases/ltp.py
> > > +++ b/meta/lib/oeqa/runtime/cases/ltp.py
> > > @@ -78,7 +78,7 @@ class LtpTest(LtpTestBase):
> > >  # copy nice log from DUT
> > >  dst = os.path.join(self.ltptest_log_dir, "%s" %  ltp_group )
> > >  remote_src = "/opt/ltp/results/%s" % ltp_group
> > > -(status, output) = self.target.copyFrom(remote_src, dst)
> > > +(status, output) = self.target.copyFrom(remote_src, dst, 
> > > True)
> > >  msg = 'File could not be copied. Output: %s' % output
> > >  self.assertEqual(status, 0, msg=msg)
> > 
> > The idea of this change is so "the ltp tests continue to run" yet if I
> > understand that code, a copy failure will throw the assertEqual and
> > hence stop the test so we don't get all the other logging information?
> 
> When warn_on_failure=True, so is ignore_status. Then 'status and not
> ignore_status' evaluates to false, and 'raise AssertionError("Command
> '%s' returned non-zero exit ...)' will not run.

Sure, but status will be non-zero and the code says:

(status, output) = self.target.copyFrom(remote_src, dst, True)
msg = 'File could not be copied. Output: %s' % output
self.assertEqual(status, 0, msg=msg)

so

self.assertEqual(status, 0, msg=msg)

will fail if copyFrom() fails?

Cheers,

Richard




-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141249): 
https://lists.openembedded.org/g/openembedded-core/message/141249
Mute This Topic: https://lists.openembedded.org/mt/76013023/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] [PATCHv3] ltp: make copyFrom scp command non-fatal

[Matthew]

Richard Purdie  writes:

> On Wed, 2020-08-05 at 14:10 -0400, Matthew wrote:
>> [YOCTO #13802]
>>
>> Make the scp failure non-fatal so the ltp tests continue to run and
>> the rest of the logs will be available to see afterwards.
>>
>> Signed-off-by: Mingde (Matthew) Zeng 
>> ---
>>  meta/lib/oeqa/core/target/ssh.py   | 7 +--
>>  meta/lib/oeqa/runtime/cases/ltp.py | 2 +-
>>  2 files changed, 6 insertions(+), 3 deletions(-)
>>
>> diff --git a/meta/lib/oeqa/core/target/ssh.py 
>> b/meta/lib/oeqa/core/target/ssh.py
>> index 090b40a814..aefb576805 100644
>> --- a/meta/lib/oeqa/core/target/ssh.py
>> +++ b/meta/lib/oeqa/core/target/ssh.py
>> @@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
>>  scpCmd = self.scp + [localSrc, remotePath]
>>  return self._run(scpCmd, ignore_status=False)
>>
>> -def copyFrom(self, remoteSrc, localDst):
>> +def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
>>  """
>>  Copy file from target.
>>  """
>>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>>  scpCmd = self.scp + [remotePath, localDst]
>> -return self._run(scpCmd, ignore_status=False)
>> +(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
>> +if warn_on_failure and status:
>> +self.logger.warning("Copy returned non-zero exit status 
>> %d:\n%s" % (status, output))
>> +return (status, output)
>>
>>  def copyDirTo(self, localSrc, remoteDst):
>>  """
>> diff --git a/meta/lib/oeqa/runtime/cases/ltp.py 
>> b/meta/lib/oeqa/runtime/cases/ltp.py
>> index 6dc5ef22ad..da529ce482 100644
>> --- a/meta/lib/oeqa/runtime/cases/ltp.py
>> +++ b/meta/lib/oeqa/runtime/cases/ltp.py
>> @@ -78,7 +78,7 @@ class LtpTest(LtpTestBase):
>>  # copy nice log from DUT
>>  dst = os.path.join(self.ltptest_log_dir, "%s" %  ltp_group )
>>  remote_src = "/opt/ltp/results/%s" % ltp_group
>> -(status, output) = self.target.copyFrom(remote_src, dst)
>> +(status, output) = self.target.copyFrom(remote_src, dst, True)
>>  msg = 'File could not be copied. Output: %s' % output
>>  self.assertEqual(status, 0, msg=msg)
>
> The idea of this change is so "the ltp tests continue to run" yet if I
> understand that code, a copy failure will throw the assertEqual and
> hence stop the test so we don't get all the other logging information?

When warn_on_failure=True, so is ignore_status. Then 'status and not 
ignore_status' evaluates to false, and 'raise AssertionError("Command '%s' 
returned non-zero exit ...)' will not run.

>
> Cheers,
>
> Richard
>
> 


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141248): 
https://lists.openembedded.org/g/openembedded-core/message/141248
Mute This Topic: https://lists.openembedded.org/mt/76013023/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] ✗ patchtest: failure for ltp: make copyFrom scp command non-fatal (rev3)

[Patchwork]

== Series Details ==

Series: ltp: make copyFrom scp command non-fatal (rev3)
Revision: 3
URL   : https://patchwork.openembedded.org/series/25475/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Series does not apply on top of target branch 
[test_series_merge_on_head] 
  Suggested fixRebase your series on top of targeted branch
  Targeted branch  master (currently at 10221e578f)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141247): 
https://lists.openembedded.org/g/openembedded-core/message/141247
Mute This Topic: https://lists.openembedded.org/mt/76013415/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] [PATCHv3] ltp: make copyFrom scp command non-fatal

[Richard Purdie]

On Wed, 2020-08-05 at 14:10 -0400, Matthew wrote:
> [YOCTO #13802]
> 
> Make the scp failure non-fatal so the ltp tests continue to run and
> the rest of the logs will be available to see afterwards.
> 
> Signed-off-by: Mingde (Matthew) Zeng 
> ---
>  meta/lib/oeqa/core/target/ssh.py   | 7 +--
>  meta/lib/oeqa/runtime/cases/ltp.py | 2 +-
>  2 files changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/meta/lib/oeqa/core/target/ssh.py 
> b/meta/lib/oeqa/core/target/ssh.py
> index 090b40a814..aefb576805 100644
> --- a/meta/lib/oeqa/core/target/ssh.py
> +++ b/meta/lib/oeqa/core/target/ssh.py
> @@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
>  scpCmd = self.scp + [localSrc, remotePath]
>  return self._run(scpCmd, ignore_status=False)
> 
> -def copyFrom(self, remoteSrc, localDst):
> +def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
>  """
>  Copy file from target.
>  """
>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>  scpCmd = self.scp + [remotePath, localDst]
> -return self._run(scpCmd, ignore_status=False)
> +(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
> +if warn_on_failure and status:
> +self.logger.warning("Copy returned non-zero exit status %d:\n%s" 
> % (status, output))
> +return (status, output)
> 
>  def copyDirTo(self, localSrc, remoteDst):
>  """
> diff --git a/meta/lib/oeqa/runtime/cases/ltp.py 
> b/meta/lib/oeqa/runtime/cases/ltp.py
> index 6dc5ef22ad..da529ce482 100644
> --- a/meta/lib/oeqa/runtime/cases/ltp.py
> +++ b/meta/lib/oeqa/runtime/cases/ltp.py
> @@ -78,7 +78,7 @@ class LtpTest(LtpTestBase):
>  # copy nice log from DUT
>  dst = os.path.join(self.ltptest_log_dir, "%s" %  ltp_group )
>  remote_src = "/opt/ltp/results/%s" % ltp_group
> -(status, output) = self.target.copyFrom(remote_src, dst)
> +(status, output) = self.target.copyFrom(remote_src, dst, True)
>  msg = 'File could not be copied. Output: %s' % output
>  self.assertEqual(status, 0, msg=msg)

The idea of this change is so "the ltp tests continue to run" yet if I
understand that code, a copy failure will throw the assertEqual and
hence stop the test so we don't get all the other logging information?

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141246): 
https://lists.openembedded.org/g/openembedded-core/message/141246
Mute This Topic: https://lists.openembedded.org/mt/76013023/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCHv3] ltp: make copyFrom scp command non-fatal

[Matthew]

[YOCTO #13802]

Make the scp failure non-fatal so the ltp tests continue to run and
the rest of the logs will be available to see afterwards.

Signed-off-by: Mingde (Matthew) Zeng 
---
 meta/lib/oeqa/core/target/ssh.py   | 7 +--
 meta/lib/oeqa/runtime/cases/ltp.py | 2 +-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 090b40a814..aefb576805 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
 scpCmd = self.scp + [localSrc, remotePath]
 return self._run(scpCmd, ignore_status=False)

-def copyFrom(self, remoteSrc, localDst):
+def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
 """
 Copy file from target.
 """
 remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
 scpCmd = self.scp + [remotePath, localDst]
-return self._run(scpCmd, ignore_status=False)
+(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
+if warn_on_failure and status:
+self.logger.warning("Copy returned non-zero exit status %d:\n%s" % 
(status, output))
+return (status, output)

 def copyDirTo(self, localSrc, remoteDst):
 """
diff --git a/meta/lib/oeqa/runtime/cases/ltp.py 
b/meta/lib/oeqa/runtime/cases/ltp.py
index 6dc5ef22ad..da529ce482 100644
--- a/meta/lib/oeqa/runtime/cases/ltp.py
+++ b/meta/lib/oeqa/runtime/cases/ltp.py
@@ -78,7 +78,7 @@ class LtpTest(LtpTestBase):
 # copy nice log from DUT
 dst = os.path.join(self.ltptest_log_dir, "%s" %  ltp_group )
 remote_src = "/opt/ltp/results/%s" % ltp_group
-(status, output) = self.target.copyFrom(remote_src, dst)
+(status, output) = self.target.copyFrom(remote_src, dst, True)
 msg = 'File could not be copied. Output: %s' % output
 self.assertEqual(status, 0, msg=msg)

--
2.27.0
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141245): 
https://lists.openembedded.org/g/openembedded-core/message/141245
Mute This Topic: https://lists.openembedded.org/mt/76013023/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] [PATCHv2] ltp: make copyFrom scp command non-fatal

[Matthew]

Richard Purdie  writes:

> On Wed, 2020-08-05 at 13:15 -0400, Mingde (Matthew) Zeng wrote:
>> [YOCTO #13802]
>>
>> Make the scp failure non-fatal so the ltp tests continue to run and
>> the rest of the logs will be available to see afterwards.
>>
>> Signed-off-by: Mingde (Matthew) Zeng 
>> ---
>>  meta/lib/oeqa/core/target/ssh.py | 11 +++
>>  1 file changed, 7 insertions(+), 4 deletions(-)
>>
>> diff --git a/meta/lib/oeqa/core/target/ssh.py 
>> b/meta/lib/oeqa/core/target/ssh.py
>> index 090b40a814..e891a4bbfa 100644
>> --- a/meta/lib/oeqa/core/target/ssh.py
>> +++ b/meta/lib/oeqa/core/target/ssh.py
>> @@ -50,7 +50,7 @@ class OESSHTarget(OETarget):
>>  def stop(self, **kwargs):
>>  pass
>>
>> -def _run(self, command, timeout=None, ignore_status=True):
>> +def _run(self, command, timeout=None, ignore_status=True, 
>> ignore_error=False):
>>  """
>>  Runs command in target using SSHProcess.
>>  """
>> @@ -62,8 +62,11 @@ class OESSHTarget(OETarget):
>>   "" % (status, time.time() - starttime))
>>
>>  if status and not ignore_status:
>> -raise AssertionError("Command '%s' returned non-zero exit "
>> - "status %d:\n%s" % (command, status, 
>> output))
>> +if ignore_error:
>> +self.logger.warning("Command '%s' returned non-zero exit 
>> status %d:\n%s" % (command, status, output))
>> +else:
>> +raise AssertionError("Command '%s' returned non-zero exit "
>> + "status %d:\n%s" % (command, status, 
>> output))
>>
>>  return (status, output)
>>
>> @@ -113,7 +116,7 @@ class OESSHTarget(OETarget):
>>  """
>>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>>  scpCmd = self.scp + [remotePath, localDst]
>> -return self._run(scpCmd, ignore_status=False)
>> +return self._run(scpCmd, ignore_error=True)
>>
>>  def copyDirTo(self, localSrc, remoteDst):
>>  """
>
> That isn't what I meant, I mean something like:
>
> diff --git a/meta/lib/oeqa/core/target/ssh.py 
> b/meta/lib/oeqa/core/target/ssh.py
> index 090b40a8143..cbc56f2e0ad 100644
> --- a/meta/lib/oeqa/core/target/ssh.py
> +++ b/meta/lib/oeqa/core/target/ssh.py
> @@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
>  scpCmd = self.scp + [localSrc, remotePath]
>  return self._run(scpCmd, ignore_status=False)
>
> -def copyFrom(self, remoteSrc, localDst):
> +def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
>  """
>  Copy file from target.
>  """
>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>  scpCmd = self.scp + [remotePath, localDst]
> -return self._run(scpCmd, ignore_status=False)
> +(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
> +if warn_on_failure and not status:
> +self.logger.warning("Copy X returned non-zero exit status 
> %d:\n%s" % (XXX, status, output))
> +return (status, output)
>
>  def copyDirTo(self, localSrc, remoteDst):
>  """
>
> so that we enable this in the specific case og the copy we might accept 
> failure for.

This works too, my implementation is not as specific as this, however it 
enables warning when the optional parameter ignore_error=True, which is only 
specified in the copyFrom function.

I'll adjust my patch.

>
> Cheers,
>
> Richard


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141244): 
https://lists.openembedded.org/g/openembedded-core/message/141244
Mute This Topic: https://lists.openembedded.org/mt/76012000/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] [PATCHv2] ltp: make copyFrom scp command non-fatal

[Matthew]

Richard Purdie  writes:

> On Wed, 2020-08-05 at 13:15 -0400, Mingde (Matthew) Zeng wrote:
>> [YOCTO #13802]
>>
>> Make the scp failure non-fatal so the ltp tests continue to run and
>> the rest of the logs will be available to see afterwards.
>>
>> Signed-off-by: Mingde (Matthew) Zeng 
>> ---
>>  meta/lib/oeqa/core/target/ssh.py | 11 +++
>>  1 file changed, 7 insertions(+), 4 deletions(-)
>>
>> diff --git a/meta/lib/oeqa/core/target/ssh.py 
>> b/meta/lib/oeqa/core/target/ssh.py
>> index 090b40a814..e891a4bbfa 100644
>> --- a/meta/lib/oeqa/core/target/ssh.py
>> +++ b/meta/lib/oeqa/core/target/ssh.py
>> @@ -50,7 +50,7 @@ class OESSHTarget(OETarget):
>>  def stop(self, **kwargs):
>>  pass
>>
>> -def _run(self, command, timeout=None, ignore_status=True):
>> +def _run(self, command, timeout=None, ignore_status=True, 
>> ignore_error=False):
>>  """
>>  Runs command in target using SSHProcess.
>>  """
>> @@ -62,8 +62,11 @@ class OESSHTarget(OETarget):
>>   "" % (status, time.time() - starttime))
>>
>>  if status and not ignore_status:
>> -raise AssertionError("Command '%s' returned non-zero exit "
>> - "status %d:\n%s" % (command, status, 
>> output))
>> +if ignore_error:
>> +self.logger.warning("Command '%s' returned non-zero exit 
>> status %d:\n%s" % (command, status, output))
>> +else:
>> +raise AssertionError("Command '%s' returned non-zero exit "
>> + "status %d:\n%s" % (command, status, 
>> output))
>>
>>  return (status, output)
>>
>> @@ -113,7 +116,7 @@ class OESSHTarget(OETarget):
>>  """
>>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>>  scpCmd = self.scp + [remotePath, localDst]
>> -return self._run(scpCmd, ignore_status=False)
>> +return self._run(scpCmd, ignore_error=True)
>>
>>  def copyDirTo(self, localSrc, remoteDst):
>>  """
>
> That isn't what I meant, I mean something like:
>
> diff --git a/meta/lib/oeqa/core/target/ssh.py 
> b/meta/lib/oeqa/core/target/ssh.py
> index 090b40a8143..cbc56f2e0ad 100644
> --- a/meta/lib/oeqa/core/target/ssh.py
> +++ b/meta/lib/oeqa/core/target/ssh.py
> @@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
>  scpCmd = self.scp + [localSrc, remotePath]
>  return self._run(scpCmd, ignore_status=False)
>
> -def copyFrom(self, remoteSrc, localDst):
> +def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
>  """
>  Copy file from target.
>  """
>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>  scpCmd = self.scp + [remotePath, localDst]
> -return self._run(scpCmd, ignore_status=False)
> +(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
> +if warn_on_failure and not status:
> +self.logger.warning("Copy X returned non-zero exit status 
> %d:\n%s" % (XXX, status, output))
> +return (status, output)
>
>  def copyDirTo(self, localSrc, remoteDst):
>  """
>
> so that we enable this in the specific case og the copy we might accept 
> failure for.

This works too, my implementation is not as specific as this but it still 
enables warning only when the optional parameter ignore_error=True, which is 
what I specified in the copyFrom function only.

I'll adjust my patch.

>
> Cheers,
>
> Richard


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141243): 
https://lists.openembedded.org/g/openembedded-core/message/141243
Mute This Topic: https://lists.openembedded.org/mt/76012000/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/4] inetutils: Fix build with -fno-common

[Khem Raj]

Fixes errors like

telnetd/state.c:69: multiple definition of `not42'; 
utility.o:/usr/src/debug/inetutils/1.9.4-r0/build/telnetd/../../inetutils-1.9.4/telnetd/utility.c:66:
 first defined here
| clang-11: error: linker command failed with exit code 1 (use -v to see 
invocation)

Signed-off-by: Khem Raj 
---
 ...-multiple-definitions-of-errcatch-an.patch | 58 +++
 .../inetutils/inetutils_1.9.4.bb  |  1 +
 2 files changed, 59 insertions(+)
 create mode 100644 
meta/recipes-connectivity/inetutils/inetutils/0001-ftpd-telnetd-Fix-multiple-definitions-of-errcatch-an.patch

diff --git 
a/meta/recipes-connectivity/inetutils/inetutils/0001-ftpd-telnetd-Fix-multiple-definitions-of-errcatch-an.patch
 
b/meta/recipes-connectivity/inetutils/inetutils/0001-ftpd-telnetd-Fix-multiple-definitions-of-errcatch-an.patch
new file mode 100644
index 00..49d319f59d
--- /dev/null
+++ 
b/meta/recipes-connectivity/inetutils/inetutils/0001-ftpd-telnetd-Fix-multiple-definitions-of-errcatch-an.patch
@@ -0,0 +1,58 @@
+From 7d39930468e272c740b0eed3c7e5b7fb3abf29e8 Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Wed, 5 Aug 2020 10:36:22 -0700
+Subject: [PATCH] ftpd,telnetd: Fix multiple definitions of errcatch and not42
+
+This helps fix build failures when -fno-common option is used
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj 
+
+Signed-off-by: Khem Raj 
+---
+ ftpd/extern.h | 2 +-
+ ftpd/ftpcmd.c | 1 +
+ telnetd/utility.c | 2 +-
+ 3 files changed, 3 insertions(+), 2 deletions(-)
+
+diff --git a/ftpd/extern.h b/ftpd/extern.h
+index ab33cf3..91dbbee 100644
+--- a/ftpd/extern.h
 b/ftpd/extern.h
+@@ -90,7 +90,7 @@ extern void user (const char *);
+ extern char *sgetsave (const char *);
+ 
+ /* Exported from ftpd.c.  */
+-jmp_buf errcatch;
++extern jmp_buf errcatch;
+ extern struct sockaddr_storage data_dest;
+ extern socklen_t data_dest_len;
+ extern struct sockaddr_storage his_addr;
+diff --git a/ftpd/ftpcmd.c b/ftpd/ftpcmd.c
+index beb1f06..d272e9d 100644
+--- a/ftpd/ftpcmd.c
 b/ftpd/ftpcmd.c
+@@ -106,6 +106,7 @@
+ #endif
+ 
+ off_t restart_point;
++jmp_buf errcatch;
+ 
+ static char cbuf[512];   /* Command Buffer.  */
+ static char *fromname;
+diff --git a/telnetd/utility.c b/telnetd/utility.c
+index e7ffb8e..46bf91e 100644
+--- a/telnetd/utility.c
 b/telnetd/utility.c
+@@ -63,7 +63,7 @@ static int ncc;
+ static char ptyibuf[BUFSIZ], *ptyip;
+ static int pcc;
+ 
+-int not42;
++extern int not42;
+ 
+ static int
+ readstream (int p, char *ibuf, int bufsize)
+-- 
+2.28.0
+
diff --git a/meta/recipes-connectivity/inetutils/inetutils_1.9.4.bb 
b/meta/recipes-connectivity/inetutils/inetutils_1.9.4.bb
index cc9410b94e..adf6d4414e 100644
--- a/meta/recipes-connectivity/inetutils/inetutils_1.9.4.bb
+++ b/meta/recipes-connectivity/inetutils/inetutils_1.9.4.bb
@@ -23,6 +23,7 @@ SRC_URI = "${GNU_MIRROR}/inetutils/inetutils-${PV}.tar.gz \
file://inetutils-only-check-pam_appl.h-when-pam-enabled.patch \
file://0001-rcp-fix-to-work-with-large-files.patch \
file://fix-buffer-fortify-tfpt.patch \
+   
file://0001-ftpd-telnetd-Fix-multiple-definitions-of-errcatch-an.patch \
 "
 
 SRC_URI[md5sum] = "04852c26c47cc8c6b825f2b74f191f52"
-- 
2.28.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141239): 
https://lists.openembedded.org/g/openembedded-core/message/141239
Mute This Topic: https://lists.openembedded.org/mt/76012621/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 4/4] distcc: Fix build with -fno-common

[Khem Raj]

Signed-off-by: Khem Raj 
---
 meta/recipes-devtools/distcc/distcc_3.3.3.bb  |  1 +
 ...cc-10-which-defaults-to-fno-common-c.patch | 34 +++
 2 files changed, 35 insertions(+)
 create mode 100644 
meta/recipes-devtools/distcc/files/0001-Fix-build-with-gcc-10-which-defaults-to-fno-common-c.patch

diff --git a/meta/recipes-devtools/distcc/distcc_3.3.3.bb 
b/meta/recipes-devtools/distcc/distcc_3.3.3.bb
index c52f136be8..d92b9c3046 100644
--- a/meta/recipes-devtools/distcc/distcc_3.3.3.bb
+++ b/meta/recipes-devtools/distcc/distcc_3.3.3.bb
@@ -17,6 +17,7 @@ RRECOMMENDS_${PN}-server = "avahi-daemon"
 SRC_URI = "git://github.com/distcc/distcc.git \
file://fix-gnome.patch \
file://separatebuilddir.patch \
+   
file://0001-Fix-build-with-gcc-10-which-defaults-to-fno-common-c.patch \
file://default \
file://distcc \
file://distcc.service"
diff --git 
a/meta/recipes-devtools/distcc/files/0001-Fix-build-with-gcc-10-which-defaults-to-fno-common-c.patch
 
b/meta/recipes-devtools/distcc/files/0001-Fix-build-with-gcc-10-which-defaults-to-fno-common-c.patch
new file mode 100644
index 00..86c07c11f8
--- /dev/null
+++ 
b/meta/recipes-devtools/distcc/files/0001-Fix-build-with-gcc-10-which-defaults-to-fno-common-c.patch
@@ -0,0 +1,34 @@
+From 9853086579538fe06bb647a6fc2dddfc7967 Mon Sep 17 00:00:00 2001
+From: Romain Geissler 
+Date: Mon, 27 Jan 2020 09:28:43 +
+Subject: [PATCH] Fix build with gcc 10 which defaults to -fno-common (cf
+ https://gcc.gnu.org/gcc-10/porting_to.html)
+
+This fixes the following link error I see when I use the latest gcc 10
+git branch:
+/opt/1A/toolchain/x86_64-v20.0.7/lib/gcc/x86_64-1a-linux-gnu/10.0.1/../../../../x86_64-1a-linux-gnu/bin/ld:
 src/serve.o:(.bss+0x0): multiple definition of `stats_text'; 
src/prefork.o:(.bss+0x0): first defined here
+/opt/1A/toolchain/x86_64-v20.0.7/lib/gcc/x86_64-1a-linux-gnu/10.0.1/../../../../x86_64-1a-linux-gnu/bin/ld:
 src/stats.o:(.data+0x20): multiple definition of `stats_text'; 
src/prefork.o:(.bss+0x0): first defined here
+collect2: error: ld returned 1 exit status
+
+Upstream-Status: Backport [https://github.com/distcc/distcc/pull/373]
+Signed-off-by: Khem Raj 
+---
+ src/stats.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/stats.h b/src/stats.h
+index 9bde285..74d4690 100644
+--- a/src/stats.h
 b/src/stats.h
+@@ -33,7 +33,7 @@ enum stats_e { STATS_TCP_ACCEPT, STATS_REJ_BAD_REQ, 
STATS_REJ_OVERLOAD,
+ STATS_COMPILE_OK, STATS_COMPILE_ERROR, STATS_COMPILE_TIMEOUT,
+ STATS_CLI_DISCONN, STATS_OTHER, STATS_ENUM_MAX };
+ 
+-const char *stats_text[20];
++extern const char *stats_text[20];
+ 
+ int  dcc_stats_init(void);
+ void dcc_stats_init_kid(void);
+-- 
+2.28.0
+
-- 
2.28.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141242): 
https://lists.openembedded.org/g/openembedded-core/message/141242
Mute This Topic: https://lists.openembedded.org/mt/76012625/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/4] libomxil: Use -fcommon compiler option

[Khem Raj]

This ensures that when compiler defaults become -fno-common it still
keeps compiling

Signed-off-by: Khem Raj 
---
 meta/recipes-multimedia/libomxil/libomxil_0.9.3.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-multimedia/libomxil/libomxil_0.9.3.bb 
b/meta/recipes-multimedia/libomxil/libomxil_0.9.3.bb
index 271c2a30a3..2061c280e4 100644
--- a/meta/recipes-multimedia/libomxil/libomxil_0.9.3.bb
+++ b/meta/recipes-multimedia/libomxil/libomxil_0.9.3.bb
@@ -26,6 +26,8 @@ EXTRA_OECONF += "--disable-doc --disable-Werror"
 
 PROVIDES += "virtual/libomxil"
 
+CFLAGS += "-fcommon"
+
 #
 # The .so files under ${libdir}/bellagio are not intended to be versioned and 
symlinked.
 # Make sure they get packaged in the main package.
-- 
2.28.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141240): 
https://lists.openembedded.org/g/openembedded-core/message/141240
Mute This Topic: https://lists.openembedded.org/mt/76012623/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/4] kexec-tools: Fix build with -fno-common

[Khem Raj]

Signed-off-by: Khem Raj 
---
 ...0001-kexec-Fix-build-with-fno-common.patch | 43 +++
 .../kexec/kexec-tools_2.0.20.bb   |  1 +
 2 files changed, 44 insertions(+)
 create mode 100644 
meta/recipes-kernel/kexec/kexec-tools/0001-kexec-Fix-build-with-fno-common.patch

diff --git 
a/meta/recipes-kernel/kexec/kexec-tools/0001-kexec-Fix-build-with-fno-common.patch
 
b/meta/recipes-kernel/kexec/kexec-tools/0001-kexec-Fix-build-with-fno-common.patch
new file mode 100644
index 00..355053363e
--- /dev/null
+++ 
b/meta/recipes-kernel/kexec/kexec-tools/0001-kexec-Fix-build-with-fno-common.patch
@@ -0,0 +1,43 @@
+From c5fec6d6368b4103557deb710150119dca438544 Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Wed, 5 Aug 2020 10:46:39 -0700
+Subject: [PATCH] kexec: Fix build with -fno-common
+
+Ensure that my_debug is not doubly defined
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj 
+---
+ kexec/arch/ppc64/kexec-elf-ppc64.c | 2 --
+ kexec/fs2dt.h  | 2 +-
+ 2 files changed, 1 insertion(+), 3 deletions(-)
+
+diff --git a/kexec/arch/ppc64/kexec-elf-ppc64.c 
b/kexec/arch/ppc64/kexec-elf-ppc64.c
+index 3510b70..695b8b0 100644
+--- a/kexec/arch/ppc64/kexec-elf-ppc64.c
 b/kexec/arch/ppc64/kexec-elf-ppc64.c
+@@ -44,8 +44,6 @@
+ uint64_t initrd_base, initrd_size;
+ unsigned char reuse_initrd = 0;
+ const char *ramdisk;
+-/* Used for enabling printing message from purgatory code */
+-int my_debug = 0;
+ 
+ int elf_ppc64_probe(const char *buf, off_t len)
+ {
+diff --git a/kexec/fs2dt.h b/kexec/fs2dt.h
+index 7633273..fe24931 100644
+--- a/kexec/fs2dt.h
 b/kexec/fs2dt.h
+@@ -30,7 +30,7 @@ extern struct bootblock bb[1];
+ 
+ /* Used for enabling printing message from purgatory code
+  * Only has implemented for PPC64 */
+-int my_debug;
++extern int my_debug;
+ extern int dt_no_old_root;
+ 
+ void reserve(unsigned long long where, unsigned long long length);
+-- 
+2.28.0
+
diff --git a/meta/recipes-kernel/kexec/kexec-tools_2.0.20.bb 
b/meta/recipes-kernel/kexec/kexec-tools_2.0.20.bb
index 871b36440f..1e81ecc2c7 100644
--- a/meta/recipes-kernel/kexec/kexec-tools_2.0.20.bb
+++ b/meta/recipes-kernel/kexec/kexec-tools_2.0.20.bb
@@ -20,6 +20,7 @@ SRC_URI = 
"${KERNELORG_MIRROR}/linux/utils/kernel/kexec/kexec-tools-${PV}.tar.gz
file://0005-Disable-PIE-during-link.patch \
file://0006-kexec-arm-undefine-__NR_kexec_file_load-for-arm.patch \
file://0007-kexec-un-break-the-build-on-32-bit-x86.patch \
+   file://0001-kexec-Fix-build-with-fno-common.patch \
"
 
 SRC_URI[md5sum] = "46724b67f32501c5d3e778161347cad9"
-- 
2.28.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141241): 
https://lists.openembedded.org/g/openembedded-core/message/141241
Mute This Topic: https://lists.openembedded.org/mt/76012624/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] [PATCHv2] ltp: make copyFrom scp command non-fatal

[Richard Purdie]

On Wed, 2020-08-05 at 13:15 -0400, Mingde (Matthew) Zeng wrote:
> [YOCTO #13802]
> 
> Make the scp failure non-fatal so the ltp tests continue to run and
> the rest of the logs will be available to see afterwards.
> 
> Signed-off-by: Mingde (Matthew) Zeng 
> ---
>  meta/lib/oeqa/core/target/ssh.py | 11 +++
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/lib/oeqa/core/target/ssh.py 
> b/meta/lib/oeqa/core/target/ssh.py
> index 090b40a814..e891a4bbfa 100644
> --- a/meta/lib/oeqa/core/target/ssh.py
> +++ b/meta/lib/oeqa/core/target/ssh.py
> @@ -50,7 +50,7 @@ class OESSHTarget(OETarget):
>  def stop(self, **kwargs):
>  pass
> 
> -def _run(self, command, timeout=None, ignore_status=True):
> +def _run(self, command, timeout=None, ignore_status=True, 
> ignore_error=False):
>  """
>  Runs command in target using SSHProcess.
>  """
> @@ -62,8 +62,11 @@ class OESSHTarget(OETarget):
>   "" % (status, time.time() - starttime))
> 
>  if status and not ignore_status:
> -raise AssertionError("Command '%s' returned non-zero exit "
> - "status %d:\n%s" % (command, status, 
> output))
> +if ignore_error:
> +self.logger.warning("Command '%s' returned non-zero exit 
> status %d:\n%s" % (command, status, output))
> +else:
> +raise AssertionError("Command '%s' returned non-zero exit "
> + "status %d:\n%s" % (command, status, 
> output))
> 
>  return (status, output)
> 
> @@ -113,7 +116,7 @@ class OESSHTarget(OETarget):
>  """
>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>  scpCmd = self.scp + [remotePath, localDst]
> -return self._run(scpCmd, ignore_status=False)
> +return self._run(scpCmd, ignore_error=True)
> 
>  def copyDirTo(self, localSrc, remoteDst):
>  """

That isn't what I meant, I mean something like:

diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 090b40a8143..cbc56f2e0ad 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -107,13 +107,16 @@ class OESSHTarget(OETarget):
 scpCmd = self.scp + [localSrc, remotePath]
 return self._run(scpCmd, ignore_status=False)
 
-def copyFrom(self, remoteSrc, localDst):
+def copyFrom(self, remoteSrc, localDst, warn_on_failure=False):
 """
 Copy file from target.
 """
 remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
 scpCmd = self.scp + [remotePath, localDst]
-return self._run(scpCmd, ignore_status=False)
+(status, output) = self._run(scpCmd, ignore_status=warn_on_failure)
+if warn_on_failure and not status:
+self.logger.warning("Copy X returned non-zero exit status %d:\n%s" 
% (XXX, status, output))
+return (status, output)
 
 def copyDirTo(self, localSrc, remoteDst):
 """

so that we enable this in the specific case og the copy we might accept failure 
for.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141238): 
https://lists.openembedded.org/g/openembedded-core/message/141238
Mute This Topic: https://lists.openembedded.org/mt/76012000/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] [PATCHv2] ltp: make copyFrom scp command non-fatal

[Matthew]

[YOCTO #13802]

Make the scp failure non-fatal so the ltp tests continue to run and
the rest of the logs will be available to see afterwards.

Signed-off-by: Mingde (Matthew) Zeng 
---
 meta/lib/oeqa/core/target/ssh.py | 11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 090b40a814..e891a4bbfa 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -50,7 +50,7 @@ class OESSHTarget(OETarget):
 def stop(self, **kwargs):
 pass

-def _run(self, command, timeout=None, ignore_status=True):
+def _run(self, command, timeout=None, ignore_status=True, 
ignore_error=False):
 """
 Runs command in target using SSHProcess.
 """
@@ -62,8 +62,11 @@ class OESSHTarget(OETarget):
  "" % (status, time.time() - starttime))

 if status and not ignore_status:
-raise AssertionError("Command '%s' returned non-zero exit "
- "status %d:\n%s" % (command, status, output))
+if ignore_error:
+self.logger.warning("Command '%s' returned non-zero exit 
status %d:\n%s" % (command, status, output))
+else:
+raise AssertionError("Command '%s' returned non-zero exit "
+ "status %d:\n%s" % (command, status, 
output))

 return (status, output)

@@ -113,7 +116,7 @@ class OESSHTarget(OETarget):
 """
 remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
 scpCmd = self.scp + [remotePath, localDst]
-return self._run(scpCmd, ignore_status=False)
+return self._run(scpCmd, ignore_error=True)

 def copyDirTo(self, localSrc, remoteDst):
 """
--
2.27.0
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141237): 
https://lists.openembedded.org/g/openembedded-core/message/141237
Mute This Topic: https://lists.openembedded.org/mt/76012000/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] ltp: make copyFrom scp command non-fatal

[Matthew]

Richard Purdie  writes:

> On Wed, 2020-08-05 at 11:50 -0400, Matthew wrote:
>> [YOCTO #13802]
>>
>> Make the scp failure non-fatal so the ltp tests continue to run and
>> the rest of the logs will be available to see afterwards.
>>
>> Signed-off-by: Mingde (Matthew) Zeng 
>> ---
>>  meta/lib/oeqa/core/target/ssh.py | 11 +++
>>  1 file changed, 7 insertions(+), 4 deletions(-)
>>
>> diff --git a/meta/lib/oeqa/core/target/ssh.py 
>> b/meta/lib/oeqa/core/target/ssh.py
>> index 090b40a814..1b78e830d4 100644
>> --- a/meta/lib/oeqa/core/target/ssh.py
>> +++ b/meta/lib/oeqa/core/target/ssh.py
>> @@ -61,9 +61,12 @@ class OESSHTarget(OETarget):
>>  self.logger.debug("[Command returned '%d' after %.2f seconds]"
>>   "" % (status, time.time() - starttime))
>>
>> -if status and not ignore_status:
>> -raise AssertionError("Command '%s' returned non-zero exit "
>> - "status %d:\n%s" % (command, status, 
>> output))
>> +if status:
>> +if not ignore_status:
>> +raise AssertionError("Command '%s' returned non-zero exit "
>> + "status %d:\n%s" % (command, status, 
>> output))
>> +else:
>> +self.logger.warning("Command '%s' returned non-zero exit 
>> status %d:\n%s" % (command, status, output))
>>
>>  return (status, output)
>>
>> @@ -113,7 +116,7 @@ class OESSHTarget(OETarget):
>>  """
>>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>>  scpCmd = self.scp + [remotePath, localDst]
>> -return self._run(scpCmd, ignore_status=False)
>> +return self._run(scpCmd, ignore_status=True)
>>
>>  def copyDirTo(self, localSrc, remoteDst):
>>  """
>
> I'm a little worried that these warnings may appear from places we
> don't expect them. I can run the test on the autobuilder and see if
> other tests trigger this I guess, I wondered if you'd looked into that?
>

You're right, the better way might be adding an additional optional parameter 
ignore_error, so it won't break any existing behavior.

I'll quickly create a new patch and let you have a look.

> An alternative may be to set ignore_status=True but check the return
> code at a higher level and issue the warning there?
>
> Cheers,
> Richard


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141236): 
https://lists.openembedded.org/g/openembedded-core/message/141236
Mute This Topic: https://lists.openembedded.org/mt/76010223/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] ltp: make copyFrom scp command non-fatal

[Richard Purdie]

On Wed, 2020-08-05 at 11:50 -0400, Matthew wrote:
> [YOCTO #13802]
> 
> Make the scp failure non-fatal so the ltp tests continue to run and
> the rest of the logs will be available to see afterwards.
> 
> Signed-off-by: Mingde (Matthew) Zeng 
> ---
>  meta/lib/oeqa/core/target/ssh.py | 11 +++
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/lib/oeqa/core/target/ssh.py 
> b/meta/lib/oeqa/core/target/ssh.py
> index 090b40a814..1b78e830d4 100644
> --- a/meta/lib/oeqa/core/target/ssh.py
> +++ b/meta/lib/oeqa/core/target/ssh.py
> @@ -61,9 +61,12 @@ class OESSHTarget(OETarget):
>  self.logger.debug("[Command returned '%d' after %.2f seconds]"
>   "" % (status, time.time() - starttime))
>  
> -if status and not ignore_status:
> -raise AssertionError("Command '%s' returned non-zero exit "
> - "status %d:\n%s" % (command, status, 
> output))
> +if status:
> +if not ignore_status:
> +raise AssertionError("Command '%s' returned non-zero exit "
> + "status %d:\n%s" % (command, status, 
> output))
> +else:
> +self.logger.warning("Command '%s' returned non-zero exit 
> status %d:\n%s" % (command, status, output))
>  
>  return (status, output)
>  
> @@ -113,7 +116,7 @@ class OESSHTarget(OETarget):
>  """
>  remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
>  scpCmd = self.scp + [remotePath, localDst]
> -return self._run(scpCmd, ignore_status=False)
> +return self._run(scpCmd, ignore_status=True)
>  
>  def copyDirTo(self, localSrc, remoteDst):
>  """

I'm a little worried that these warnings may appear from places we
don't expect them. I can run the test on the autobuilder and see if
other tests trigger this I guess, I wondered if you'd looked into that?

An alternative may be to set ignore_status=True but check the return
code at a higher level and issue the warning there?

Cheers,
Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141235): 
https://lists.openembedded.org/g/openembedded-core/message/141235
Mute This Topic: https://lists.openembedded.org/mt/76010223/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] kernel-devicetree: Fix intermittent build failures caused by DTB builds

[Tyler Hicks]

Fix a build-time race condition that resulted in intermittent build
failures in the do_assemble_fitimage task. The race condition involved
the do_assemble_fitimage task reading the vmlinux file while the
do_compile_kernelmodules task was re-writing the vmlinux file.

This can be seen with an aarch64 image build that uses a 5.4 based
kernel and sets KERNEL_DEVICETREE. The problem is that the do_compile
snippet that the kernel-devicetree class appends did not specify the
full kernel build environment when building the DTB(s) from the kernel
tree. This resulted in CONFIG_CC_CAN_LINK=y being removed from the
kernel config file just before the do_compile task completed. The
CONFIG_CC_CAN_LINK=y line was then re-inserted into the kernel config
file as part of the do_compile_kernelmodules task.

In some cases, this resulted in the do_compile_kernelmodules task to
re-link vmlinux which sometimes occured at the same time that the
do_assemble_fitimage task was attempting to use vmlinux. The
do_assemble_fitimage task would fail with the following error message:

 aarch64-poky-linux-objcopy:vmlinux: file format not recognized

We can use the pine-a64-lts machine, from the meta-pine64 layer, to show
that the kernel config file was changed between do_compile and
do_compile_kernelmodules:

 $ 
C=tmp/work/pine_a64_lts-poky-linux/linux-pine64/5.7+gitAUTOINC+ae03bade3b-r0/linux-pine_a64_lts-standard-build/.config
 $ bitbake -c do_kernel_configcheck virtual/kernel
 ...
 $ md5sum $C; grep CC_CAN_LINK $C
 32b133cf8a749a91f698a7ca8616c84f  ...
 CONFIG_CC_CAN_LINK=y

 $ bitbake -c do_compile virtual/kernel
 ...
 $ md5sum $C; grep CC_CAN_LINK $C
 2fd2ec2a66eecc329dcb5afaf005eada  ...

 $ bitbake -c do_compile_kernelmodules virtual/kernel
 ...
 $ md5sum $C; grep CC_CAN_LINK $C
 32b133cf8a749a91f698a7ca8616c84f  ...
 CONFIG_CC_CAN_LINK=y

With this change, the do_compile snippet appended by the
kernel-devicetree class does not modify the kernel config. The kernel
config is unchanged across the do_compile and do_compile_kernelmodules
tasks and do_compile_kernelmodules will not attempt to re-link vmlinux.

Signed-off-by: Tyler Hicks 
---
 meta/classes/kernel-devicetree.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/kernel-devicetree.bbclass 
b/meta/classes/kernel-devicetree.bbclass
index 522c46575d12..81dda8003f46 100644
--- a/meta/classes/kernel-devicetree.bbclass
+++ b/meta/classes/kernel-devicetree.bbclass
@@ -52,7 +52,7 @@ do_configure_append() {
 do_compile_append() {
for dtbf in ${KERNEL_DEVICETREE}; do
dtb=`normalize_dtb "$dtbf"`
-   oe_runmake $dtb
+   oe_runmake $dtb CC="${KERNEL_CC} $cc_extra " LD="${KERNEL_LD}" 
${KERNEL_EXTRA_ARGS}
done
 }
 
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141234): 
https://lists.openembedded.org/g/openembedded-core/message/141234
Mute This Topic: https://lists.openembedded.org/mt/76011593/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] libunwind: Use -fcommon option to compile

[Khem Raj]

On Wed, Aug 5, 2020 at 2:26 AM Adrian Bunk  wrote:
>
> On Tue, Aug 04, 2020 at 11:14:48PM -0700, Khem Raj wrote:
> > defaults for gcc is to use -fno-common this ensures that it keeps using
> > -fcommon until the real issue is fixed
> >
> > Fixes
> > src/arm/Ginit.c:60: multiple definition of `_U_dyn_info_list'; 
> > mi/.libs/dyn-info-list.o:/usr/src/debug/libunwind/1.4.0-r0/build/src/../../libunwind-1.4.0/src/mi/dyn-info-list.c:28:
> >  first defined here
> >...
>
> The upstream fix is
> https://github.com/libunwind/libunwind/commit/29e17d8d2ccbca07c423e3089a6d5ae8a1c9cb6e
>

Thanks for suggesting this, I have sent a v2

> cu
> Adrian
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141233): 
https://lists.openembedded.org/g/openembedded-core/message/141233
Mute This Topic: https://lists.openembedded.org/mt/76002561/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] libunwind: Backport a fix for -fno-common option to compile

[Khem Raj]

defaults for gcc is to use -fno-common this ensures that it keeps
building with gcc -fno-common

Fixes
src/arm/Ginit.c:60: multiple definition of `_U_dyn_info_list'; 
mi/.libs/dyn-info-list.o:/usr/src/debug/libunwind/1.4.0-r0/build/src/../../libunwind-1.4.0/src/mi/dyn-info-list.c:28:
 first defined here

Signed-off-by: Khem Raj 
---
v2: Use a backported patch instead of using -fcommon

 ...0001-Fix-compilation-with-fno-common.patch | 448 ++
 .../libunwind/libunwind_1.4.0.bb  |   1 +
 2 files changed, 449 insertions(+)
 create mode 100644 
meta/recipes-support/libunwind/libunwind/0001-Fix-compilation-with-fno-common.patch

diff --git 
a/meta/recipes-support/libunwind/libunwind/0001-Fix-compilation-with-fno-common.patch
 
b/meta/recipes-support/libunwind/libunwind/0001-Fix-compilation-with-fno-common.patch
new file mode 100644
index 00..e11487fe97
--- /dev/null
+++ 
b/meta/recipes-support/libunwind/libunwind/0001-Fix-compilation-with-fno-common.patch
@@ -0,0 +1,448 @@
+From 00d18b21dcb9723c4f13889a39a760a654782370 Mon Sep 17 00:00:00 2001
+From: Yichao Yu 
+Date: Tue, 31 Mar 2020 00:43:32 -0400
+Subject: [PATCH] Fix compilation with -fno-common.
+
+Making all other archs consistent with IA64 which should not have this problem.
+Also move the FIXME to the correct place.
+
+Also add some minimum comments about this...
+
+Upstream-Status: Backport [https://github.com/libunwind/libunwind/pull/166]
+Signed-off-by: Khem Raj 
+---
+ src/aarch64/Ginit.c| 15 +++
+ src/arm/Ginit.c| 15 +++
+ src/coredump/_UPT_get_dyn_info_list_addr.c |  5 +
+ src/hppa/Ginit.c   | 15 +++
+ src/ia64/Ginit.c   |  1 +
+ src/mi/Gfind_dynamic_proc_info.c   |  1 +
+ src/mips/Ginit.c   | 15 +++
+ src/ppc32/Ginit.c  | 11 +++
+ src/ppc64/Ginit.c  | 11 +++
+ src/ptrace/_UPT_get_dyn_info_list_addr.c   |  5 +
+ src/s390x/Ginit.c  | 15 +++
+ src/sh/Ginit.c | 15 +++
+ src/tilegx/Ginit.c | 15 +++
+ src/x86/Ginit.c| 15 +++
+ src/x86_64/Ginit.c | 15 +++
+ 15 files changed, 89 insertions(+), 80 deletions(-)
+
+diff --git a/src/aarch64/Ginit.c b/src/aarch64/Ginit.c
+index dec235c..3538976 100644
+--- a/src/aarch64/Ginit.c
 b/src/aarch64/Ginit.c
+@@ -61,13 +61,6 @@ tdep_uc_addr (unw_tdep_context_t *uc, int reg)
+ 
+ # endif /* UNW_LOCAL_ONLY */
+ 
+-HIDDEN unw_dyn_info_list_t _U_dyn_info_list;
+-
+-/* XXX fix me: there is currently no way to locate the dyn-info list
+-   by a remote unwinder.  On ia64, this is done via a special
+-   unwind-table entry.  Perhaps something similar can be done with
+-   DWARF2 unwind info.  */
+-
+ static void
+ put_unwind_info (unw_addr_space_t as, unw_proc_info_t *proc_info, void *arg)
+ {
+@@ -78,7 +71,13 @@ static int
+ get_dyn_info_list_addr (unw_addr_space_t as, unw_word_t *dyn_info_list_addr,
+ void *arg)
+ {
+-  *dyn_info_list_addr = (unw_word_t) &_U_dyn_info_list;
++#ifndef UNW_LOCAL_ONLY
++# pragma weak _U_dyn_info_list_addr
++  if (!_U_dyn_info_list_addr)
++return -UNW_ENOINFO;
++#endif
++  // Access the `_U_dyn_info_list` from `LOCAL_ONLY` library, i.e. 
libunwind.so.
++  *dyn_info_list_addr = _U_dyn_info_list_addr ();
+   return 0;
+ }
+ 
+diff --git a/src/arm/Ginit.c b/src/arm/Ginit.c
+index 2720d06..0bac0d7 100644
+--- a/src/arm/Ginit.c
 b/src/arm/Ginit.c
+@@ -57,18 +57,17 @@ tdep_uc_addr (unw_tdep_context_t *uc, int reg)
+ 
+ # endif /* UNW_LOCAL_ONLY */
+ 
+-HIDDEN unw_dyn_info_list_t _U_dyn_info_list;
+-
+-/* XXX fix me: there is currently no way to locate the dyn-info list
+-   by a remote unwinder.  On ia64, this is done via a special
+-   unwind-table entry.  Perhaps something similar can be done with
+-   DWARF2 unwind info.  */
+-
+ static int
+ get_dyn_info_list_addr (unw_addr_space_t as, unw_word_t *dyn_info_list_addr,
+ void *arg)
+ {
+-  *dyn_info_list_addr = (unw_word_t) &_U_dyn_info_list;
++#ifndef UNW_LOCAL_ONLY
++# pragma weak _U_dyn_info_list_addr
++  if (!_U_dyn_info_list_addr)
++return -UNW_ENOINFO;
++#endif
++  // Access the `_U_dyn_info_list` from `LOCAL_ONLY` library, i.e. 
libunwind.so.
++  *dyn_info_list_addr = _U_dyn_info_list_addr ();
+   return 0;
+ }
+ 
+diff --git a/src/coredump/_UPT_get_dyn_info_list_addr.c 
b/src/coredump/_UPT_get_dyn_info_list_addr.c
+index 0d11905..739ed05 100644
+--- a/src/coredump/_UPT_get_dyn_info_list_addr.c
 b/src/coredump/_UPT_get_dyn_info_list_addr.c
+@@ -74,6 +74,11 @@ get_list_addr (unw_addr_space_t as, unw_word_t *dil_addr, 
void *arg,
+ 
+ #else
+ 
++/* XXX fix me: there is currently no way to

Re: [OE-core] [PATCHv2] ltp: remove OOM tests from runtest/mm

[Matthew]

Richard Purdie  writes:

> On Wed, 2020-07-29 at 09:09 -0400, Matthew wrote:
>> Fixes [YOCTO #13802]
>
> Thanks, we can reference the bug but I don't think we can yet claim it
> fixes it.
>
> The missing log issue happened again:
> https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/908
> so I think there are multiple issues at play here.
>
> Is there any way to have the logging logged to disk straight away
> rather than stuck internally in oeqa's logging buffers? That way we
> could see which tests had run before a hang?

Ltp comes with zoo file already, is it what you're looking for?

>
> Also, could we make the scp failure non-fatal, maybe a warning so that
> when it fails we can look at the rest of the logs?

I submitted a patch that I think should achieve this idea, please have a look.

>
> Cheers,
>
> Richard
>
>> Remove the OOM tests, since they might cause oeqa ssh
>> connection lost.
>>
>> Signed-off-by: Mingde (Matthew) Zeng 
>> ---
>>  ...001-Remove-OOM-tests-from-runtest-mm.patch | 33
>> +++
>>  meta/recipes-extended/ltp/ltp_20200515.bb |  1 +
>>  2 files changed, 34 insertions(+)
>>  create mode 100644 meta/recipes-extended/ltp/ltp/0001-Remove-OOM-
>> tests-from-runtest-mm.patch
>>
>> diff --git a/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-
>> from-runtest-mm.patch b/meta/recipes-extended/ltp/ltp/0001-Remove-
>> OOM-tests-from-runtest-mm.patch
>> new file mode 100644
>> index 00..2d8262be78
>> --- /dev/null
>> +++ b/meta/recipes-extended/ltp/ltp/0001-Remove-OOM-tests-from-
>> runtest-mm.patch
>> @@ -0,0 +1,33 @@
>> +From 373b21dcf7a2879c27cc3dbeac5d83f5d45d22ae Mon Sep 17 00:00:00
>> 2001
>> +From: "Mingde (Matthew) Zeng" 
>> +Date: Wed, 29 Jul 2020 08:47:09 -0400
>> +Subject: [PATCH] Remove OOM tests from runtest/mm
>> +
>> +Disable OOM tests, as they might cause oeqa ssh connection lost
>> +
>> +Upstream-Status: Inappropriate [oe-core specific]
>> +Signed-off-by: Mingde (Matthew) Zeng 
>> +---
>> + runtest/mm | 6 --
>> + 1 file changed, 6 deletions(-)
>> +
>> +diff --git a/runtest/mm b/runtest/mm
>> +index 4701a14bd..2f9888ec6 100644
>> +--- a/runtest/mm
>>  b/runtest/mm
>> +@@ -75,12 +75,6 @@ ksm06_2 ksm06 -n 1
>> +
>> + cpuset01 cpuset01
>> +
>> +-oom01 oom01
>> +-oom02 oom02
>> +-oom03 oom03
>> +-oom04 oom04
>> +-oom05 oom05
>> +-
>> + swapping01 swapping01 -i 5
>> +
>> + thp01 thp01 -I 120
>> +--
>> +2.27.0
>> +
>> diff --git a/meta/recipes-extended/ltp/ltp_20200515.bb
>> b/meta/recipes-extended/ltp/ltp_20200515.bb
>> index ece3acf0f9..0c7044d044 100644
>> --- a/meta/recipes-extended/ltp/ltp_20200515.bb
>> +++ b/meta/recipes-extended/ltp/ltp_20200515.bb
>> @@ -37,6 +37,7 @@ SRC_URI = "git://github.com/linux-test-
>> project/ltp.git \
>> file://0001-ptrace01-Fix-missing-format-string.patch \
>> file://0001-sigwaitinfo-Do-not-run-invalid-undefined-
>> test-cases.patch \
>> file://0001-syscalls-copy_file_range02-Expect-EFBIG-in-
>> subcase-m.patch \
>> +   file://0001-Remove-OOM-tests-from-runtest-mm.patch \
>> "
>>
>>  S = "${WORKDIR}/git"
>> --
>> 2.27.0
>> 


--
Mingde (Matthew) Zeng
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141231): 
https://lists.openembedded.org/g/openembedded-core/message/141231
Mute This Topic: https://lists.openembedded.org/mt/75864108/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] ltp: make copyFrom scp command non-fatal

[Matthew]

[YOCTO #13802]

Make the scp failure non-fatal so the ltp tests continue to run and
the rest of the logs will be available to see afterwards.

Signed-off-by: Mingde (Matthew) Zeng 
---
 meta/lib/oeqa/core/target/ssh.py | 11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 090b40a814..1b78e830d4 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -61,9 +61,12 @@ class OESSHTarget(OETarget):
 self.logger.debug("[Command returned '%d' after %.2f seconds]"
  "" % (status, time.time() - starttime))
 
-if status and not ignore_status:
-raise AssertionError("Command '%s' returned non-zero exit "
- "status %d:\n%s" % (command, status, output))
+if status:
+if not ignore_status:
+raise AssertionError("Command '%s' returned non-zero exit "
+ "status %d:\n%s" % (command, status, 
output))
+else:
+self.logger.warning("Command '%s' returned non-zero exit 
status %d:\n%s" % (command, status, output))
 
 return (status, output)
 
@@ -113,7 +116,7 @@ class OESSHTarget(OETarget):
 """
 remotePath = '%s@%s:%s' % (self.user, self.ip, remoteSrc)
 scpCmd = self.scp + [remotePath, localDst]
-return self._run(scpCmd, ignore_status=False)
+return self._run(scpCmd, ignore_status=True)
 
 def copyDirTo(self, localSrc, remoteDst):
 """
-- 
2.27.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141230): 
https://lists.openembedded.org/g/openembedded-core/message/141230
Mute This Topic: https://lists.openembedded.org/mt/76010223/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] kernel-devsrc: fix x86 (32bit) on target module build

[Bruce Ashfield]

From: Bruce Ashfield 

The previous devsrc v5.8 on target build fixup [532ae127c
kernel-devsrc: fix on-target module build for v5.8+], missed
on condition: the defconfig

On 32bit x86 the arch can be queried differently on the host
and target. We aren't building against the defconfig, so we
remove it from auto.conf.cmd and we can again prepare to build
modules on target.

Signed-off-by: Bruce Ashfield 
---
 meta/recipes-kernel/linux/kernel-devsrc.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb 
b/meta/recipes-kernel/linux/kernel-devsrc.bb
index 2f20ca25b3..6c49b4891f 100644
--- a/meta/recipes-kernel/linux/kernel-devsrc.bb
+++ b/meta/recipes-kernel/linux/kernel-devsrc.bb
@@ -277,6 +277,8 @@ do_install() {
 sed -i 's/ifneq "$(LD)" ".*-linux-ld.bfd.*$/ifneq "$(LD)" "ld"/' 
"$kerneldir/build/include/config/auto.conf.cmd"
 sed -i 's/ifneq "$(CC_VERSION_TEXT)".*\(gcc.*\)"/ifneq 
"$(CC_VERSION_TEXT)" "\1"/' "$kerneldir/build/include/config/auto.conf.cmd"
 sed -i 's/ifneq "$(srctree)" ".*"/ifneq "$(srctree)" "."/' 
"$kerneldir/build/include/config/auto.conf.cmd"
+# we don't build against the defconfig, so make sure it isn't the 
trigger for syncconfig
+sed -i 's/ifneq "$(KBUILD_DEFCONFIG)".*"\(.*\)"/ifneq "\1" "\1"/' 
"$kerneldir/build/include/config/auto.conf.cmd"
 fi
 
 # make the scripts python3 safe. We won't be running these, and if they are
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141229): 
https://lists.openembedded.org/g/openembedded-core/message/141229
Mute This Topic: https://lists.openembedded.org/mt/76010075/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 11/11] Revert "gtk-icon-cache.bbclass: add runtime dependency"

[Steve Sakoman]

This reverts commit e921edd27e71af00f07e3f1e973d2b0c6c777bf4 since
it breaks adwaita-icon-theme and other recipes using gtk-icon-cache

Signed-off-by: Steve Sakoman 
---
 meta/classes/gtk-icon-cache.bbclass | 19 +++
 1 file changed, 3 insertions(+), 16 deletions(-)

diff --git a/meta/classes/gtk-icon-cache.bbclass 
b/meta/classes/gtk-icon-cache.bbclass
index dd394af27c..91cb4ad409 100644
--- a/meta/classes/gtk-icon-cache.bbclass
+++ b/meta/classes/gtk-icon-cache.bbclass
@@ -1,10 +1,6 @@
 FILES_${PN} += "${datadir}/icons/hicolor"
 
-DEPENDS +=" ${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} \
-${@['gdk-pixbuf', '']['${BPN}' == 'gdk-pixbuf']} \
-${@['gtk+3', '']['${BPN}' == 'gtk+3']} \
-gtk+3-native \
-"
+DEPENDS +=" ${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} 
gtk+3-native"
 
 PACKAGE_WRITE_DEPS += "gtk+3-native gdk-pixbuf-native"
 
@@ -52,18 +48,9 @@ python populate_packages_append () {
 bb.note("adding hicolor-icon-theme dependency to %s" % pkg)
 rdepends = ' ' + d.getVar('MLPREFIX', False) + "hicolor-icon-theme"
 d.appendVar('RDEPENDS_%s' % pkg, rdepends)
-
-#gtk_icon_cache_postinst depend on gdk-pixbuf and gtk+3
-bb.note("adding gdk-pixbuf dependency to %s" % pkg)
-rdepends = ' ' + d.getVar('MLPREFIX', False) + "gdk-pixbuf"
-d.appendVar('RDEPENDS_%s' % pkg, rdepends)
-
-bb.note("adding gtk+3 dependency to %s" % pkg)
-rdepends = ' ' + d.getVar('MLPREFIX', False) + "gtk+3"
-d.appendVar('RDEPENDS_%s' % pkg, rdepends)
-
+
 bb.note("adding gtk-icon-cache postinst and postrm scripts to %s" % 
pkg)
-
+
 postinst = d.getVar('pkg_postinst_%s' % pkg)
 if not postinst:
 postinst = '#!/bin/sh\n'
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141228): 
https://lists.openembedded.org/g/openembedded-core/message/141228
Mute This Topic: https://lists.openembedded.org/mt/76008809/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 08/11] gnutls: upgrade 3.6.13 -> 3.6.14

[Steve Sakoman]

From: Alexander Kanavin 

Security and bug fix release on the stable 3.6.x branch

Fixes CVE-2020-13777

Detailed list of changes at:

https://lists.gnupg.org/pipermail/gnutls-help/2020-June/004648.html

Signed-off-by: Alexander Kanavin 
Signed-off-by: Richard Purdie 
(cherry picked from commit cd88c81804a4a52b9875f2244c9f35911539be96)
Signed-off-by: Steve Sakoman 
---
 .../gnutls/{gnutls_3.6.13.bb => gnutls_3.6.14.bb}  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
 rename meta/recipes-support/gnutls/{gnutls_3.6.13.bb => gnutls_3.6.14.bb} (93%)

diff --git a/meta/recipes-support/gnutls/gnutls_3.6.13.bb 
b/meta/recipes-support/gnutls/gnutls_3.6.14.bb
similarity index 93%
rename from meta/recipes-support/gnutls/gnutls_3.6.13.bb
rename to meta/recipes-support/gnutls/gnutls_3.6.14.bb
index f56d42a613..a285e5a1f1 100644
--- a/meta/recipes-support/gnutls/gnutls_3.6.13.bb
+++ b/meta/recipes-support/gnutls/gnutls_3.6.14.bb
@@ -21,8 +21,7 @@ SRC_URI = 
"https://www.gnupg.org/ftp/gcrypt/gnutls/v${SHRT_VER}/gnutls-${PV}.tar
file://arm_eabi.patch \
 "
 
-SRC_URI[md5sum] = "bb1fe696a11543433785b4fc70ca225f"
-SRC_URI[sha256sum] = 
"32041df447d9f4644570cf573c9f60358e865637d69b7e59d1159b7240b52f38"
+SRC_URI[sha256sum] = 
"5630751adec7025b8ef955af4d141d00d252a985769f51b4059e5affa3d39d63"
 
 inherit autotools texinfo pkgconfig gettext lib_package gtk-doc
 
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141225): 
https://lists.openembedded.org/g/openembedded-core/message/141225
Mute This Topic: https://lists.openembedded.org/mt/76008799/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 05/11] cogl-1.0: correct X11 dependencies

[Steve Sakoman]

From: Matt Madison 

cogl depends on libxdamage when configured for X11 support.
A dependency was added for this in 2012 with
959a2f6d88d8fa6874fff83b7a1f0e7d4e36b887, but that addition
was lost when the recipe was reworked in 2013 with
b508fdd2b19ca30da8d09caf646897dc4cf195c8. That commit also
added a dependency on libxi, which is not actually needed.

The missing dependency doesn't cause problems in most cases,
since mesa also depends on libxdamage, but when an alternative
opengl implementation such as libglvnd is used, cogl fails
to configure.

Signed-off-by: Matt Madison 
Signed-off-by: Richard Purdie 
(cherry picked from commit 7651a63c508562caf1a3732cb49af50bccc98e8b)
Signed-off-by: Steve Sakoman 
---
 meta/recipes-graphics/cogl/cogl-1.0.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-graphics/cogl/cogl-1.0.inc 
b/meta/recipes-graphics/cogl/cogl-1.0.inc
index 1c5d0b79cb..d581ad1c04 100644
--- a/meta/recipes-graphics/cogl/cogl-1.0.inc
+++ b/meta/recipes-graphics/cogl/cogl-1.0.inc
@@ -21,7 +21,7 @@ EDEPENDS_GL = "virtual/libgl libdrm"
 EDEPENDS_GLES2 = "virtual/libgles2"
 EDEPENDS_KMS = "libdrm virtual/egl"
 EDEPENDS_EGL = "virtual/egl"
-EDEPENDS_X11 = "virtual/libx11 libxcomposite libxfixes libxi libxrandr"
+EDEPENDS_X11 = "virtual/libx11 libxcomposite libxdamage libxfixes libxrandr"
 EDEPENDS_WAYLAND = "virtual/egl virtual/libgles2 wayland"
 
 # Extra RDEPENDS for PACKAGECONFIG
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141222): 
https://lists.openembedded.org/g/openembedded-core/message/141222
Mute This Topic: https://lists.openembedded.org/mt/76008794/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 01/11] initscripts: Fix various shellcheck warnings in populate-volatile.sh

[Steve Sakoman]

From: Andrei Gherzan 

Signed-off-by: Andrei Gherzan 
Signed-off-by: Richard Purdie 
(cherry picked from commit 0a128a238f63d52aa82f8c63ee2f84ab528b3346)
Signed-off-by: Steve Sakoman 
---
 .../initscripts-1.0/populate-volatile.sh  | 58 +--
 1 file changed, 29 insertions(+), 29 deletions(-)

diff --git a/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh 
b/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh
index 1c525b71bd..3a265329f0 100755
--- a/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh
+++ b/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh
@@ -9,10 +9,10 @@
 ### END INIT INFO
 
 # Get ROOT_DIR
-DIRNAME=`dirname $0`
-ROOT_DIR=`echo $DIRNAME | sed -ne 's:/etc/.*::p'`
+DIRNAME="$(dirname "$0")"
+ROOT_DIR="$(echo "$DIRNAME" | sed -ne 's:/etc/.*::p')"
 
-[ -e ${ROOT_DIR}/etc/default/rcS ] && . ${ROOT_DIR}/etc/default/rcS
+[ -e "${ROOT_DIR}/etc/default/rcS" ] && . "${ROOT_DIR}/etc/default/rcS"
 # When running populate-volatile.sh at rootfs time, disable cache.
 [ -n "$ROOT_DIR" ] && VOLATILE_ENABLE_CACHE=no
 # If rootfs is read-only, disable cache.
@@ -46,13 +46,13 @@ create_file() {
[ "${VERBOSE}" != "no" ] && echo "Target already exists. 
Skipping."
} || {
if [ -z "$ROOT_DIR" ]; then
-   eval $EXEC
+   eval "$EXEC"
else
# Creating some files at rootfs time may fail and 
should fail,
# but these failures should not be logged to make sure 
the do_rootfs
# process doesn't fail. This does no harm, as this 
script will
# run on target to set up the correct files and 
directories.
-   eval $EXEC > /dev/null 2>&1
+   eval "$EXEC" > /dev/null 2>&1
fi
}
 }
@@ -68,11 +68,11 @@ mk_dir() {
[ "${VERBOSE}" != "no" ] && echo "Target already exists. 
Skipping."
} || {
if [ -z "$ROOT_DIR" ]; then
-   eval $EXEC
+   eval "$EXEC"
else
# For the same reason with create_file(), failures 
should
# not be logged.
-   eval $EXEC > /dev/null 2>&1
+   eval "$EXEC" > /dev/null 2>&1
fi
}
 }
@@ -96,11 +96,11 @@ link_file() {
test "$VOLATILE_ENABLE_CACHE" = yes && echo "   $EXEC" >> 
/etc/volatile.cache.build
 
if [ -z "$ROOT_DIR" ]; then
-   eval $EXEC
+   eval "$EXEC"
else
# For the same reason with create_file(), failures should
# not be logged.
-   eval $EXEC > /dev/null 2>&1
+   eval "$EXEC" > /dev/null 2>&1
fi
 }
 
@@ -117,11 +117,11 @@ check_requirements() {
TMP_DEFINED="${TMPROOT}/tmpdefined.$$"
TMP_COMBINED="${TMPROOT}/tmpcombined.$$"
 
-   sed 's@\(^:\)*:.*@\1@' ${ROOT_DIR}/etc/passwd | sort | uniq > 
"${TMP_DEFINED}"
-   cat ${CFGFILE} | grep -v "^#" | cut -s -d " " -f 2 > "${TMP_INTERMED}"
+   sed 's@\(^:\)*:.*@\1@' "${ROOT_DIR}/etc/passwd" | sort | uniq > 
"${TMP_DEFINED}"
+   grep -v "^#" "${CFGFILE}" | cut -s -d " " -f 2 > "${TMP_INTERMED}"
cat "${TMP_DEFINED}" "${TMP_INTERMED}" | sort | uniq > "${TMP_COMBINED}"
-   NR_DEFINED_USERS="`cat "${TMP_DEFINED}" | wc -l`"
-   NR_COMBINED_USERS="`cat "${TMP_COMBINED}" | wc -l`"
+   NR_DEFINED_USERS="$(wc -l < "${TMP_DEFINED}")"
+   NR_COMBINED_USERS="$(wc -l < "${TMP_COMBINED}")"
 
[ "${NR_DEFINED_USERS}" -ne "${NR_COMBINED_USERS}" ] && {
echo "Undefined users:"
@@ -131,12 +131,12 @@ check_requirements() {
}
 
 
-   sed 's@\(^:\)*:.*@\1@' ${ROOT_DIR}/etc/group | sort | uniq > 
"${TMP_DEFINED}"
-   cat ${CFGFILE} | grep -v "^#" | cut -s -d " " -f 3 > "${TMP_INTERMED}"
+   sed 's@\(^:\)*:.*@\1@' "${ROOT_DIR}/etc/group" | sort | uniq > 
"${TMP_DEFINED}"
+   grep -v "^#" "${CFGFILE}" | cut -s -d " " -f 3 > "${TMP_INTERMED}"
cat "${TMP_DEFINED}" "${TMP_INTERMED}" | sort | uniq > "${TMP_COMBINED}"
 
-   NR_DEFINED_GROUPS="`cat "${TMP_DEFINED}" | wc -l`"
-   NR_COMBINED_GROUPS="`cat "${TMP_COMBINED}" | wc -l`"
+   NR_DEFINED_GROUPS="$(wc -l < "${TMP_DEFINED}")"
+   NR_COMBINED_GROUPS="$(wc -l < "${TMP_COMBINED}")"
 
[ "${NR_DEFINED_GROUPS}" -ne "${NR_COMBINED_GROUPS}" ] && {
echo "Undefined groups:"
@@ -157,13 +157,13 @@ apply_cfgfile() {
 
[ "${VERBOSE}" != "no" ] && echo "Applying ${CFGFILE}"
 
-   [ "${SKIP_REQUIREMENTS}" == "yes" ] || check_requirements "${CFGFILE}" 
|| {
+   [ "${SKIP_REQUIREMENTS}" = "yes" ] || check_requirements "${CFGFILE}" 
|| {
echo "Skipping ${CFGFILE}"
return 1
}
 
-   cat ${CFGFILE} | sed 's

[OE-core][dunfell v2 02/11] initscripts: Fix populate-volatile.sh bug when file/dir exists

[Steve Sakoman]

From: Andrei Gherzan 

The blocks which test for entry exitence (file or directory) use a
`A && B || C` syntax. This form is not behaving as a if-then-else block
even the code logic assumes that. C may run when A is true which breaks
the case where VERBOSE is 'no' but the file/directory exists.

Along with fixing these specific issues, this patch fixes the other
instances where blocks of form `A && B || C` are used as if-then-else.

Signed-off-by: Andrei Gherzan 
Signed-off-by: Richard Purdie 
(cherry picked from commit 1b9ea22acb66554925720e04cf24100664234574)
Signed-off-by: Steve Sakoman 
---
 .../initscripts-1.0/populate-volatile.sh  | 24 +--
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh 
b/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh
index 3a265329f0..f21f48dd30 100755
--- a/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh
+++ b/meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh
@@ -26,15 +26,15 @@ COREDEF="00_core"
 
 create_file() {
EXEC=""
-   [ -z "$2" ] && {
+   if [ -z "$2" ]; then
EXEC="
touch \"$1\";
"
-   } || {
+   else
EXEC="
cp \"$2\" \"$1\";
"
-   }
+   fi
EXEC="
${EXEC}
chown ${TUSER}:${TGROUP} $1 || echo \"Failed to set owner -${TUSER}- 
for -$1-.\" >/dev/tty0 2>&1;
@@ -42,9 +42,9 @@ create_file() {
 
test "$VOLATILE_ENABLE_CACHE" = yes && echo "$EXEC" >> 
/etc/volatile.cache.build
 
-   [ -e "$1" ] && {
+   if [ -e "$1" ]; then
[ "${VERBOSE}" != "no" ] && echo "Target already exists. 
Skipping."
-   } || {
+   else
if [ -z "$ROOT_DIR" ]; then
eval "$EXEC"
else
@@ -54,7 +54,7 @@ create_file() {
# run on target to set up the correct files and 
directories.
eval "$EXEC" > /dev/null 2>&1
fi
-   }
+   fi
 }
 
 mk_dir() {
@@ -64,9 +64,9 @@ mk_dir() {
chmod ${TMODE} $1 || echo \"Failed to set mode -${TMODE}- for -$1-.\" 
>/dev/tty0 2>&1 "
 
test "$VOLATILE_ENABLE_CACHE" = yes && echo "$EXEC" >> 
/etc/volatile.cache.build
-   [ -e "$1" ] && {
+   if [ -e "$1" ]; then
[ "${VERBOSE}" != "no" ] && echo "Target already exists. 
Skipping."
-   } || {
+   else
if [ -z "$ROOT_DIR" ]; then
eval "$EXEC"
else
@@ -74,7 +74,7 @@ mk_dir() {
# not be logged.
eval "$EXEC" > /dev/null 2>&1
fi
-   }
+   fi
 }
 
 link_file() {
@@ -188,13 +188,13 @@ apply_cfgfile() {
[ -L "${TNAME}" ] && {
[ "${VERBOSE}" != "no" ] && echo "Found link."
NEWNAME=$(ls -l "${TNAME}" | sed -e 's/^.*-> 
\(.*\)$/\1/')
-   echo "${NEWNAME}" | grep -v "^/" >/dev/null && {
+   if echo "${NEWNAME}" | grep -v "^/" >/dev/null; then
TNAME="$(echo "${TNAME}" | sed -e 
's@\(.*\)/.*@\1@')/${NEWNAME}"
[ "${VERBOSE}" != "no" ] && echo "Converted 
relative linktarget to absolute path -${TNAME}-."
-   } || {
+   else
TNAME="${NEWNAME}"
[ "${VERBOSE}" != "no" ] && echo "Using 
absolute link target -${TNAME}-."
-   }
+   fi
}
 
case "${TTYPE}" in
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141219): 
https://lists.openembedded.org/g/openembedded-core/message/141219
Mute This Topic: https://lists.openembedded.org/mt/76008789/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 06/11] boost: backport fix to make async_pipes work with asio

[Steve Sakoman]

From: Viktor Rosendahl 

async_pipe is missing the executor_type type, which is expected by
asio in /usr/include/boost/asio/impl/read.hpp. Without this, it's
not possible to even compile code that uses constructs such as:

boost::asio::io_service foo;
boost::process::async_pipe foopipe{ boost::process::async_pipe(foo) };

This is only relevant for Dunfell because master has already moved to
boost-1.73.0 in which this bug has been fixed. The bug is also not
present in Zeus, which uses boost-1.71.0.

Signed-off-by: Viktor Rosendahl 
Signed-off-by: Steve Sakoman 
---
 .../0001-added-typedef-executor_type.patch| 54 +++
 meta/recipes-support/boost/boost_1.72.0.bb|  1 +
 2 files changed, 55 insertions(+)
 create mode 100644 
meta/recipes-support/boost/boost/0001-added-typedef-executor_type.patch

diff --git 
a/meta/recipes-support/boost/boost/0001-added-typedef-executor_type.patch 
b/meta/recipes-support/boost/boost/0001-added-typedef-executor_type.patch
new file mode 100644
index 00..7b8178fd76
--- /dev/null
+++ b/meta/recipes-support/boost/boost/0001-added-typedef-executor_type.patch
@@ -0,0 +1,54 @@
+From 6a4d2ff72114ef47c7afaf92e1042aca3dfa41b0 Mon Sep 17 00:00:00 2001
+From: Klemens David Morgenstern 
+Date: Fri, 22 Nov 2019 14:03:22 +0800
+Subject: [PATCH] added typedef executor_type;
+
+Upstream-Status: Backport 
[https://github.com/boostorg/process/commit/6a4d2ff72114ef47c7afaf92e1042aca3dfa41b0]
+
+Signed-off-by: Viktor Rosendahl 
+---
+ include/boost/process/async_pipe.hpp| 2 ++
+ include/boost/process/detail/posix/async_pipe.hpp   | 1 +
+ include/boost/process/detail/windows/async_pipe.hpp | 1 +
+ 3 files changed, 4 insertions(+)
+
+diff --git a/include/boost/process/async_pipe.hpp 
b/include/boost/process/async_pipe.hpp
+index 101fe1d..a562432 100644
+--- a/boost/process/async_pipe.hpp
 b/boost/process/async_pipe.hpp
+@@ -47,6 +47,8 @@ public:
+  */
+ typedef platform_specific handle_type;
+ 
++typedef typename handle_type::executor_type executor_type;
++
+ /** Construct a new async_pipe, does automatically open the pipe.
+  * Initializes source and sink with the same io_context.
+  * @note Windows creates a named pipe here, where the name is 
automatically generated.
+diff --git a/include/boost/process/detail/posix/async_pipe.hpp 
b/include/boost/process/detail/posix/async_pipe.hpp
+index 725a078..a82c057 100644
+--- a/boost/process/detail/posix/async_pipe.hpp
 b/boost/process/detail/posix/async_pipe.hpp
+@@ -23,6 +23,7 @@ class async_pipe
+ public:
+ typedef int native_handle_type;
+ typedef ::boost::asio::posix::stream_descriptor handle_type;
++typedef typename handle_type::executor_type executor_type;
+ 
+ inline async_pipe(boost::asio::io_context & ios) : async_pipe(ios, ios) {}
+ 
+diff --git a/include/boost/process/detail/windows/async_pipe.hpp 
b/include/boost/process/detail/windows/async_pipe.hpp
+index 06d5f2d..0b447f9 100644
+--- a/boost/process/detail/windows/async_pipe.hpp
 b/boost/process/detail/windows/async_pipe.hpp
+@@ -48,6 +48,7 @@ class async_pipe
+ public:
+ typedef ::boost::winapi::HANDLE_ native_handle_type;
+ typedef ::boost::asio::windows::stream_handle   handle_type;
++typedef typename handle_type::executor_type executor_type;
+ 
+ async_pipe(boost::asio::io_context & ios) : async_pipe(ios, ios, 
make_pipe_name(), true) {}
+ async_pipe(boost::asio::io_context & ios_source, boost::asio::io_context 
& ios_sink)
+-- 
+2.17.1
+
diff --git a/meta/recipes-support/boost/boost_1.72.0.bb 
b/meta/recipes-support/boost/boost_1.72.0.bb
index 0b7badbc76..51c84bc935 100644
--- a/meta/recipes-support/boost/boost_1.72.0.bb
+++ b/meta/recipes-support/boost/boost_1.72.0.bb
@@ -8,4 +8,5 @@ SRC_URI += "file://arm-intrinsics.patch \

file://0001-Don-t-set-up-arch-instruction-set-flags-we-do-that-o.patch \
file://0001-dont-setup-compiler-flags-m32-m64.patch \
file://0001-revert-cease-dependence-on-range.patch \
+   file://0001-added-typedef-executor_type.patch \
"
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141223): 
https://lists.openembedded.org/g/openembedded-core/message/141223
Mute This Topic: https://lists.openembedded.org/mt/76008796/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 09/11] gnutls: Fix krb5 code license to GPLv2.1+ to match the LICENSE file.

[Steve Sakoman]

From: zhengruoqin 

This was discussed and accepted upstream by the project so their license is 
consistent.
Please reference to https://gitlab.com/gnutls/gnutls/-/issues/1018
and https://gitlab.com/gnutls/gnutls/-/merge_requests/1285.

Signed-off-by: Zheng Ruoqin 
Signed-off-by: Richard Purdie 
(cherry picked from commit 267d07301c79c24969c169add05284f612c41d77)
Signed-off-by: Steve Sakoman 
---
 ...se-to-GPLv2.1-to-keep-with-LICENSE-f.patch | 90 +++
 meta/recipes-support/gnutls/gnutls_3.6.14.bb  |  1 +
 2 files changed, 91 insertions(+)
 create mode 100644 
meta/recipes-support/gnutls/gnutls/0001-Modied-the-license-to-GPLv2.1-to-keep-with-LICENSE-f.patch

diff --git 
a/meta/recipes-support/gnutls/gnutls/0001-Modied-the-license-to-GPLv2.1-to-keep-with-LICENSE-f.patch
 
b/meta/recipes-support/gnutls/gnutls/0001-Modied-the-license-to-GPLv2.1-to-keep-with-LICENSE-f.patch
new file mode 100644
index 00..a610abf9b5
--- /dev/null
+++ 
b/meta/recipes-support/gnutls/gnutls/0001-Modied-the-license-to-GPLv2.1-to-keep-with-LICENSE-f.patch
@@ -0,0 +1,90 @@
+From c0ae3f659c6c130d151378ba4d7d861e3b7b970f Mon Sep 17 00:00:00 2001
+From: Lei Maohui 
+Date: Wed, 8 Jul 2020 14:50:27 +0900
+Subject: [PATCH] Modied the license to GPLv2.1+ to keep with LICENSE file.
+
+Signed-off-by: Lei Maohui 
+Please reference to https://gitlab.com/gnutls/gnutls/-/issues/1018.
+Upstream-Status: Backport 
[https://gitlab.com/gnutls/gnutls/-/merge_requests/1285].
+---
+ lib/x509/krb5.c | 20 +++-
+ lib/x509/krb5.h | 20 +++-
+ 2 files changed, 22 insertions(+), 18 deletions(-)
+
+diff --git a/lib/x509/krb5.c b/lib/x509/krb5.c
+index 7fe84e6..d68c737 100644
+--- a/lib/x509/krb5.c
 b/lib/x509/krb5.c
+@@ -1,21 +1,23 @@
+ /*
+  * Copyright (C) 2015 Red Hat, Inc.
+  *
++ * Author: Nikos Mavrogiannopoulos
++ *
+  * This file is part of GnuTLS.
+  *
+- * GnuTLS is free software: you can redistribute it and/or modify it
+- * under the terms of the GNU General Public License as published by
+- * the Free Software Foundation, either version 3 of the License, or
+- * (at your option) any later version.
++ * The GnuTLS is free software; you can redistribute it and/or
++ * modify it under the terms of the GNU Lesser General Public License
++ * as published by the Free Software Foundation; either version 2.1 of
++ * the License, or (at your option) any later version.
+  *
+- * GnuTLS is distributed in the hope that it will be useful, but
++ * This library is distributed in the hope that it will be useful, but
+  * WITHOUT ANY WARRANTY; without even the implied warranty of
+  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+- * General Public License for more details.
++ * Lesser General Public License for more details.
++ *
++ * You should have received a copy of the GNU Lesser General Public License
++ * along with this program.  If not, see 
+  *
+- * You should have received a copy of the GNU General Public License
+- * along with this program.  If not, see
+- * .
+  */
+ 
+ #include 
+diff --git a/lib/x509/krb5.h b/lib/x509/krb5.h
+index d8926af..815bb28 100644
+--- a/lib/x509/krb5.h
 b/lib/x509/krb5.h
+@@ -1,21 +1,23 @@
+ /*
+  * Copyright (C) 2015 Red Hat, Inc.
+  *
++ * Author: Nikos Mavrogiannopoulos
++ *
+  * This file is part of GnuTLS.
+  *
+- * GnuTLS is free software: you can redistribute it and/or modify it
+- * under the terms of the GNU General Public License as published by
+- * the Free Software Foundation, either version 3 of the License, or
+- * (at your option) any later version.
++ * The GnuTLS is free software; you can redistribute it and/or
++ * modify it under the terms of the GNU Lesser General Public License
++ * as published by the Free Software Foundation; either version 2.1 of
++ * the License, or (at your option) any later version.
+  *
+- * GnuTLS is distributed in the hope that it will be useful, but
++ * This library is distributed in the hope that it will be useful, but
+  * WITHOUT ANY WARRANTY; without even the implied warranty of
+  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+- * General Public License for more details.
++ * Lesser General Public License for more details.
++ *
++ * You should have received a copy of the GNU Lesser General Public License
++ * along with this program.  If not, see 
+  *
+- * You should have received a copy of the GNU General Public License
+- * along with this program.  If not, see
+- * .
+  */
+ 
+ #ifndef GNUTLS_LIB_X509_KRB5_H
+-- 
+2.17.1
+
diff --git a/meta/recipes-support/gnutls/gnutls_3.6.14.bb 
b/meta/recipes-support/gnutls/gnutls_3.6.14.bb
index a285e5a1f1..cc0454a561 100644
--- a/meta/recipes-support/gnutls/gnutls_3.6.14.bb
+++ b/meta/recipes-support/gnutls/gnutls_3.6.14.bb
@@ -19,6 +19,7 @@ SHRT_VER = 
"${@d.getVar('PV').split('.')[0]}.${@d.getVar('P

[OE-core][dunfell v2 10/11] glib-networking: upgrade 2.62.3 to 2.62.4

[Steve Sakoman]

Bug and CVE fix release

Fixes CVE-2020-13645

Details of changes at:

https://gitlab.gnome.org/GNOME/glib-networking/-/commits/glib-2-62

Signed-off-by: Steve Sakoman 
---
 .../{glib-networking_2.62.3.bb => glib-networking_2.62.4.bb}  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-core/glib-networking/{glib-networking_2.62.3.bb => 
glib-networking_2.62.4.bb} (88%)

diff --git a/meta/recipes-core/glib-networking/glib-networking_2.62.3.bb 
b/meta/recipes-core/glib-networking/glib-networking_2.62.4.bb
similarity index 88%
rename from meta/recipes-core/glib-networking/glib-networking_2.62.3.bb
rename to meta/recipes-core/glib-networking/glib-networking_2.62.4.bb
index e9596ce1a4..b74532087c 100644
--- a/meta/recipes-core/glib-networking/glib-networking_2.62.3.bb
+++ b/meta/recipes-core/glib-networking/glib-networking_2.62.4.bb
@@ -9,8 +9,8 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=4fbd65380cdd255951079008b364516c"
 SECTION = "libs"
 DEPENDS = "glib-2.0"
 
-SRC_URI[archive.md5sum] = "a758ca62bd54982a798b39c744cbf783"
-SRC_URI[archive.sha256sum] = 
"8ca1f86f23a76b5c7640624f7d5490705c78e81375e1741c9a1c41ce7f8f7ff7"
+SRC_URI[archive.md5sum] = "fad14a4a2cac73f0c2f9d426f1a9e5af"
+SRC_URI[archive.sha256sum] = 
"c18f289eec480fdce12044c0a06f77521edf9f460d16ad4213de61f2a3b294cf"
 
 PACKAGECONFIG ??= "gnutls"
 
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141227): 
https://lists.openembedded.org/g/openembedded-core/message/141227
Mute This Topic: https://lists.openembedded.org/mt/76008803/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 04/11] cve-update: handle baseMetricV2 as optional

[Steve Sakoman]

From: Konrad Weihmann 

Currently in NVD DB an item popped up, which hasn't set baseMetricV2.
Let the parser handle it as an optional item.
In case use baseMetricV2 before baseMetricV3

Signed-off-by: Konrad Weihmann 
Signed-off-by: Richard Purdie 
(cherry picked from commit fdcbf3f28289188c5a97664d1421d4a5c4991eda)
Signed-off-by: Steve Sakoman 
---
 meta/recipes-core/meta/cve-update-db-native.bb | 13 +
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-core/meta/cve-update-db-native.bb 
b/meta/recipes-core/meta/cve-update-db-native.bb
index f27ade40db..32d6dbdffc 100644
--- a/meta/recipes-core/meta/cve-update-db-native.bb
+++ b/meta/recipes-core/meta/cve-update-db-native.bb
@@ -176,15 +176,20 @@ def update_db(c, jsondata):
 if not elt['impact']:
 continue
 
+accessVector = None
 cveId = elt['cve']['CVE_data_meta']['ID']
 cveDesc = elt['cve']['description']['description_data'][0]['value']
 date = elt['lastModifiedDate']
-accessVector = elt['impact']['baseMetricV2']['cvssV2']['accessVector']
-cvssv2 = elt['impact']['baseMetricV2']['cvssV2']['baseScore']
-
 try:
+accessVector = 
elt['impact']['baseMetricV2']['cvssV2']['accessVector']
+cvssv2 = elt['impact']['baseMetricV2']['cvssV2']['baseScore']
+except KeyError:
+cvssv2 = 0.0
+try:
+accessVector = accessVector or 
elt['impact']['baseMetricV3']['cvssV3']['attackVector']
 cvssv3 = elt['impact']['baseMetricV3']['cvssV3']['baseScore']
-except:
+except KeyError:
+accessVector = accessVector or "UNKNOWN"
 cvssv3 = 0.0
 
 c.execute("insert or replace into NVD values (?, ?, ?, ?, ?, ?)",
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141221): 
https://lists.openembedded.org/g/openembedded-core/message/141221
Mute This Topic: https://lists.openembedded.org/mt/76008793/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 03/11] checklayer: check layer in BBLAYERS before test

[Steve Sakoman]

From: Lee Chee Yang 

layer under test should absent from BBLAYERS when running
yocto-check-layer. This allow to get signatures before layer
under test. There are existing steps to add the layer under
test to BBLAYERS after getting initial signatures.

add steps to check for layer under test in BBLAYERS before
running any test, skip test for the layer if the layer under
test exist in BBLAYERS.

[YOCTO #13176]

Signed-off-by: Lee Chee Yang 
Signed-off-by: Richard Purdie 
(cherry picked from commit be02e8dbfb0d1decce125322f9f1e11a649756c0)
Signed-off-by: Steve Sakoman 
---
 scripts/lib/checklayer/__init__.py | 14 ++
 scripts/yocto-check-layer  |  9 -
 2 files changed, 22 insertions(+), 1 deletion(-)

diff --git a/scripts/lib/checklayer/__init__.py 
b/scripts/lib/checklayer/__init__.py
index f625d59896..fe545607bb 100644
--- a/scripts/lib/checklayer/__init__.py
+++ b/scripts/lib/checklayer/__init__.py
@@ -229,6 +229,20 @@ def add_layers(bblayersconf, layers, logger):
 f.write("\nBBLAYERS += \"%s\"\n" % path)
 return True
 
+def check_bblayers(bblayersconf, layer_path, logger):
+'''
+If layer_path found in BBLAYERS return True
+'''
+import bb.parse
+import bb.data
+
+ldata = bb.parse.handle(bblayersconf, bb.data.init(), include=True)
+for bblayer in (ldata.getVar('BBLAYERS') or '').split():
+if os.path.normpath(bblayer) == os.path.normpath(layer_path):
+return True
+
+return False
+
 def check_command(error_msg, cmd, cwd=None):
 '''
 Run a command under a shell, capture stdout and stderr in a single stream,
diff --git a/scripts/yocto-check-layer b/scripts/yocto-check-layer
index ca6c79bc8d..b7c83c8b54 100755
--- a/scripts/yocto-check-layer
+++ b/scripts/yocto-check-layer
@@ -24,7 +24,7 @@ import scriptpath
 scriptpath.add_oe_lib_path()
 scriptpath.add_bitbake_lib_path()
 
-from checklayer import LayerType, detect_layers, add_layers, 
add_layer_dependencies, get_signatures
+from checklayer import LayerType, detect_layers, add_layers, 
add_layer_dependencies, get_signatures, check_bblayers
 from oeqa.utils.commands import get_bb_vars
 
 PROGNAME = 'yocto-check-layer'
@@ -138,6 +138,13 @@ def main():
 layer['type'] == LayerType.ERROR_BSP_DISTRO:
 continue
 
+if check_bblayers(bblayersconf, layer['path'], logger):
+logger.info("%s already in %s. To capture initial signatures, 
layer under test should not present "
+   "in BBLAYERS. Please remove %s from BBLAYERS." % 
(layer['name'], bblayersconf, layer['name']))
+results[layer['name']] = None
+results_status[layer['name']] = 'SKIPPED (Layer under test should 
not present in BBLAYERS)'
+continue
+
 logger.info('')
 logger.info("Setting up for %s(%s), %s" % (layer['name'], 
layer['type'],
 layer['path']))
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141220): 
https://lists.openembedded.org/g/openembedded-core/message/141220
Mute This Topic: https://lists.openembedded.org/mt/76008792/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 07/11] bind: upgrade 9.11.19 -> 9.11.21

[Steve Sakoman]

From: Yi Zhao 

Bug and CVE fixes only

Detailed list of changes at:
https://gitlab.isc.org/isc-projects/bind9/-/blob/v9_11_21/CHANGES

Signed-off-by: Yi Zhao 
Signed-off-by: Richard Purdie 
(cherry picked from commit c6749532f94f435e6771d66d3fa225e676753478)
Signed-off-by: Steve Sakoman 
---
 .../bind/{bind_9.11.19.bb => bind_9.11.21.bb}   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-connectivity/bind/{bind_9.11.19.bb => bind_9.11.21.bb} 
(98%)

diff --git a/meta/recipes-connectivity/bind/bind_9.11.19.bb 
b/meta/recipes-connectivity/bind/bind_9.11.21.bb
similarity index 98%
rename from meta/recipes-connectivity/bind/bind_9.11.19.bb
rename to meta/recipes-connectivity/bind/bind_9.11.21.bb
index 276173e09f..ee546a0a2c 100644
--- a/meta/recipes-connectivity/bind/bind_9.11.19.bb
+++ b/meta/recipes-connectivity/bind/bind_9.11.21.bb
@@ -20,7 +20,7 @@ SRC_URI = 
"https://ftp.isc.org/isc/bind9/${PV}/${BPN}-${PV}.tar.gz \
file://0001-avoid-start-failure-with-bind-user.patch \
"
 
-SRC_URI[sha256sum] = 
"0dee554a4caa368948b32da9a0c97b516c19103bc13ff5b3762c5d8552f52329"
+SRC_URI[sha256sum] = 
"668158b005b3de4328fa0d3f524b66f28f024c67538aa9412a9e69c9dfbc"
 
 UPSTREAM_CHECK_URI = "https://ftp.isc.org/isc/bind9/";
 # stay at 9.11 until 9.16, from 9.16 follow the ESV versions divisible by 4
-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141224): 
https://lists.openembedded.org/g/openembedded-core/message/141224
Mute This Topic: https://lists.openembedded.org/mt/76008797/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core][dunfell v2 00/11] Patch review

[Steve Sakoman]

Sending a second version since there have been a number of comments
on this series.  Please have any further comments back by end of day Friday.

Version 2 drops the following patches from version 1:

  layer.conf: fix adwaita-icon-theme signature change problem
  gtk-icon-cache.bbclass: add features_check

And adds a revert of:

  gtk-icon-cache.bbclass: add runtime dependency

The following changes since commit 7ce425fa1295a9dca48f8474be58db3ac8aa540d:

  glibc: Secruity fix for CVE-2020-6096 (2020-07-27 12:15:56 -1000)

are available in the Git repository at:

  git://git.openembedded.org/openembedded-core-contrib stable/dunfell-nut
  
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=stable/dunfell-nut

Alexander Kanavin (1):
  gnutls: upgrade 3.6.13 -> 3.6.14

Andrei Gherzan (2):
  initscripts: Fix various shellcheck warnings in populate-volatile.sh
  initscripts: Fix populate-volatile.sh bug when file/dir exists

Konrad Weihmann (1):
  cve-update: handle baseMetricV2 as optional

Lee Chee Yang (1):
  checklayer: check layer in BBLAYERS before test

Matt Madison (1):
  cogl-1.0: correct X11 dependencies

Steve Sakoman (2):
  glib-networking: upgrade 2.62.3 to 2.62.4
  Revert "gtk-icon-cache.bbclass: add runtime dependency"

Viktor Rosendahl (1):
  boost: backport fix to make async_pipes work with asio

Yi Zhao (1):
  bind: upgrade 9.11.19 -> 9.11.21

zhengruoqin (1):
  gnutls: Fix krb5 code license to GPLv2.1+ to match the LICENSE file.

 meta/classes/gtk-icon-cache.bbclass   | 19 +---
 .../bind/{bind_9.11.19.bb => bind_9.11.21.bb} |  2 +-
 ...ng_2.62.3.bb => glib-networking_2.62.4.bb} |  4 +-
 .../initscripts-1.0/populate-volatile.sh  | 80 -
 .../recipes-core/meta/cve-update-db-native.bb | 13 ++-
 meta/recipes-graphics/cogl/cogl-1.0.inc   |  2 +-
 .../0001-added-typedef-executor_type.patch| 54 +++
 meta/recipes-support/boost/boost_1.72.0.bb|  1 +
 ...se-to-GPLv2.1-to-keep-with-LICENSE-f.patch | 90 +++
 .../{gnutls_3.6.13.bb => gnutls_3.6.14.bb}|  4 +-
 scripts/lib/checklayer/__init__.py| 14 +++
 scripts/yocto-check-layer |  9 +-
 12 files changed, 225 insertions(+), 67 deletions(-)
 rename meta/recipes-connectivity/bind/{bind_9.11.19.bb => bind_9.11.21.bb} 
(98%)
 rename meta/recipes-core/glib-networking/{glib-networking_2.62.3.bb => 
glib-networking_2.62.4.bb} (88%)
 create mode 100644 
meta/recipes-support/boost/boost/0001-added-typedef-executor_type.patch
 create mode 100644 
meta/recipes-support/gnutls/gnutls/0001-Modied-the-license-to-GPLv2.1-to-keep-with-LICENSE-f.patch
 rename meta/recipes-support/gnutls/{gnutls_3.6.13.bb => gnutls_3.6.14.bb} (92%)

-- 
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141217): 
https://lists.openembedded.org/g/openembedded-core/message/141217
Mute This Topic: https://lists.openembedded.org/mt/76008770/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][dunfell 05/12] layer.conf: fix adwaita-icon-theme signature change problem

[Steve Sakoman]

On Tue, Aug 4, 2020 at 8:18 PM Khem Raj  wrote:
>
> On Tue, Aug 4, 2020 at 10:59 PM Martin Jansa  wrote:
> >
> > We did the gtk-icon-cache.bbclass backport in:
> > https://git.openembedded.org/openembedded-core/commit/meta/classes/gtk-icon-cache.bbclass?h=dunfell&id=e921edd27e71af00f07e3f1e973d2b0c6c777bf4
> > but this fix, is only for adwaita-icon-them and other recipes using 
> > gtk-icon-cache might need identical fix as mentioned by Khem in review of 
> > this work around in master:
> > https://lists.openembedded.org/g/openembedded-core/message/140910
> >
> > I guess it would be better to just revert 
> > e921edd27e71af00f07e3f1e973d2b0c6c777bf4 in dunfell and find better 
> > solution for master first.
> >
>
> +1

OK, sounds like a plan.  WIll revert
e921edd27e71af00f07e3f1e973d2b0c6c777bf4 and drop this patch.

Steve


> > View/Reply Online (#141192): 
> > https://lists.openembedded.org/g/openembedded-core/message/141192
> > Mute This Topic: https://lists.openembedded.org/mt/75966624/1997914
> > Group Owner: openembedded-core+ow...@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
> > [raj.k...@gmail.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141216): 
https://lists.openembedded.org/g/openembedded-core/message/141216
Mute This Topic: https://lists.openembedded.org/mt/75966624/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] allarch: Add missing allarch ttf-bitstream-vera

[Daniel Gomez]

allarch is missing in ttf-bitstream-vera recipe. Add it and include the
recipe in the SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS list.

Signed-off-by: Daniel Gomez 
---
 meta/conf/layer.conf   | 1 +
 meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
index c115c87043..1a01d02fef 100644
--- a/meta/conf/layer.conf
+++ b/meta/conf/layer.conf
@@ -71,6 +71,7 @@ SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS += " \
   grub-efi->grub-bootconf \
   liberation-fonts->fontconfig \
   cantarell-fonts->fontconfig \
+  ttf-bitstream-vera->fontconfig \
   gnome-icon-theme->librsvg \
   font-alias->font-util \
   systemd-boot->systemd-bootconf \
diff --git a/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb 
b/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb
index 70b32cf8f1..3e1ba196b5 100644
--- a/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb
+++ b/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb
@@ -9,7 +9,7 @@ LICENSE = "BitstreamVera"
 LIC_FILES_CHKSUM = "file://COPYRIGHT.TXT;md5=27d7484b1e18d0ee4ce538644a3f04be"
 PR = "r7"

-inherit fontcache
+inherit allarch fontcache

 FONT_PACKAGES = "${PN}"

--
2.27.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141215): 
https://lists.openembedded.org/g/openembedded-core/message/141215
Mute This Topic: https://lists.openembedded.org/mt/76008040/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] lttng-modules: update to 2.12.2 (fixes v5.8+ builds)

[Bruce Ashfield]

From: Bruce Ashfield 

Updating to the 2.12.2 lttng-modules release. This is the standard
collection of bug fixes as well as changes to support building against
the v5.8 kernel.

Changelog follows:

 2020-08-04 (National Chocolate Chip Cookie Day) LTTng modules 2.12.2
 * Fix: Lock metadata cache on session destroy
 * Fix: metadata stream leak, missing list removal and locking
 * Fix: coherent state not changed atomically with metadata written
 * fix: include module.h for EXPORT_SYMBOL_GPL
 * fix: __lttng_vmalloc_node_range const caller introduced in v3.6
 * fix: version range for overflow_callback
 * fix: global_dirty_limit was introduced in v3.1
 * fix: wrapper_uprobe_unregister is a void function
 * fix: prior to v4.0, __vmalloc_node_range had no vm_flags param
 * fix: vmalloc on v5.8 without KALLSYMS
 * Detect missing symbols used with kallsyms_lookup at compile time
 * Use exported symbol bdevname() instead of disk_name()
 * Add git-review config
 * fix: mm: remove vmalloc_sync_(un)mappings() (v5.8)
 * fix: mm/writeback: discard NR_UNSTABLE_NFS, use NR_WRITEBACK (v5.8)
 * fix: block: remove the error argument to the block_bio_complete 
(v5.8)
 * fix: pipe_buf_operations rework (v5.8)

Signed-off-by: Bruce Ashfield 
---
 .../lttng/{lttng-modules_2.12.1.bb => lttng-modules_2.12.2.bb}  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-kernel/lttng/{lttng-modules_2.12.1.bb => 
lttng-modules_2.12.2.bb} (95%)

diff --git a/meta/recipes-kernel/lttng/lttng-modules_2.12.1.bb 
b/meta/recipes-kernel/lttng/lttng-modules_2.12.2.bb
similarity index 95%
rename from meta/recipes-kernel/lttng/lttng-modules_2.12.1.bb
rename to meta/recipes-kernel/lttng/lttng-modules_2.12.2.bb
index c0df0cab33..49b7a116bc 100644
--- a/meta/recipes-kernel/lttng/lttng-modules_2.12.1.bb
+++ b/meta/recipes-kernel/lttng/lttng-modules_2.12.2.bb
@@ -13,7 +13,7 @@ SRC_URI = 
"https://lttng.org/files/${BPN}/${BPN}-${PV}.tar.bz2 \
file://BUILD_RUNTIME_BUG_ON-vs-gcc7.patch \
"
 
-SRC_URI[sha256sum] = 
"639c03bf8f0e920567f45c96cc7ef749a8fc29ff7fb3fb97263d67f66fb71aef"
+SRC_URI[sha256sum] = 
"df50bc3bd58679705714f17721acf619a8b0cedc694f8a97052aa5099626feca"
 
 export INSTALL_MOD_DIR="kernel/lttng-modules"
 
-- 
2.19.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141214): 
https://lists.openembedded.org/g/openembedded-core/message/141214
Mute This Topic: https://lists.openembedded.org/mt/76007903/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] allarch: Fix wrong dependencies

[Daniel Gomez]

Hi Richard,

On Wed, 5 Aug 2020 at 15:31, Richard Purdie
 wrote:
>
> On Wed, 2020-08-05 at 15:22 +0200, Daniel Gomez wrote:
> > In all these cases, allarch is used but the package dependencies are not
> > allarch. It generates a conflict when the same package is built for 
> > different
> > MACHINES at the same time.
> >
> > Here a build error for the ttf-ubuntu-font-family in a multiconfig 
> > enviroment:
>
> This is on the wrong mailing list. I also thought there was code
> somewhere to remove font dependencies from changing the task
> signatures?
Sorry! I mixed the lists.
>
> http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/conf/layer.conf
>
> see the lines like:
>
> liberation-fonts->fontconfig \
> cantarell-fonts->fontconfig \
>
> Perhaps meta-oe has some of those missing?
I didn't notice about that SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS list and looks like
meta-oe already implements it so, what we actually need is to include
that recipe in our layer
conf.

On the other hand, I think the following recipes are missing the
inherit allarch:
meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb
meta/recipes-graphics/cantarell-fonts/cantarell-fonts_git.bb
meta/recipes-graphics/ttf-fonts/liberation-fonts_2.00.1.bb

And then, add the ttf-bitstream-vera recipe as part of the
SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS.

Will send a patch to fix it.

Thanks!
>
> Cheers,
>
> Richard
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141213): 
https://lists.openembedded.org/g/openembedded-core/message/141213
Mute This Topic: https://lists.openembedded.org/mt/76007129/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] ✗ patchtest: failure for allarch: Fix wrong dependencies

[Patchwork]

== Series Details ==

Series: allarch: Fix wrong dependencies
Revision: 1
URL   : https://patchwork.openembedded.org/series/25470/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Series sent to the wrong mailing list or some patches from 
the series correspond to different mailing lists [test_target_mailing_list] 
  Suggested fixSend the series again to the correct mailing list (ML)
  Suggested ML openembedded-de...@lists.openembedded.org 
[http://git.openembedded.org/meta-openembedded/]
  Patch's path:meta-oe/recipes-graphics/terminus-font/terminus-font_4.38.bb

* Issue Series does not apply on top of target branch 
[test_series_merge_on_head] 
  Suggested fixRebase your series on top of targeted branch
  Targeted branch  master (currently at 10221e578f)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141212): 
https://lists.openembedded.org/g/openembedded-core/message/141212
Mute This Topic: https://lists.openembedded.org/mt/76007298/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] allarch: Fix wrong dependencies

[Richard Purdie]

On Wed, 2020-08-05 at 15:22 +0200, Daniel Gomez wrote:
> In all these cases, allarch is used but the package dependencies are not
> allarch. It generates a conflict when the same package is built for different
> MACHINES at the same time.
> 
> Here a build error for the ttf-ubuntu-font-family in a multiconfig enviroment:

This is on the wrong mailing list. I also thought there was code
somewhere to remove font dependencies from changing the task
signatures?

http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/conf/layer.conf

see the lines like:

liberation-fonts->fontconfig \
cantarell-fonts->fontconfig \

Perhaps meta-oe has some of those missing?

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141211): 
https://lists.openembedded.org/g/openembedded-core/message/141211
Mute This Topic: https://lists.openembedded.org/mt/76007129/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] gcc10: Don't default back to -fcommon

[Richard Purdie]

On Wed, 2020-08-05 at 15:15 +0200, Jacob Kroon wrote:
> On 8/5/20 8:17 AM, Khem Raj wrote:
> > here is a recent build with clang11 which also switched to using
> > -fno-common and errors are pretty much same you will see with gcc10
> > with this patch to revert the -fcommon defaults.
> > 
> > https://errors.yoctoproject.org/Errors/Build/107234/?limit=100
> > 
> > I was waiting for upstream packages to get fixed before switching
> > the
> > defaults back to -fno-common.
> > 
> 
> Yes I understand, maybe its better to hold this for now.
> 
> Having a list of recipes that needs fixing would probably speed
> things up. I can have a look at some of them.

I ran this through with just OE-Core and the list of failures is:

https://autobuilder.yoctoproject.org/typhoon/#/builders/52/builds/2237

I'm a little frustrated that patches like this get sent and them people
like Khem and I have to spot and avoid the problems as there isn't
testing :/.

I'll drop the patch for now until more of these issues are resolved.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141210): 
https://lists.openembedded.org/g/openembedded-core/message/141210
Mute This Topic: https://lists.openembedded.org/mt/75983138/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] allarch: Fix wrong dependencies

[Daniel Gomez]

In all these cases, allarch is used but the package dependencies are not
allarch. It generates a conflict when the same package is built for different
MACHINES at the same time.

Here a build error for the ttf-ubuntu-font-family in a multiconfig enviroment:

ERROR: mc:qt5507:ttf-ubuntu-font-family-0.83-r1
do_package_write_deb:
Fatal errors occurred in subprocesses:
Command
'PATH="/workdir/build/tmp/sysroots-uninative/x86_64-linux/usr/bin:/workdir/repo/poky/scripts:/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/recipe-sysroot-native/usr/bin/allarch-poky-linux:/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/recipe-sysroot/usr/bin/crossscripts:/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/recipe-sysroot-native/usr/sbin:/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/recipe-sysroot-native/usr/bin:/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/recipe-sysroot-native/sbin:/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/recipe-sysroot-native/bin:/workdir/repo/poky/bitbake/bin:/workdir/build/tmp/hosttools"
dpkg-deb -b
/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/packages-split/ttf-ubuntu-sans
/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/deploy-debs/all'
returned non-zero exit status 2
Subprocess output:dpkg-deb: building package 'ttf-ubuntu-sans' in
'/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/deploy-debs/all'.
dpkg-deb: error: unable to create
'/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/deploy-debs/all':
No such file or directory

ERROR: mc:qt5507:ttf-ubuntu-font-family-0.83-r1
do_package_write_deb:
ERROR: mc:qt5507:ttf-ubuntu-font-family-0.83-r1
do_package_write_deb:
Function failed: do_package_deb
ERROR: Logfile of failure stored in:
/workdir/build/tmp/work/all-poky-linux/ttf-ubuntu-font-family/0.83-r1/temp/log.do_package_write_deb.11826
NOTE: recipe ttf-ubuntu-font-family-0.83-r1: task
do_package_write_deb:
Failed
ERROR: Task
(multiconfig:qt5507:/workdir/repo/poky/../meta-qtec-hockey/recipes-graphics/ttf-fonts/ttf-ubuntu-font-family_0.83.bb:do_package_write_deb)
failed with exit code '1'

Similar patch:
https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-graphics/ttf-fonts?id=0d9f7900dec4ec68d8ad6c8a455e5b4f068e3715

Signed-off-by: Daniel Gomez 
---
 meta-oe/recipes-graphics/terminus-font/terminus-font_4.38.bb| 2 +-
 .../ttf-fonts/source-han-sans-cn-fonts_1.004.bb | 2 +-
 .../ttf-fonts/source-han-sans-jp-fonts_1.004.bb | 2 +-
 .../ttf-fonts/source-han-sans-kr-fonts_1.004.bb | 2 +-
 .../ttf-fonts/source-han-sans-tw-fonts_1.004.bb | 2 +-
 meta-oe/recipes-graphics/ttf-fonts/ttf-pt-sans_1.1.bb   | 2 +-
 meta-oe/recipes-graphics/ttf-fonts/ttf.inc  | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/meta-oe/recipes-graphics/terminus-font/terminus-font_4.38.bb 
b/meta-oe/recipes-graphics/terminus-font/terminus-font_4.38.bb
index c460fc35c..23a73b37b 100644
--- a/meta-oe/recipes-graphics/terminus-font/terminus-font_4.38.bb
+++ b/meta-oe/recipes-graphics/terminus-font/terminus-font_4.38.bb
@@ -14,7 +14,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz"
 SRC_URI[md5sum] = "a8e792fe6e84c86ed2b6ed3e2a12ba66"
 SRC_URI[sha256sum] = 
"f6f4876a4dabe6a37c270c20bb9e141e38fb50e0bba200e1b9d0470e5eed97b7"

-inherit allarch fontcache
+inherit fontcache

 PACKAGECONFIG ?= "${@bb.utils.filter('DISTRO_FEATURES', 'x11', d)}"
 PACKAGECONFIG[x11] = ""
diff --git 
a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_1.004.bb 
b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_1.004.bb
index 9fbfc8b49..04ab188e6 100644
--- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_1.004.bb
+++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-cn-fonts_1.004.bb
@@ -5,7 +5,7 @@ HOMEPAGE = "https://github.com/adobe-fonts/source-han-sans";
 LICENSE = "OFL-1.1"
 LIC_FILES_CHKSUM = "file://LICENSE.txt;md5=55719faa0112708e946b820b24b14097"

-inherit allarch fontcache
+inherit fontcache

 # Download tends to break - so - or not?
 #EXCLUDE_FROM_WORLD = "1"
diff --git 
a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_1.004.bb 
b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_1.004.bb
index 4a26a2f57..bb54e8aa5 100644
--- a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_1.004.bb
+++ b/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-jp-fonts_1.004.bb
@@ -5,7 +5,7 @@ HOMEPAGE = "https://github.com/adobe-fonts/source-han-sans";
 LICENSE = "OFL-1.1"
 LIC_FILES_CHKSUM = "file://LICENSE.txt;md5=55719faa0112708e946b820b24b14097"

-inherit allarch fontcache
+inherit fontcache

 # Download tends to break - so - or not?
 #EXCLUDE_FROM_WORLD = "1"
diff --git 
a/meta-oe/recipes-graphics/ttf-fonts/source-han-sans-kr-fon

Re: [OE-core] [PATCH] gcc10: Don't default back to -fcommon

[Jacob Kroon]

On 8/5/20 8:17 AM, Khem Raj wrote:

here is a recent build with clang11 which also switched to using
-fno-common and errors are pretty much same you will see with gcc10
with this patch to revert the -fcommon defaults.

https://errors.yoctoproject.org/Errors/Build/107234/?limit=100

I was waiting for upstream packages to get fixed before switching the
defaults back to -fno-common.



Yes I understand, maybe its better to hold this for now.

Having a list of recipes that needs fixing would probably speed things 
up. I can have a look at some of them.


Jacob
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141208): 
https://lists.openembedded.org/g/openembedded-core/message/141208
Mute This Topic: https://lists.openembedded.org/mt/75983138/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[OE-core] ✗ patchtest: failure for linux-yocto/5.4: update to v5.4.54

[Patchwork]

== Series Details ==

Series: linux-yocto/5.4: update to v5.4.54
Revision: 1
URL   : https://patchwork.openembedded.org/series/25469/
State : failure

== Summary ==


Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:



* Issue Series does not apply on top of target branch 
[test_series_merge_on_head] 
  Suggested fixRebase your series on top of targeted branch
  Targeted branch  master (currently at 10221e578f)



If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines: 
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141207): 
https://lists.openembedded.org/g/openembedded-core/message/141207
Mute This Topic: https://lists.openembedded.org/mt/76006686/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] linux-yocto/5.4: update to v5.4.54

[Bruce Ashfield]

From: Bruce Ashfield 

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

58a12e3368db Linux 5.4.54
c15d59b94511 ath9k: Fix regression with Atheros 9271
e6eb815beccc ath9k: Fix general protection fault in ath9k_hif_usb_rx_cb
6d4448ca54bc dm integrity: fix integrity recalculation that is improperly 
skipped
2ca71b807383 ASoC: topology: fix tlvs in error handling for widget_dmixer
a4fd00dd8299 ASoC: topology: fix kernel oops on route addition error
e60e53e685d9 ASoC: qcom: Drop HAS_DMA dependency to fix link failure
8f64dc9e1d49 ASoC: rt5670: Add new gpio1_is_ext_spk_en quirk and enable it 
on the Lenovo Miix 2 10
697bd3e4aa4b x86, vmlinux.lds: Page-align end of ..page_aligned sections
c89af82f64a0 parisc: Add atomic64_set_release() define to avoid CPU soft 
lockups
d1bab3cf71dd drm/amd/powerplay: fix a crash when overclocking Vega M
33ab3f2dc444 drm/amdgpu: Fix NULL dereference in dpm sysfs handlers
c3de96686db9 mmc: sdhci-of-aspeed: Fix clock divider calculation
615f44e04792 io-mapping: indicate mapping failure
40c5836b4a48 khugepaged: fix null-pointer dereference due to race
95750e1edbcd mm: memcg/slab: fix memory leak at non-root kmem_cache destroy
db949f60d983 mm/memcg: fix refcount error while moving and swapping
549bfc142706 mm/mmap.c: close race between munmap() and 
expand_upwards()/downwards()
5835e6d5988f Makefile: Fix GCC_TOOLCHAIN_DIR prefix for Clang cross 
compilation
23e8b741c8a1 vt: Reject zero-sized screen buffer size.
028b478f2231 fbdev: Detect integer underflow at "struct 
fbcon_ops"->clear_margins.
bf331efc8ea4 /dev/mem: Add missing memory barriers for devmem_inode
3c52751df236 serial: 8250_mtk: Fix high-speed baud rates clamping
af811869db06 serial: 8250: fix null-ptr-deref in serial8250_start_tx()
fb8d832978bb serial: tegra: fix CREAD handling for PIO
c76a1dacc28d staging: comedi: addi_apci_1564: check 
INSN_CONFIG_DIGITAL_TRIG shift
178a09b0fb0d staging: comedi: addi_apci_1500: check 
INSN_CONFIG_DIGITAL_TRIG shift
7ee8d78bc12b staging: comedi: ni_6527: fix INSN_CONFIG_DIGITAL_TRIG support
747558b1c737 staging: comedi: addi_apci_1032: check 
INSN_CONFIG_DIGITAL_TRIG shift
c9afe420c53a staging: wlan-ng: properly check endpoint types
a44c859323c2 tty: xilinx_uartps: Really fix id assignment
f32718cfa5db iwlwifi: mvm: don't call iwl_mvm_free_inactive_queue() under 
RCU
3e84602475f7 Revert "cifs: Fix the target file was deleted when rename 
failed."
86894c3797ed usb: xhci: Fix ASM2142/ASM3142 DMA addressing
1d91547f2fc8 usb: xhci-mtk: fix the failure of bandwidth allocation
93f1e16af4a5 binder: Don't use mmput() from shrinker function.
35728cac176a RISC-V: Upgrade smp_mb__after_spinlock() to iorw,iorw
5345ede4acde drivers/perf: Prevent forced unbinding of PMU drivers
0821295b23cc asm-generic/mmiowb: Allow mmiowb_set_pending() when 
preemptible()
90e78ec7d725 x86: math-emu: Fix up 'cmp' insn for clang ias
679fe09188c1 arm64: Use test_tsk_thread_flag() for checking TIF_SINGLESTEP
7fc7942f52cf drivers/perf: Fix kernel panic when rmmod PMU modules during 
perf sampling
347f735ca82b ALSA: hda/realtek - fixup for yet another Intel reference board
30a17b51d80d hwmon: (scmi) Fix potential buffer overflow in 
scmi_hwmon_probe()
76361edb5559 platform/x86: asus-wmi: allow BAT1 battery name
41a7fdf90ce2 platform/x86: ISST: Add new PCI device ids
ace6e8b448b9 hwmon: (nct6775) Accept PECI Calibration as temperature source 
for NCT6798D
6627a265c598 drm/amdgpu: fix preemption unit test
ffb5604d2043 drm/amdgpu/gfx10: fix race condition for kiq
c04a48251314 hwmon: (adm1275) Make sure we are reading enough data for 
different chips
a2a380bd4575 usb: cdns3: trace: fix some endian issues
103a90ad4e64 usb: cdns3: ep0: fix some endian issues
89fe6eba178d usb: gadget: udc: gr_udc: fix memleak on error handling path 
in gr_ep_init()
74ec2cc5bfff usb: dwc3: pci: add support for the Intel Jasper Lake
c4c6363b8e68 usb: dwc3: pci: add support for the Intel Tiger Lake PCH -H 
variant
4f5eb2735fa3 Input: elan_i2c - only increment wakeup count on touch
186d3fe73e27 Input: synaptics - enable InterTouch for ThinkPad X1E 1st gen
460c0dafea92 dmaengine: ioat setting ioat timeout as module parameter
493aed3263ca dmaengine: fsl-edma: fix wrong tcd endianness for big-endian 
cpu
6a3015ae35f5 hwmon: (aspeed-pwm-tacho) Avoid possible buffer overflow
01d7bd8903d8 regmap: dev_get_regmap_match(): fix string comparison
bbc0b6e18405 spi: mediatek: use correct SPI_CFG2_REG MACRO
126a0ab6b83b ARM: dts: n900: remove mmc1 card detect gpio
80fed4024c39 Input: add `SW_MACHINE_COVER`
db886ec71fe4 dmaengine: tegra210-adma: Fix runtime PM imbalance on error
5cbe437d5968 HID: apple: Disable Fn-key key-re-mapping on clone keyboards
2

Re: [OE-core] [PATCH 0/4] kernel/yocto: conslidated pull request

[Bruce Ashfield]

On Wed, Aug 5, 2020 at 7:06 AM Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:

> On Fri, 2020-07-17 at 08:56 -0400, bruce.ashfi...@gmail.com wrote:
> > Here's my latest set of patches.
> >
> > For 5.4: a -stable update, and -rt bug fix
> >
> > For the build: a tweak to the config generation (as reported
> > and discussed on the mailing list)
> >
> > For 5.8 (aka the next reference kernel): a fix for devsrc and
> > on target kernel building. With this tweak, I have all of the
> > refrences booting and passing most tests on 5.8, so we are in
> > good shape on that front. I'll be doing libc-headers testing
> > next.
> >
> > I ran these through the AB and local testing, no issues popped
> > up, except a warning on qemuarm-alt in my last run. It was
> > about a console login and didn't look related to the change,
> > but I'm keeping my eye on it.
>
> Thanks, I ran this combined with Khem's binutils change through the
> autobuilder and your changes are likely fine but we need more work to
> get binutils in. Basically:
>
> * meta-arm needs to add the kernel patch for perf
> * meta-intel needs to add the kernel patch for perf
> * the hardware BSPs need to add the kernel patch for perf
>
> I've added a few maintainers/people to the address list to try and sort
> that out. Who is going to handle the hardware platforms?
>

I have a meta-yocto-bsp bump that I did for the latest 5.4, it'll have that
fix as well. I can send it out now for the reference platforms.

Bruce



>
> Cheers,
>
> Richard
>
>
>
>
>

-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141205): 
https://lists.openembedded.org/g/openembedded-core/message/141205
Mute This Topic: https://lists.openembedded.org/mt/75611071/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 0/4] kernel/yocto: conslidated pull request

[Andrey Zhizhikin]

On Wed, Aug 5, 2020 at 1:06 PM Richard Purdie
 wrote:
>
> On Fri, 2020-07-17 at 08:56 -0400, bruce.ashfi...@gmail.com wrote:
> > Here's my latest set of patches.
> >
> > For 5.4: a -stable update, and -rt bug fix
> >
> > For the build: a tweak to the config generation (as reported
> > and discussed on the mailing list)
> >
> > For 5.8 (aka the next reference kernel): a fix for devsrc and
> > on target kernel building. With this tweak, I have all of the
> > refrences booting and passing most tests on 5.8, so we are in
> > good shape on that front. I'll be doing libc-headers testing
> > next.
> >
> > I ran these through the AB and local testing, no issues popped
> > up, except a warning on qemuarm-alt in my last run. It was
> > about a console login and didn't look related to the change,
> > but I'm keeping my eye on it.
>
> Thanks, I ran this combined with Khem's binutils change through the
> autobuilder and your changes are likely fine but we need more work to
> get binutils in. Basically:
>
> * meta-arm needs to add the kernel patch for perf
> * meta-intel needs to add the kernel patch for perf
> * the hardware BSPs need to add the kernel patch for perf

meta-freescale has this patch already included for 5.4.y kernel:
https://github.com/Freescale/linux-fslc/commit/415afad5ca95f303000b58b7c77871e99c8244b5

>
> I've added a few maintainers/people to the address list to try and sort
> that out. Who is going to handle the hardware platforms?
>
> Cheers,
>
> Richard
>
>
>
>
> 



-- 
Regards,
Andrey.
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141204): 
https://lists.openembedded.org/g/openembedded-core/message/141204
Mute This Topic: https://lists.openembedded.org/mt/75611071/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 0/4] kernel/yocto: conslidated pull request

[Richard Purdie]

On Wed, 2020-08-05 at 12:06 +0100, Richard Purdie via
lists.openembedded.org wrote:
> On Fri, 2020-07-17 at 08:56 -0400, bruce.ashfi...@gmail.com wrote:
> > Here's my latest set of patches.
> > 
> > For 5.4: a -stable update, and -rt bug fix
> > 
> > For the build: a tweak to the config generation (as reported
> > and discussed on the mailing list)
> > 
> > For 5.8 (aka the next reference kernel): a fix for devsrc and
> > on target kernel building. With this tweak, I have all of the
> > refrences booting and passing most tests on 5.8, so we are in
> > good shape on that front. I'll be doing libc-headers testing
> > next.
> > 
> > I ran these through the AB and local testing, no issues popped
> > up, except a warning on qemuarm-alt in my last run. It was
> > about a console login and didn't look related to the change,
> > but I'm keeping my eye on it.
> 
> Thanks, I ran this combined with Khem's binutils change through the
> autobuilder and your changes are likely fine but we need more work to
> get binutils in. Basically:
> 
> * meta-arm needs to add the kernel patch for perf
> * meta-intel needs to add the kernel patch for perf
> * the hardware BSPs need to add the kernel patch for perf
> 
> I've added a few maintainers/people to the address list to try and
> sort
> that out. Who is going to handle the hardware platforms?

Sorry, the url for the failing builds:

https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1231

The kernel tweak needed for binutils 2.35:

http://git.yoctoproject.org/cgit/cgit.cgi/linux-yocto/commit/?h=v5.4/standard/base&id=1fe4ee7f680c3f69f620d68a5d2f7341e8db693b

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141203): 
https://lists.openembedded.org/g/openembedded-core/message/141203
Mute This Topic: https://lists.openembedded.org/mt/75611071/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 0/4] kernel/yocto: conslidated pull request

[Richard Purdie]

On Fri, 2020-07-17 at 08:56 -0400, bruce.ashfi...@gmail.com wrote:
> Here's my latest set of patches.
> 
> For 5.4: a -stable update, and -rt bug fix
> 
> For the build: a tweak to the config generation (as reported
> and discussed on the mailing list)
> 
> For 5.8 (aka the next reference kernel): a fix for devsrc and
> on target kernel building. With this tweak, I have all of the
> refrences booting and passing most tests on 5.8, so we are in
> good shape on that front. I'll be doing libc-headers testing
> next.
> 
> I ran these through the AB and local testing, no issues popped
> up, except a warning on qemuarm-alt in my last run. It was
> about a console login and didn't look related to the change,
> but I'm keeping my eye on it.

Thanks, I ran this combined with Khem's binutils change through the
autobuilder and your changes are likely fine but we need more work to
get binutils in. Basically:

* meta-arm needs to add the kernel patch for perf
* meta-intel needs to add the kernel patch for perf
* the hardware BSPs need to add the kernel patch for perf

I've added a few maintainers/people to the address list to try and sort
that out. Who is going to handle the hardware platforms?

Cheers,

Richard




-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141202): 
https://lists.openembedded.org/g/openembedded-core/message/141202
Mute This Topic: https://lists.openembedded.org/mt/75611071/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] wic: fix typo

[Daniel Ammann]

Signed-off-by: Daniel Ammann 
---
 scripts/lib/wic/plugins/source/bootimg-partition.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/lib/wic/plugins/source/bootimg-partition.py 
b/scripts/lib/wic/plugins/source/bootimg-partition.py
index 138986a71e..5dbe2558d2 100644
--- a/scripts/lib/wic/plugins/source/bootimg-partition.py
+++ b/scripts/lib/wic/plugins/source/bootimg-partition.py
@@ -141,7 +141,7 @@ class BootimgPartitionPlugin(SourcePlugin):
 break
 
 if not kernel_name:
-raise WicError('No kernel file founded')
+raise WicError('No kernel file found')
 
 # Compose the extlinux.conf
 extlinux_conf = "default Yocto\n"
-- 
2.20.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141201): 
https://lists.openembedded.org/g/openembedded-core/message/141201
Mute This Topic: https://lists.openembedded.org/mt/76004838/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] libunwind: Use -fcommon option to compile

[Adrian Bunk]

On Tue, Aug 04, 2020 at 11:14:48PM -0700, Khem Raj wrote:
> defaults for gcc is to use -fno-common this ensures that it keeps using
> -fcommon until the real issue is fixed
> 
> Fixes
> src/arm/Ginit.c:60: multiple definition of `_U_dyn_info_list'; 
> mi/.libs/dyn-info-list.o:/usr/src/debug/libunwind/1.4.0-r0/build/src/../../libunwind-1.4.0/src/mi/dyn-info-list.c:28:
>  first defined here
>...

The upstream fix is
https://github.com/libunwind/libunwind/commit/29e17d8d2ccbca07c423e3089a6d5ae8a1c9cb6e

cu
Adrian
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141200): 
https://lists.openembedded.org/g/openembedded-core/message/141200
Mute This Topic: https://lists.openembedded.org/mt/76002561/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] file checksums versus SRC_URI globs

[Rasmus Villemoes]

On 09/07/2020 21.31, Rasmus Villemoes via lists.openembedded.org wrote:
> On 02/07/2020 08.42, Rasmus Villemoes via lists.openembedded.org wrote:
>> On 01/07/2020 16.03, Quentin Schulz wrote:
>>> Hi Rasmus,
>>>
>>> On Wed, Jul 01, 2020 at 03:51:19PM +0200, Rasmus Villemoes wrote:
 Hi,

 We have a recipe that uses

   SRC_URI += "file://somedir/*"

>>>
>>> Glob aren't supported. Use "file://somedir/" instead.
>>
>> Thanks, that actually works for one of the cases we have (there are
>> others that use globs which cannot be solved quite that simply, but for
>> now I'm just listing files explicitly instead).
>>
>> However, I'm not sure that "globs aren't supported". The commit I
>> referenced clearly tried to make that work (better), it also "works" in
>> the sense of unpacking the expected things when building from scratch -
>> there's even
>>
>> def test_local_wildcard(self):
>> tree = self.fetchUnpack(['file://a', 'file://dir/*'])
>> self.assertEqual(tree, ['a',  'dir/c', 'dir/d', 'dir/subdir/e'])
>>
>> in bitbake/lib/bb/tests/fetch.py. And the two upstream recipes
>> connman-gnome_0.7.bb and matchbox-desktop_2.2.bb both use that exact
>> pattern.
>>
>> So either
>>
>> - this is a plain bug in the signature computation,
> 
> I'm guessing the culprit is
> 
> commit 6c0706a28d72c591f1b75b6e3f3b645859387c7e
> Author: Richard Purdie 
> Date:   Mon Dec 8 21:25:23 2014 +
> 
> cache/fetch2/siggen: Ensure we track include history for file checksums
> 
> which was the one that introduced the ":True: or ":False" suffixing via
> 
> +filelist.append(f + ":" + str(os.path.exists(f)))
> 
> and then also did
> 
> @@ -981,6 +980,10 @@ def get_file_checksums(filelist, pn):
> 
>  checksums = []
>  for pth in filelist.split():
> +exist = pth.split(":")[1]
> +if exist == "False":
> +continue
> +pth = pth.split(":")[0]
>  if '*' in pth:
>  # Handle globs
>  for f in glob.glob(pth):
> 
> which practically guaranteed that the "if '*' in pth" would be dead code.
> 
> Richard, do you agree that this is a bug in the signature computation?

Richard, ping?

Rasmus
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141199): 
https://lists.openembedded.org/g/openembedded-core/message/141199
Mute This Topic: https://lists.openembedded.org/mt/75235594/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] minicom: Fix build when using -fno-common

[Khem Raj]

gcc10 defaults to -fno-common which flags the multiple global variables
as errors, these patches ( from fedora ) fixes the issue

Signed-off-by: Khem Raj 
---
 ...erfluous-global-variable-definitions.patch | 35 
 ...erfluous-global-variable-definitions.patch | 37 
 ...erfluous-global-variable-definitions.patch | 42 +++
 .../recipes-extended/minicom/minicom_2.7.1.bb |  3 ++
 4 files changed, 117 insertions(+)
 create mode 100644 
meta/recipes-extended/minicom/minicom/0001-Drop-superfluous-global-variable-definitions.patch
 create mode 100644 
meta/recipes-extended/minicom/minicom/0002-Drop-superfluous-global-variable-definitions.patch
 create mode 100644 
meta/recipes-extended/minicom/minicom/0003-Drop-superfluous-global-variable-definitions.patch

diff --git 
a/meta/recipes-extended/minicom/minicom/0001-Drop-superfluous-global-variable-definitions.patch
 
b/meta/recipes-extended/minicom/minicom/0001-Drop-superfluous-global-variable-definitions.patch
new file mode 100644
index 00..4c6e249315
--- /dev/null
+++ 
b/meta/recipes-extended/minicom/minicom/0001-Drop-superfluous-global-variable-definitions.patch
@@ -0,0 +1,35 @@
+From b65152ebc03832972115e6d98e50cb6190d01793 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Ond=C5=99ej=20Lyson=C4=9Bk?= 
+Date: Mon, 3 Feb 2020 13:18:13 +0100
+Subject: [PATCH 1/3] Drop superfluous global variable definitions
+
+The file minicom.c, by including the minicom.h header, already defines
+the global variables 'dial_user' and 'dial_pass'. The object file
+minicom.o is always linked to dial.o. Thus the definitions in dial.c
+can be dropped.
+
+This fixes linking with gcc 10 which uses -fno-common by default,
+disallowing multiple global variable definitions.
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj 
+---
+ src/dial.c | 2 --
+ 1 file changed, 2 deletions(-)
+
+diff --git a/src/dial.c b/src/dial.c
+index eada5ee..d9d481f 100644
+--- a/src/dial.c
 b/src/dial.c
+@@ -146,8 +146,6 @@ static int newtype;
+ /* Access to ".dialdir" denied? */
+ static int dendd = 0;
+ static char *tagged;
+-char *dial_user;
+-char *dial_pass;
+ 
+ /* Change the baud rate.  Treat all characters in the given array as if
+  * they were key presses within the comm parameters dialog (C-A P) and
+-- 
+2.24.1
+
diff --git 
a/meta/recipes-extended/minicom/minicom/0002-Drop-superfluous-global-variable-definitions.patch
 
b/meta/recipes-extended/minicom/minicom/0002-Drop-superfluous-global-variable-definitions.patch
new file mode 100644
index 00..1740051e0a
--- /dev/null
+++ 
b/meta/recipes-extended/minicom/minicom/0002-Drop-superfluous-global-variable-definitions.patch
@@ -0,0 +1,37 @@
+From 924bd2da3a00e030e29d82b74ef82900bd50b475 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Ond=C5=99ej=20Lyson=C4=9Bk?= 
+Date: Mon, 3 Feb 2020 13:18:33 +0100
+Subject: [PATCH 2/3] Drop superfluous global variable definitions
+
+The only place where the EXTERN macro mechanism is used to define the
+global variables 'vt_outmap' and 'vt_inmap' is minicom.c (by defining
+an empty EXTERN macro and including the minicom.h header). The file
+vt100.c already defines these variables. The vt100.o object file is
+always linked to minicom.o. Thus it is safe not to define the
+variables in minicom.c and only declare them in the minicom.h header.
+
+This fixes linking with gcc 10 which uses -fno-common by default,
+disallowing multiple global variable definitions.
+
+Upstream-Status: Pending
+Signed-off-by: Khem Raj 
+---
+ src/minicom.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/minicom.h b/src/minicom.h
+index 061c013..0f9693b 100644
+--- a/src/minicom.h
 b/src/minicom.h
+@@ -141,7 +141,7 @@ EXTERN int sbcolor; /* Status Bar Background Color */
+ EXTERN int st_attr;   /* Status Bar attributes. */
+ 
+ /* jl 04.09.97 conversion tables */
+-EXTERN unsigned char vt_outmap[256], vt_inmap[256];
++extern unsigned char vt_outmap[256], vt_inmap[256];
+ 
+ /* MARK updated 02/17/95 - history buffer */
+ EXTERN int num_hist_lines;  /* History buffer size */
+-- 
+2.24.1
+
diff --git 
a/meta/recipes-extended/minicom/minicom/0003-Drop-superfluous-global-variable-definitions.patch
 
b/meta/recipes-extended/minicom/minicom/0003-Drop-superfluous-global-variable-definitions.patch
new file mode 100644
index 00..58cd58eda8
--- /dev/null
+++ 
b/meta/recipes-extended/minicom/minicom/0003-Drop-superfluous-global-variable-definitions.patch
@@ -0,0 +1,42 @@
+From a4fc603b3641d2efe31479116eb7ba66932901c7 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Ond=C5=99ej=20Lyson=C4=9Bk?= 
+Date: Mon, 3 Feb 2020 13:21:41 +0100
+Subject: [PATCH 3/3] Drop superfluous global variable definitions
+
+The only place where the EXTERN macro mechanism is used to define the
+global variables 'portfd_is_socket', 'portfd_is_connected' and
+'portfd_sock_addr' is minicom.c (by defining an empty EXTERN macro and
+including the minicom.h header). The source file sysdep1_s.c alre