date:20211004

[OE-core] [PATCH v2] buildhistory: Fix package output files for SDKs

2021-10-04 Thread Andres Beltran

Currently, installed packages are listed for images in image-info.txt, but
not for SDKs in sdk-info.txt. Add TOOLCHAIN_HOST_TASK and
TOOLCHAIN_TARGET_TASK to the output variables in sdk-info.txt.

Moreover, package output files for the SDK host are empty because
PKGDATA_DIR defaults to the target directory. Fix this bug and create a new
variable called PKGDATA_DIR_SDK which stores the correct path for the SDK
host package data.

Signed-off-by: Andres Beltran 
---
 meta/classes/buildhistory.bbclass   | 21 -
 meta/classes/cross-canadian.bbclass |  2 +-
 meta/classes/nativesdk.bbclass  |  2 +-
 meta/conf/bitbake.conf  |  1 +
 meta/lib/oe/package_manager/__init__.py |  2 +-
 5 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/meta/classes/buildhistory.bbclass 
b/meta/classes/buildhistory.bbclass
index a613306270..7c44fec2d1 100644
--- a/meta/classes/buildhistory.bbclass
+++ b/meta/classes/buildhistory.bbclass
@@ -442,11 +442,16 @@ def buildhistory_list_installed(d, rootfs_type="image"):
 else:
 pkgs = sdk_list_installed_packages(d, rootfs_type == "sdk_target")
 
+if rootfs_type == "sdk_host":
+pkgdata_dir = d.getVar('PKGDATA_DIR_SDK')
+else:
+pkgdata_dir = d.getVar('PKGDATA_DIR')
+
 for output_type, output_file in process_list:
 output_file_full = os.path.join(d.getVar('WORKDIR'), output_file)
 
 with open(output_file_full, 'w') as output:
-output.write(format_pkg_list(pkgs, output_type, 
d.getVar('PKGDATA_DIR')))
+output.write(format_pkg_list(pkgs, output_type, pkgdata_dir))
 
 python buildhistory_list_installed_image() {
 buildhistory_list_installed(d)
@@ -496,13 +501,19 @@ buildhistory_get_installed() {
echo "}" >>  $1/depends.dot
rm $1/depends.tmp
 
+   # Set correct pkgdatadir
+   pkgdatadir=${PKGDATA_DIR}
+   if [ "$2" == "sdk" ] && [ "$3" == "host" ]; then
+   pkgdatadir="${PKGDATA_DIR_SDK}"
+   fi
+
# Produce installed package sizes list
-   oe-pkgdata-util -p ${PKGDATA_DIR} read-value "PKGSIZE" -n -f $pkgcache 
> $1/installed-package-sizes.tmp
+   oe-pkgdata-util -p $pkgdatadir read-value "PKGSIZE" -n -f $pkgcache > 
$1/installed-package-sizes.tmp
cat $1/installed-package-sizes.tmp | awk '{print $2 "\tKiB\t" $1}' | 
sort -n -r > $1/installed-package-sizes.txt
rm $1/installed-package-sizes.tmp
 
# Produce package info: runtime_name, buildtime_name, recipe, version, 
size
-   oe-pkgdata-util -p ${PKGDATA_DIR} read-value "PACKAGE,PN,PV,PKGSIZE" -n 
-f $pkgcache > $1/installed-package-info.tmp
+   oe-pkgdata-util -p $pkgdatadir read-value "PACKAGE,PN,PV,PKGSIZE" -n -f 
$pkgcache > $1/installed-package-info.tmp
cat $1/installed-package-info.tmp | sort -n -r -k 5 > 
$1/installed-package-info.txt
rm $1/installed-package-info.tmp
 
@@ -542,7 +553,7 @@ buildhistory_get_sdk_installed() {
return
fi
 
-   buildhistory_get_installed ${BUILDHISTORY_DIR_SDK}/$1 sdk
+   buildhistory_get_installed ${BUILDHISTORY_DIR_SDK}/$1 sdk $1
 }
 
 buildhistory_get_sdk_installed_host() {
@@ -773,7 +784,7 @@ def buildhistory_get_imagevars(d):
 def buildhistory_get_sdkvars(d):
 if d.getVar('BB_WORKERCONTEXT') != '1':
 return ""
-sdkvars = "DISTRO DISTRO_VERSION SDK_NAME SDK_VERSION SDKMACHINE 
SDKIMAGE_FEATURES BAD_RECOMMENDATIONS NO_RECOMMENDATIONS PACKAGE_EXCLUDE"
+sdkvars = "DISTRO DISTRO_VERSION SDK_NAME SDK_VERSION SDKMACHINE 
SDKIMAGE_FEATURES TOOLCHAIN_HOST_TASK TOOLCHAIN_TARGET_TASK BAD_RECOMMENDATIONS 
NO_RECOMMENDATIONS PACKAGE_EXCLUDE"
 if d.getVar('BB_CURRENTTASK') == 'populate_sdk_ext':
 # Extensible SDK uses some additional variables
 sdkvars += " SDK_LOCAL_CONF_WHITELIST SDK_LOCAL_CONF_BLACKLIST 
SDK_INHERIT_BLACKLIST SDK_UPDATE_URL SDK_EXT_TYPE SDK_RECRDEP_TASKS 
SDK_INCLUDE_PKGDATA SDK_INCLUDE_TOOLCHAIN"
diff --git a/meta/classes/cross-canadian.bbclass 
b/meta/classes/cross-canadian.bbclass
index ffbc2167e3..ac82e86356 100644
--- a/meta/classes/cross-canadian.bbclass
+++ b/meta/classes/cross-canadian.bbclass
@@ -169,7 +169,7 @@ USE_NLS = "${SDKUSE_NLS}"
 # and not any particular tune that is enabled.
 TARGET_ARCH[vardepsexclude] = "TUNE_ARCH"
 
-PKGDATA_DIR = "${TMPDIR}/pkgdata/${SDK_SYS}"
+PKGDATA_DIR = "${PKGDATA_DIR_SDK}"
 # If MLPREFIX is set by multilib code, shlibs
 # points to the wrong place so force it
 SHLIBSDIRS = "${PKGDATA_DIR}/nativesdk-shlibs2"
diff --git a/meta/classes/nativesdk.bbclass b/meta/classes/nativesdk.bbclass
index c66de8c787..14e210562f 100644
--- a/meta/classes/nativesdk.bbclass
+++ b/meta/classes/nativesdk.bbclass
@@ -31,7 +31,7 @@ PACKAGE_ARCHS = "${SDK_PACKAGE_ARCHS}"
 DEPENDS:append = " chrpath-replacement-native"
 EXTRANATIVEPATH += "chrpath-native"
 
-PKGDATA_DIR = "${TMPDIR}/pkgdata/${SDK_SYS}"
+PKGDATA_DIR = "${PKGDATA_DIR_SDK}"
 
 HOST_ARCH = "${SDK_ARCH}"

[OE-core] Yocto Project Newcomer & Unassigned Bugs - Help Needed

2021-10-04 Thread Stephen Jolley

All,

 

The triage team is starting to try and collect up and classify bugs which a
newcomer to the project would be able to work on in a way which means people
can find them. They're being listed on the triage page under the appropriate
heading:

https://wiki.yoctoproject.org/wiki/Bug_Triage#Newcomer_Bugs  Also please
review:
https://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded and
how to create a bugzilla account at:

https://bugzilla.yoctoproject.org/createaccount.cgi

The idea is these bugs should be straight forward for a person to help work
on who doesn't have deep experience with the project.  If anyone can help,
please take ownership of the bug and send patches!  If anyone needs
help/advice there are people on irc who can likely do so, or some of the
more experienced contributors will likely be happy to help too.

 

Also, the triage team meets weekly and does its best to handle the bugs
reported into the Bugzilla. The number of people attending that meeting has
fallen, as have the number of people available to help fix bugs. One of the
things we hear users report is they don't know how to help. We (the triage
team) are therefore going to start reporting out the currently 392
unassigned or newcomer bugs.

 

We're hoping people may be able to spare some time now and again to help out
with these.  Bugs are split into two types, "true bugs" where things don't
work as they should and "enhancements" which are features we'd want to add
to the system.  There are also roughly four different "priority" classes
right now, "3.4", "3.5, "3.99" and "Future", the more pressing/urgent issues
being in "3.4" and then "3.5".

 

Please review this link and if a bug is something you would be able to help
with either take ownership of the bug, or send me (sjolley.yp...@gmail.com
 ) an e-mail with the bug number you would
like and I will assign it to you (please make sure you have a Bugzilla
account).  The list is at:
https://wiki.yoctoproject.org/wiki/Bug_Triage_Archive#Unassigned_or_Newcomer
_Bugs

 

Thanks,

 

Stephen K. Jolley

Yocto Project Program Manager

*Cell:(208) 244-4460

* Email:  sjolley.yp...@gmail.com
 

 


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

[OE-core] Reminder: Yocto Project Technical Team Meeting @ Monthly from 8am on the first Tuesday (PDT)

2021-10-04 Thread Stephen Jolley

All,

 

Just a reminder we will hold the monthly Yocto Project Technical Meeting at
8am PST tomorrow. (10/5)  

 

Yocto Project Technical Team Meeting: We encourage people attending the
meeting to logon and announce themselves on the Yocto Project IRC chancel
during the meeting (optional):

Yocto IRC: https://web.libera.chat/#yocto 

Wiki: https://www.yoctoproject.org/public-virtual-meetings/

 

WhenMonthly from 8am to 9am on the first Tuesday Pacific Time

Where   Zoom Meeting:
https://zoom.us/j/990892712?pwd=cHU1MjhoM2x6ck81bkcrYjRrcmJsUT09

 

We are tracking the minutes at:
https://docs.google.com/document/d/1ly8nyhO14kDNnFcW2QskANXW3ZT7QwKC5wWVDg9d
DH4/edit?pli=1 Please request access if you want to assist in editing them.
The world should have view access. 

 

Thanks,

 

Stephen K. Jolley

Yocto Project Program Manager

*Cell:(208) 244-4460

* Email:  sjolley.yp...@gmail.com
 

 


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156634): 
https://lists.openembedded.org/g/openembedded-core/message/156634
Mute This Topic: https://lists.openembedded.org/mt/86082843/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 04/10] cmake: update 3.20.2 -> 3.20.3

2021-10-04 Thread Michael Halstead

I'll go ahead and switch to CentOS8-Stream once the current a-full is
complete. If there is a need to follow RHEL8 exactly we can add a Rocky
Linux worker in the future.


On Mon, Oct 4, 2021 at 11:28 AM Alexander Kanavin 
wrote:

> Since the original centos 8 is going away (support-wise) in less than 3
> months, should we just convert all workers to stream and move on?
>
> Alex
>
> On Mon, 4 Oct 2021 at 18:27, Michael Halstead <
> mhalst...@linuxfoundation.org> wrote:
>
>> This error was fixed upstream
>> https://bugzilla.redhat.com/show_bug.cgi?id=1975562 kernel 4.18.0-325.x
>> which has landed in CentOS8 Stream on centos8-ty-2 but not plain CentOS8
>> where the newest kernel is still at 4.18.0-305.x.
>>
>> I'll reboot into the older 4.18.0-240.15 kernel again on centos8-ty-1 as
>> a fix for now.
>>
>> On Tue, Sep 28, 2021 at 6:12 PM Anuj Mittal 
>> wrote:
>>
>>> Did we find out the reason why this was happening? I have started
>>> getting this error on Centos-8 while building hardknott.
>>>
>>> https://autobuilder.yoctoproject.org/typhoon/#/builders/63/builds/4045
>>>
>>> Thanks,
>>>
>>> Anuj
>>>
>>> On Wed, 2021-06-16 at 23:45 +0100, Richard Purdie wrote:
>>> > On Sun, 2021-06-06 at 21:51 +0200, Alexander Kanavin wrote:
>>> > > On Sun, 6 Jun 2021 at 01:10, Richard Purdie
>>> > >  wrote:
>>> > > > I tried again with the autobuilder, still fails:
>>> > > >
>>> > > >
>>> https://autobuilder.yoctoproject.org/typhoon/#/builders/48/builds/3516
>>> > > >
>>> > > > so whatever it is, it is still "live".
>>> > > >
>>> > >
>>> > >
>>> > > I did some digging. The issue happens when:
>>> > > - host is centos8
>>> > > - SDKMACHINE is i686 (e.g. cmake is 32 bit)
>>> > >
>>> > > Then there's a failing syscall attempting to set file times:
>>> > > utimensat_time64(AT_FDCWD, "../install/usr/local/lib/cmake/assimp-
>>> > > 4.1/assimp-config.cmake",
>>> > > [{tv_sec=1622966723, tv_nsec=6319439026193432576},
>>> > > {tv_sec=1622966579, tv_nsec=17840053692309438464}], 0) = -1
>>> > > EINVAL (Invalid argument)
>>> > >
>>> > > On latest Fedora, there's no issue:
>>> > > utimensat_time64(AT_FDCWD, "../install2/usr/local/lib/cmake/assimp-
>>> > > 4.1/assimp-config.cmake",
>>> > > [{tv_sec=1623002886, tv_nsec=6369724778172907520},
>>> > > {tv_sec=1623002886, tv_nsec=17839174083007217664}], 0) = 0
>>> > >
>>> > > utimensat_time64 only appeared with 5.1 kernels, however, 4.18 should
>>> > > be returning ENOSYS in that case
>>> > > probably?
>>> >
>>> > I hacked up a quick test bit of code (which makes assumptions
>>> > about 32 bit):
>>> >
>>> > #include 
>>> > #include 
>>> > #include 
>>> > #include 
>>> > #include 
>>> > #include 
>>> >
>>> > struct timespec64 {
>>> > long long   tv_sec; /* seconds */
>>> > long long   tv_nsec;/* nanoseconds */
>>> > };
>>> >
>>> > int main() {
>>> >   int fd = open("foo", O_RDWR | O_CREAT, 0644);
>>> >   write(fd, "foo", 3);
>>> >   struct timespec64 times[2] = {};
>>> >   times[0].tv_sec = 1622966723;
>>> >   times[0].tv_nsec = 631943;
>>> >   times[1].tv_sec = 1622966579;
>>> >   times[1].tv_nsec = 178400;
>>> >   int rc = syscall(SYS_utimensat_time64, fd, NULL, ×[0], 0);
>>> >   printf("rc=%d\n", rc);
>>> >   close(fd);
>>> >   return rc;
>>> > }
>>> >
>>> > built with "gcc -m32 test-syscall.c -o test" and run with "strace
>>> > ./test".
>>> > This works on all the systems I tried it in. As does:
>>> >
>>> >
>>> >   times[0].tv_sec = 1;
>>> >   times[0].tv_nsec = 2;
>>> >   times[1].tv_sec = 3;
>>> >   times[1].tv_nsec = 4;
>>> >
>>> > however if you set (and ignore the compiler warning):
>>> >
>>> >   times[0].tv_sec = 1622966723;
>>> >   times[0].tv_nsec = 6319439026193432576;
>>> >   times[1].tv_sec = 1622966579;
>>> >   times[1].tv_nsec = 17840053692309438464;
>>> >
>>> > then you see EINVAL on the centos system but not on my ubuntu one. It
>>> > will
>>> > do that until you reduce the values of tv_nsec right now. So it seems
>>> > most
>>> > systems accept large tv_nsec values but the Centos one does not.
>>> >
>>> > I think tv_nsec may be being clamped to LONG_MAX of 4 bytes but should
>>> > be
>>> > a LONG_LONG_MAX of 8 bytes on a 32 bit since the field is a 64 bit
>>> > long.
>>> >
>>> > Michael: Hopefully that gives you something to raise with them?
>>> >
>>> > Cheers,
>>> >
>>> > Richard
>>> >
>>> >
>>> >
>>> >
>>> >
>>> >
>>> >
>>> >
>>>
>>>
>>> 
>>>
>>>
>>
>> --
>> Michael Halstead
>> Linux Foundation / Yocto Project
>> Systems Operations Engineer
>>
>

-- 
Michael Halstead
Linux Foundation / Yocto Project
Systems Operations Engineer

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156633): 
https://lists.openembedded.org/g/openembedded-core/message/156633
Mute This Topic: https://lists.openembedded.org/mt/83304703/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] buildhistory: Fix package output files for SDKs

2021-10-04 Thread Richard Purdie

Hi,

On Mon, 2021-10-04 at 20:13 +, Andres Beltran wrote:
> Currently, installed packages are listed for images in image-info.txt, but
> not for SDKs in sdk-info.txt. Add TOOLCHAIN_HOST_TASK and
> TOOLCHAIN_TARGET_TASK to the output variables in sdk-info.txt.
> 
> Moreover, package output files for the SDK host are empty because
> PKGDATA_DIR defaults to the target directory. Fix this bug and create a new
> variable called PKGDATA_DIR_SDK which stores the correct path for the SDK
> host package data.
> 
> Signed-off-by: Andres Beltran 
> ---
>  meta/classes/buildhistory.bbclass   | 21 -
>  meta/classes/cross-canadian.bbclass |  2 +-
>  meta/classes/nativesdk.bbclass  |  2 +-
>  meta/conf/bitbake.conf  |  1 +
>  meta/lib/oe/package_manager.py  |  2 +-
>  5 files changed, 20 insertions(+), 8 deletions(-)

The patch looks like a good fix but it doesn't apply against master since
package_mamager.py was split up. Could you rebase against master please?

Cheers,

Richard


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156632): 
https://lists.openembedded.org/g/openembedded-core/message/156632
Mute This Topic: https://lists.openembedded.org/mt/86077464/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] buildhistory: Fix package output files for SDKs

2021-10-04 Thread Andres Beltran

Currently, installed packages are listed for images in image-info.txt, but
not for SDKs in sdk-info.txt. Add TOOLCHAIN_HOST_TASK and
TOOLCHAIN_TARGET_TASK to the output variables in sdk-info.txt.

Moreover, package output files for the SDK host are empty because
PKGDATA_DIR defaults to the target directory. Fix this bug and create a new
variable called PKGDATA_DIR_SDK which stores the correct path for the SDK
host package data.

Signed-off-by: Andres Beltran 
---
 meta/classes/buildhistory.bbclass   | 21 -
 meta/classes/cross-canadian.bbclass |  2 +-
 meta/classes/nativesdk.bbclass  |  2 +-
 meta/conf/bitbake.conf  |  1 +
 meta/lib/oe/package_manager.py  |  2 +-
 5 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/meta/classes/buildhistory.bbclass 
b/meta/classes/buildhistory.bbclass
index 20609c435b..ab9d882752 100644
--- a/meta/classes/buildhistory.bbclass
+++ b/meta/classes/buildhistory.bbclass
@@ -434,11 +434,16 @@ def buildhistory_list_installed(d, rootfs_type="image"):
 else:
 pkgs = sdk_list_installed_packages(d, rootfs_type == "sdk_target")
 
+if rootfs_type == "sdk_host":
+pkgdata_dir = d.getVar('PKGDATA_DIR_SDK')
+else:
+pkgdata_dir = d.getVar('PKGDATA_DIR')
+
 for output_type, output_file in process_list:
 output_file_full = os.path.join(d.getVar('WORKDIR'), output_file)
 
 with open(output_file_full, 'w') as output:
-output.write(format_pkg_list(pkgs, output_type, 
d.getVar('PKGDATA_DIR')))
+output.write(format_pkg_list(pkgs, output_type, pkgdata_dir))
 
 python buildhistory_list_installed_image() {
 buildhistory_list_installed(d)
@@ -488,13 +493,19 @@ buildhistory_get_installed() {
echo "}" >>  $1/depends.dot
rm $1/depends.tmp
 
+   # Set correct pkgdatadir
+   pkgdatadir=${PKGDATA_DIR}
+   if [ "$2" == "sdk" ] && [ "$3" == "host" ]; then
+   pkgdatadir="${PKGDATA_DIR_SDK}"
+   fi
+
# Produce installed package sizes list
-   oe-pkgdata-util -p ${PKGDATA_DIR} read-value "PKGSIZE" -n -f $pkgcache 
> $1/installed-package-sizes.tmp
+   oe-pkgdata-util -p $pkgdatadir read-value "PKGSIZE" -n -f $pkgcache > 
$1/installed-package-sizes.tmp
cat $1/installed-package-sizes.tmp | awk '{print $2 "\tKiB\t" $1}' | 
sort -n -r > $1/installed-package-sizes.txt
rm $1/installed-package-sizes.tmp
 
# Produce package info: runtime_name, buildtime_name, recipe, version, 
size
-   oe-pkgdata-util -p ${PKGDATA_DIR} read-value "PACKAGE,PN,PV,PKGSIZE" -n 
-f $pkgcache > $1/installed-package-info.tmp
+   oe-pkgdata-util -p $pkgdatadir read-value "PACKAGE,PN,PV,PKGSIZE" -n -f 
$pkgcache > $1/installed-package-info.tmp
cat $1/installed-package-info.tmp | sort -n -r -k 5 > 
$1/installed-package-info.txt
rm $1/installed-package-info.tmp
 
@@ -534,7 +545,7 @@ buildhistory_get_sdk_installed() {
return
fi
 
-   buildhistory_get_installed ${BUILDHISTORY_DIR_SDK}/$1 sdk
+   buildhistory_get_installed ${BUILDHISTORY_DIR_SDK}/$1 sdk $1
 }
 
 buildhistory_get_sdk_installed_host() {
@@ -762,7 +773,7 @@ def buildhistory_get_imagevars(d):
 def buildhistory_get_sdkvars(d):
 if d.getVar('BB_WORKERCONTEXT') != '1':
 return ""
-sdkvars = "DISTRO DISTRO_VERSION SDK_NAME SDK_VERSION SDKMACHINE 
SDKIMAGE_FEATURES BAD_RECOMMENDATIONS NO_RECOMMENDATIONS PACKAGE_EXCLUDE"
+sdkvars = "DISTRO DISTRO_VERSION SDK_NAME SDK_VERSION SDKMACHINE 
SDKIMAGE_FEATURES TOOLCHAIN_HOST_TASK TOOLCHAIN_TARGET_TASK BAD_RECOMMENDATIONS 
NO_RECOMMENDATIONS PACKAGE_EXCLUDE"
 if d.getVar('BB_CURRENTTASK') == 'populate_sdk_ext':
 # Extensible SDK uses some additional variables
 sdkvars += " SDK_LOCAL_CONF_WHITELIST SDK_LOCAL_CONF_BLACKLIST 
SDK_INHERIT_BLACKLIST SDK_UPDATE_URL SDK_EXT_TYPE SDK_RECRDEP_TASKS 
SDK_INCLUDE_PKGDATA SDK_INCLUDE_TOOLCHAIN"
diff --git a/meta/classes/cross-canadian.bbclass 
b/meta/classes/cross-canadian.bbclass
index 1e54035084..1974b33943 100644
--- a/meta/classes/cross-canadian.bbclass
+++ b/meta/classes/cross-canadian.bbclass
@@ -167,7 +167,7 @@ USE_NLS = "${SDKUSE_NLS}"
 # and not any particular tune that is enabled.
 TARGET_ARCH[vardepsexclude] = "TUNE_ARCH"
 
-PKGDATA_DIR = "${TMPDIR}/pkgdata/${SDK_SYS}"
+PKGDATA_DIR = "${PKGDATA_DIR_SDK}"
 # If MLPREFIX is set by multilib code, shlibs
 # points to the wrong place so force it
 SHLIBSDIRS = "${PKGDATA_DIR}/nativesdk-shlibs2"
diff --git a/meta/classes/nativesdk.bbclass b/meta/classes/nativesdk.bbclass
index 7f2692c51a..d2a3f0bec9 100644
--- a/meta/classes/nativesdk.bbclass
+++ b/meta/classes/nativesdk.bbclass
@@ -31,7 +31,7 @@ PACKAGE_ARCHS = "${SDK_PACKAGE_ARCHS}"
 DEPENDS_append = " chrpath-replacement-native"
 EXTRANATIVEPATH += "chrpath-native"
 
-PKGDATA_DIR = "${TMPDIR}/pkgdata/${SDK_SYS}"
+PKGDATA_DIR = "${PKGDATA_DIR_SDK}"
 
 HOST_ARCH = "${SDK_ARCH}"
 HOST_VENDOR = "${SD

Re: [OE-core] [PATCH] sstate.bbclass: adds warnings for the exceptions of os.utime in siginfo

2021-10-04 Thread Richard Purdie

On Mon, 2021-10-04 at 18:59 +0100, Jose Quaresma wrote:
> I understand the exception for the use cases but I think it would be useful
> to show this to the users. Will it be more appropriate perhaps to log this as
> debug messages?

The trouble is in the situations that is guarding against, the user cannot do
anything about it so warnings definitely aren't appropriate and I'm not sure
debug messages would be welcome either. Those would be less invasive though.

> I have a final question that I don't understand clearly.
> Can the omission of these timestamps updates on siginfo invalidate the use of
> the sstate-cache for that task,
> bitbake-dumpsig can complain about that?

I think there is a misunderstanding here. The timestamps I've been talking about
are part of reproducible builds and the SOURCE_DATE_EPOCH variable and code.
These aim to make the output identical. If the output is identical, hash
equivalence is more successful and the more successful hash equivalence is, the
more sstate reuse you get.

The timestamps in this part of the sstate code are simply there to handle sstate
artefact aging (e.g. delete things which haven't been used in X weeks). The
timestamp change is therefore "nice" but if it doesn't happen, it isn't a
problem. Hope that helps clarify.

Cheers,

Richard

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

[OE-core] [hardknott][PATCH 06/11] vim: Backport fix for CVE-2021-3770

2021-10-04 Thread Anuj Mittal

From: Richard Purdie 

Signed-off-by: Richard Purdie 
(cherry picked from commit 54d3d023ce55ba4a7160ed25a283f0918e7d8e2e)
Signed-off-by: Anuj Mittal 
---
 ...1e135a16091c93f6f5f7525a5c58fb7ca9f9.patch | 207 ++
 meta/recipes-support/vim/vim.inc  |   2 +
 2 files changed, 209 insertions(+)
 create mode 100644 
meta/recipes-support/vim/files/b7081e135a16091c93f6f5f7525a5c58fb7ca9f9.patch

diff --git 
a/meta/recipes-support/vim/files/b7081e135a16091c93f6f5f7525a5c58fb7ca9f9.patch 
b/meta/recipes-support/vim/files/b7081e135a16091c93f6f5f7525a5c58fb7ca9f9.patch
new file mode 100644
index 00..1cee759502
--- /dev/null
+++ 
b/meta/recipes-support/vim/files/b7081e135a16091c93f6f5f7525a5c58fb7ca9f9.patch
@@ -0,0 +1,207 @@
+From b7081e135a16091c93f6f5f7525a5c58fb7ca9f9 Mon Sep 17 00:00:00 2001
+From: Bram Moolenaar 
+Date: Sat, 4 Sep 2021 18:47:28 +0200
+Subject: [PATCH] patch 8.2.3402: invalid memory access when using :retab with
+ large value
+
+Problem:Invalid memory access when using :retab with large value.
+Solution:   Check the number is positive.
+
+CVE: CVE-2021-3770
+Signed-off-by: Richard Purdie 
+Upstream-Status: Backport 
[https://github.com/vim/vim/commit/b7081e135a16091c93f6f5f7525a5c58fb7ca9f9]
+---
+ src/indent.c   | 34 +-
+ src/option.c   | 12 ++--
+ src/optionstr.c|  4 ++--
+ src/testdir/test_retab.vim |  3 +++
+ src/version.c  |  2 ++
+ 5 files changed, 34 insertions(+), 21 deletions(-)
+
+Index: git/src/indent.c
+===
+--- git.orig/src/indent.c
 git/src/indent.c
+@@ -18,18 +18,19 @@
+ /*
+  * Set the integer values corresponding to the string setting of 'vartabstop'.
+  * "array" will be set, caller must free it if needed.
++ * Return FAIL for an error.
+  */
+ int
+ tabstop_set(char_u *var, int **array)
+ {
+-int valcount = 1;
+-int t;
+-char_u *cp;
++int   valcount = 1;
++int   t;
++char_u  *cp;
+ 
+ if (var[0] == NUL || (var[0] == '0' && var[1] == NUL))
+ {
+   *array = NULL;
+-  return TRUE;
++  return OK;
+ }
+ 
+ for (cp = var; *cp != NUL; ++cp)
+@@ -43,8 +44,8 @@ tabstop_set(char_u *var, int **array)
+   if (cp != end)
+   emsg(_(e_positive));
+   else
+-  emsg(_(e_invarg));
+-  return FALSE;
++  semsg(_(e_invarg2), cp);
++  return FAIL;
+   }
+   }
+ 
+@@ -55,26 +56,33 @@ tabstop_set(char_u *var, int **array)
+   ++valcount;
+   continue;
+   }
+-  emsg(_(e_invarg));
+-  return FALSE;
++  semsg(_(e_invarg2), var);
++  return FAIL;
+ }
+ 
+ *array = ALLOC_MULT(int, valcount + 1);
+ if (*array == NULL)
+-  return FALSE;
++  return FAIL;
+ (*array)[0] = valcount;
+ 
+ t = 1;
+ for (cp = var; *cp != NUL;)
+ {
+-  (*array)[t++] = atoi((char *)cp);
+-  while (*cp  != NUL && *cp != ',')
++  int n = atoi((char *)cp);
++
++  if (n < 0 || n > )
++  {
++  semsg(_(e_invarg2), cp);
++  return FAIL;
++  }
++  (*array)[t++] = n;
++  while (*cp != NUL && *cp != ',')
+   ++cp;
+   if (*cp != NUL)
+   ++cp;
+ }
+ 
+-return TRUE;
++return OK;
+ }
+ 
+ /*
+@@ -1556,7 +1564,7 @@ ex_retab(exarg_T *eap)
+ 
+ #ifdef FEAT_VARTABS
+ new_ts_str = eap->arg;
+-if (!tabstop_set(eap->arg, &new_vts_array))
++if (tabstop_set(eap->arg, &new_vts_array) == FAIL)
+   return;
+ while (vim_isdigit(*(eap->arg)) || *(eap->arg) == ',')
+   ++(eap->arg);
+Index: git/src/option.c
+===
+--- git.orig/src/option.c
 git/src/option.c
+@@ -2292,9 +2292,9 @@ didset_options2(void)
+ #endif
+ #ifdef FEAT_VARTABS
+ vim_free(curbuf->b_p_vsts_array);
+-tabstop_set(curbuf->b_p_vsts, &curbuf->b_p_vsts_array);
++(void)tabstop_set(curbuf->b_p_vsts, &curbuf->b_p_vsts_array);
+ vim_free(curbuf->b_p_vts_array);
+-tabstop_set(curbuf->b_p_vts,  &curbuf->b_p_vts_array);
++(void)tabstop_set(curbuf->b_p_vts,  &curbuf->b_p_vts_array);
+ #endif
+ }
+ 
+@@ -5756,7 +5756,7 @@ buf_copy_options(buf_T *buf, int flags)
+   buf->b_p_vsts = vim_strsave(p_vsts);
+   COPY_OPT_SCTX(buf, BV_VSTS);
+   if (p_vsts && p_vsts != empty_option)
+-  tabstop_set(p_vsts, &buf->b_p_vsts_array);
++  (void)tabstop_set(p_vsts, &buf->b_p_vsts_array);
+   else
+   buf->b_p_vsts_array = 0;
+   buf->b_p_vsts_nopaste = p_vsts_nopaste
+@@ -5914,7 +5914,7 @@ buf_copy_options(buf_T *buf, int flags)
+   buf->b_p_isk = save_p_isk;
+ #ifdef FEAT_VARTABS
+   if (p_vts && p_vts != empty_option && !buf->b_p_vts_array)
+-  tabstop_set(p_vts, &buf->

Re: [OE-core] [PATCH 04/10] cmake: update 3.20.2 -> 3.20.3

2021-10-04 Thread Alexander Kanavin

Since the original centos 8 is going away (support-wise) in less than 3
months, should we just convert all workers to stream and move on?

Alex

On Mon, 4 Oct 2021 at 18:27, Michael Halstead 
wrote:

> This error was fixed upstream
> https://bugzilla.redhat.com/show_bug.cgi?id=1975562 kernel 4.18.0-325.x
> which has landed in CentOS8 Stream on centos8-ty-2 but not plain CentOS8
> where the newest kernel is still at 4.18.0-305.x.
>
> I'll reboot into the older 4.18.0-240.15 kernel again on centos8-ty-1 as a
> fix for now.
>
> On Tue, Sep 28, 2021 at 6:12 PM Anuj Mittal  wrote:
>
>> Did we find out the reason why this was happening? I have started
>> getting this error on Centos-8 while building hardknott.
>>
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/63/builds/4045
>>
>> Thanks,
>>
>> Anuj
>>
>> On Wed, 2021-06-16 at 23:45 +0100, Richard Purdie wrote:
>> > On Sun, 2021-06-06 at 21:51 +0200, Alexander Kanavin wrote:
>> > > On Sun, 6 Jun 2021 at 01:10, Richard Purdie
>> > >  wrote:
>> > > > I tried again with the autobuilder, still fails:
>> > > >
>> > > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/48/builds/3516
>> > > >
>> > > > so whatever it is, it is still "live".
>> > > >
>> > >
>> > >
>> > > I did some digging. The issue happens when:
>> > > - host is centos8
>> > > - SDKMACHINE is i686 (e.g. cmake is 32 bit)
>> > >
>> > > Then there's a failing syscall attempting to set file times:
>> > > utimensat_time64(AT_FDCWD, "../install/usr/local/lib/cmake/assimp-
>> > > 4.1/assimp-config.cmake",
>> > > [{tv_sec=1622966723, tv_nsec=6319439026193432576},
>> > > {tv_sec=1622966579, tv_nsec=17840053692309438464}], 0) = -1
>> > > EINVAL (Invalid argument)
>> > >
>> > > On latest Fedora, there's no issue:
>> > > utimensat_time64(AT_FDCWD, "../install2/usr/local/lib/cmake/assimp-
>> > > 4.1/assimp-config.cmake",
>> > > [{tv_sec=1623002886, tv_nsec=6369724778172907520},
>> > > {tv_sec=1623002886, tv_nsec=17839174083007217664}], 0) = 0
>> > >
>> > > utimensat_time64 only appeared with 5.1 kernels, however, 4.18 should
>> > > be returning ENOSYS in that case
>> > > probably?
>> >
>> > I hacked up a quick test bit of code (which makes assumptions
>> > about 32 bit):
>> >
>> > #include 
>> > #include 
>> > #include 
>> > #include 
>> > #include 
>> > #include 
>> >
>> > struct timespec64 {
>> > long long   tv_sec; /* seconds */
>> > long long   tv_nsec;/* nanoseconds */
>> > };
>> >
>> > int main() {
>> >   int fd = open("foo", O_RDWR | O_CREAT, 0644);
>> >   write(fd, "foo", 3);
>> >   struct timespec64 times[2] = {};
>> >   times[0].tv_sec = 1622966723;
>> >   times[0].tv_nsec = 631943;
>> >   times[1].tv_sec = 1622966579;
>> >   times[1].tv_nsec = 178400;
>> >   int rc = syscall(SYS_utimensat_time64, fd, NULL, ×[0], 0);
>> >   printf("rc=%d\n", rc);
>> >   close(fd);
>> >   return rc;
>> > }
>> >
>> > built with "gcc -m32 test-syscall.c -o test" and run with "strace
>> > ./test".
>> > This works on all the systems I tried it in. As does:
>> >
>> >
>> >   times[0].tv_sec = 1;
>> >   times[0].tv_nsec = 2;
>> >   times[1].tv_sec = 3;
>> >   times[1].tv_nsec = 4;
>> >
>> > however if you set (and ignore the compiler warning):
>> >
>> >   times[0].tv_sec = 1622966723;
>> >   times[0].tv_nsec = 6319439026193432576;
>> >   times[1].tv_sec = 1622966579;
>> >   times[1].tv_nsec = 17840053692309438464;
>> >
>> > then you see EINVAL on the centos system but not on my ubuntu one. It
>> > will
>> > do that until you reduce the values of tv_nsec right now. So it seems
>> > most
>> > systems accept large tv_nsec values but the Centos one does not.
>> >
>> > I think tv_nsec may be being clamped to LONG_MAX of 4 bytes but should
>> > be
>> > a LONG_LONG_MAX of 8 bytes on a 32 bit since the field is a 64 bit
>> > long.
>> >
>> > Michael: Hopefully that gives you something to raise with them?
>> >
>> > Cheers,
>> >
>> > Richard
>> >
>> >
>> >
>> >
>> >
>> >
>> >
>> >
>>
>>
>> 
>>
>>
>
> --
> Michael Halstead
> Linux Foundation / Yocto Project
> Systems Operations Engineer
>

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

[OE-core] [hardknott][PATCH 05/11] wic: keep rootfs_size as integer

2021-10-04 Thread Anuj Mittal

From: Alexander Kanavin 

The corrected line accidentally converted it to float,
which causes problems later on with python 3.10:

|   File "/home/alex/development/poky/scripts/lib/wic/partition.py", line 278, 
in prepare_rootfs_ext
| os.ftruncate(sparse.fileno(), rootfs_size * 1024)
| TypeError: 'float' object cannot be interpreted as an integer

Signed-off-by: Alexander Kanavin 
Signed-off-by: Richard Purdie 
(cherry picked from commit d1d260dd2d196d10379ed9e238bcb34f39f3a3b7)
Signed-off-by: Anuj Mittal 
---
 scripts/lib/wic/partition.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/lib/wic/partition.py b/scripts/lib/wic/partition.py
index 76d144d12d..a0b7291a6b 100644
--- a/scripts/lib/wic/partition.py
+++ b/scripts/lib/wic/partition.py
@@ -104,7 +104,7 @@ class Partition():
 extra_blocks = self.extra_space
 
 rootfs_size = actual_rootfs_size + extra_blocks
-rootfs_size *= self.overhead_factor
+rootfs_size = int(rootfs_size * self.overhead_factor)
 
 logger.debug("Added %d extra blocks to %s to get to %d total 
blocks",
  extra_blocks, self.mountpoint, rootfs_size)
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 02/11] qemu: fix CVE-2021-3682

2021-10-04 Thread Anuj Mittal

From: Sakib Sajal 

Signed-off-by: Sakib Sajal 
Signed-off-by: Richard Purdie 
(cherry picked from commit e16cd155c5ef7cfe8b4d3a94485cb7b13fd95036)
Signed-off-by: Anuj Mittal 
---
 meta/recipes-devtools/qemu/qemu.inc   |  1 +
 .../qemu/qemu/CVE-2021-3682.patch | 41 +++
 2 files changed, 42 insertions(+)
 create mode 100644 meta/recipes-devtools/qemu/qemu/CVE-2021-3682.patch

diff --git a/meta/recipes-devtools/qemu/qemu.inc 
b/meta/recipes-devtools/qemu/qemu.inc
index c3eecea9d4..463339e42b 100644
--- a/meta/recipes-devtools/qemu/qemu.inc
+++ b/meta/recipes-devtools/qemu/qemu.inc
@@ -69,6 +69,7 @@ SRC_URI = "https://download.qemu.org/${BPN}-${PV}.tar.xz \
file://CVE-2021-3582.patch \
file://CVE-2021-3607.patch \
file://CVE-2021-3608.patch \
+   file://CVE-2021-3682.patch \
"
 UPSTREAM_CHECK_REGEX = "qemu-(?P\d+(\.\d+)+)\.tar"
 
diff --git a/meta/recipes-devtools/qemu/qemu/CVE-2021-3682.patch 
b/meta/recipes-devtools/qemu/qemu/CVE-2021-3682.patch
new file mode 100644
index 00..50a49233d3
--- /dev/null
+++ b/meta/recipes-devtools/qemu/qemu/CVE-2021-3682.patch
@@ -0,0 +1,41 @@
+From 5e796671e6b8d5de4b0b423dce1b3eba144a92c9 Mon Sep 17 00:00:00 2001
+From: Gerd Hoffmann 
+Date: Thu, 22 Jul 2021 09:27:56 +0200
+Subject: [PATCH] usbredir: fix free call
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+data might point into the middle of a larger buffer, there is a separate
+free_on_destroy pointer passed into bufp_alloc() to handle that.  It is
+only used in the normal workflow though, not when dropping packets due
+to the queue being full.  Fix that.
+
+Resolves: https://gitlab.com/qemu-project/qemu/-/issues/491
+Signed-off-by: Gerd Hoffmann 
+Reviewed-by: Marc-André Lureau 
+Message-Id: <20210722072756.647673-1-kra...@redhat.com>
+
+CVE: CVE-2021-3682
+Upstream-Status: Backport [5e796671e6b8d5de4b0b423dce1b3eba144a92c9]
+Signed-off-by: Sakib Sajal 
+---
+ hw/usb/redirect.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c
+index 4ec9326e05..1ec909a63a 100644
+--- a/hw/usb/redirect.c
 b/hw/usb/redirect.c
+@@ -476,7 +476,7 @@ static int bufp_alloc(USBRedirDevice *dev, uint8_t *data, 
uint16_t len,
+ if (dev->endpoint[EP2I(ep)].bufpq_dropping_packets) {
+ if (dev->endpoint[EP2I(ep)].bufpq_size >
+ dev->endpoint[EP2I(ep)].bufpq_target_size) {
+-free(data);
++free(free_on_destroy);
+ return -1;
+ }
+ dev->endpoint[EP2I(ep)].bufpq_dropping_packets = 0;
+-- 
+2.25.1
+
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 04/11] Update mailing list address

2021-10-04 Thread Anuj Mittal

From: Jon Mason 

Signed-off-by: Jon Mason 
Signed-off-by: Richard Purdie 
(cherry picked from commit 83169c33f7585da25560784f79eaad2c6f029f3c)
Signed-off-by: Anuj Mittal 
---
 meta/conf/distro/include/maintainers.inc  | 2 +-
 meta/recipes-core/glibc/ldconfig-native-2.12.1/ldconfig.patch | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/conf/distro/include/maintainers.inc 
b/meta/conf/distro/include/maintainers.inc
index 6276331857..5d453a6fcd 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -4,7 +4,7 @@
 #
 # Please submit any patches against recipes in meta to the
 # OE-Core mail list (openembedded-core@lists.openembedded.org)
-# For recipes in meta-yocto please use the Poky list (p...@yoctoproject.org)
+# For recipes in meta-yocto please use the Poky list 
(p...@lists.yoctoproject.org)
 #
 # If you have problems with or questions about a particular recipe, feel
 # free to contact the maintainer directly (cc:ing the appropriate mailing list
diff --git a/meta/recipes-core/glibc/ldconfig-native-2.12.1/ldconfig.patch 
b/meta/recipes-core/glibc/ldconfig-native-2.12.1/ldconfig.patch
index 52986e61c7..d1835c7a10 100644
--- a/meta/recipes-core/glibc/ldconfig-native-2.12.1/ldconfig.patch
+++ b/meta/recipes-core/glibc/ldconfig-native-2.12.1/ldconfig.patch
@@ -400,7 +400,7 @@ Index: ldconfig-native-2.12.1/ldconfig.c
return 0;
  }
  
-+#define REPORT_BUGS_TO "mailing list : p...@yoctoproject.org"
++#define REPORT_BUGS_TO "mailing list : p...@lists.yoctoproject.org"
  /* Print bug-reporting information in the help message.  */
  static char *
  more_help (int key, const char *text, void *input)
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 03/11] bind: Exclude CVE-2019-6470 from cve-check

2021-10-04 Thread Anuj Mittal

From: Richard Purdie 

Issue only affects dhcpd with recent bind versions. We don't ship dhcpd anymore
so the issue doesn't affect us.

Signed-off-by: Richard Purdie 
(cherry picked from commit 30106ae676124ba3c0e496a4f19c919c8418b59b)
Signed-off-by: Anuj Mittal 
---
 meta/recipes-connectivity/bind/bind_9.16.16.bb | 4 
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-connectivity/bind/bind_9.16.16.bb 
b/meta/recipes-connectivity/bind/bind_9.16.16.bb
index 27aa6221ba..b152598402 100644
--- a/meta/recipes-connectivity/bind/bind_9.16.16.bb
+++ b/meta/recipes-connectivity/bind/bind_9.16.16.bb
@@ -26,6 +26,10 @@ UPSTREAM_CHECK_URI = "https://ftp.isc.org/isc/bind9/";
 # stay at 9.16 follow the ESV versions divisible by 4
 UPSTREAM_CHECK_REGEX = "(?P9.(16|20|24|28)(\.\d+)+(-P\d+)*)/"
 
+# Issue only affects dhcpd with recent bind versions. We don't ship dhcpd 
anymore
+# so the issue doesn't affect us.
+CVE_CHECK_WHITELIST += "CVE-2019-6470"
+
 inherit autotools update-rc.d systemd useradd pkgconfig multilib_header 
update-alternatives
 
 # PACKAGECONFIGs readline and libedit should NOT be set at same time
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 01/11] systemd: fix CVE-2021-33910

2021-10-04 Thread Anuj Mittal

From: Chen Qi 

Backport patch to fix CVE-2021-33910.

Signed-off-by: Chen Qi 
Signed-off-by: Anuj Mittal 
---
 ...it-name-do-not-use-strdupa-on-a-path.patch | 72 +++
 meta/recipes-core/systemd/systemd_247.6.bb|  1 +
 2 files changed, 73 insertions(+)
 create mode 100644 
meta/recipes-core/systemd/systemd/0001-basic-unit-name-do-not-use-strdupa-on-a-path.patch

diff --git 
a/meta/recipes-core/systemd/systemd/0001-basic-unit-name-do-not-use-strdupa-on-a-path.patch
 
b/meta/recipes-core/systemd/systemd/0001-basic-unit-name-do-not-use-strdupa-on-a-path.patch
new file mode 100644
index 00..0ab8174441
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/0001-basic-unit-name-do-not-use-strdupa-on-a-path.patch
@@ -0,0 +1,72 @@
+From b00674347337b7531c92fdb65590ab253bb57538 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= 
+Date: Wed, 23 Jun 2021 11:46:41 +0200
+Subject: [PATCH] basic/unit-name: do not use strdupa() on a path
+
+The path may have unbounded length, for example through a fuse mount.
+
+CVE-2021-33910: attacked controlled alloca() leads to crash in systemd and
+ultimately a kernel panic. Systemd parses the content of /proc/self/mountinfo
+and each mountpoint is passed to mount_setup_unit(), which calls
+unit_name_path_escape() underneath. A local attacker who is able to mount a
+filesystem with a very long path can crash systemd and the whole system.
+
+https://bugzilla.redhat.com/show_bug.cgi?id=1970887
+
+The resulting string length is bounded by UNIT_NAME_MAX, which is 256. But we
+can't easily check the length after simplification before doing the
+simplification, which in turns uses a copy of the string we can write to.
+So we can't reject paths that are too long before doing the duplication.
+Hence the most obvious solution is to switch back to strdup(), as before
+7410616cd9dbbec97cf98d75324da5cda2b2f7a2.
+
+(cherry picked from commit 441e0115646d54f080e5c3bb0ba477c892861ab9)
+(cherry picked from commit 764b74113e36ac5219a4b82a05f311b5a92136ce)
+(cherry picked from commit 4a1c5f34bd3e1daed4490e9d97918e504d19733b)
+
+CVE: CVE-2021-33910
+Upstream-Status: Backport [b00674347337b7531c92fdb65590ab253bb57538]
+Signed-off-by: Chen Qi 
+---
+ src/basic/unit-name.c | 13 +
+ 1 file changed, 5 insertions(+), 8 deletions(-)
+
+diff --git a/src/basic/unit-name.c b/src/basic/unit-name.c
+index 5f595af944..9b6cacde87 100644
+--- a/src/basic/unit-name.c
 b/src/basic/unit-name.c
+@@ -378,12 +378,13 @@ int unit_name_unescape(const char *f, char **ret) {
+ }
+ 
+ int unit_name_path_escape(const char *f, char **ret) {
+-char *p, *s;
++_cleanup_free_ char *p = NULL;
++char *s;
+ 
+ assert(f);
+ assert(ret);
+ 
+-p = strdupa(f);
++p = strdup(f);
+ if (!p)
+ return -ENOMEM;
+ 
+@@ -395,13 +396,9 @@ int unit_name_path_escape(const char *f, char **ret) {
+ if (!path_is_normalized(p))
+ return -EINVAL;
+ 
+-/* Truncate trailing slashes */
++/* Truncate trailing slashes and skip leading slashes */
+ delete_trailing_chars(p, "/");
+-
+-/* Truncate leading slashes */
+-p = skip_leading_chars(p, "/");
+-
+-s = unit_name_escape(p);
++s = unit_name_escape(skip_leading_chars(p, "/"));
+ }
+ if (!s)
+ return -ENOMEM;
+-- 
+2.33.0
+
diff --git a/meta/recipes-core/systemd/systemd_247.6.bb 
b/meta/recipes-core/systemd/systemd_247.6.bb
index f1db1e922b..e79c79a7fd 100644
--- a/meta/recipes-core/systemd/systemd_247.6.bb
+++ b/meta/recipes-core/systemd/systemd_247.6.bb
@@ -31,6 +31,7 @@ SRC_URI += "file://touchscreen.rules \
file://0002-sd-dhcp-client-shorten-code-a-bit.patch \

file://0003-sd-dhcp-client-logs-when-dhcp-client-unexpectedly-ga.patch \

file://0004-sd-dhcp-client-tentatively-ignore-FORCERENEW-command.patch \
+   file://0001-basic-unit-name-do-not-use-strdupa-on-a-path.patch \
"
 
 # patches needed by musl
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 00/11] Review request

2021-10-04 Thread Anuj Mittal

Please review these changes for hardknott. No issues seen on
autobuilder.

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

Thanks,

Anuj

The following changes since commit 97d6a8452779fe511a354a70a72dd338f52a92cb:

  bash: Ensure deterministic build (2021-09-29 16:22:10 +0800)

are available in the Git repository at:

  git://push.openembedded.org/openembedded-core-contrib anujm/hardknott

Alexander Kanavin (1):
  wic: keep rootfs_size as integer

Anibal Limon (1):
  recipes-support/ptest-runner: Bump to v2.4.2

Chandana kalluri (1):
  scriptutils.py: Add check before deleting path

Chen Qi (1):
  systemd: fix CVE-2021-33910

Jon Mason (1):
  Update mailing list address

Kenfe-Mickael Laventure (1):
  package_ipk: Use localdata store when signing packages

Richard Purdie (3):
  bind: Exclude CVE-2019-6470 from cve-check
  vim: Backport fix for CVE-2021-3770
  glew: Stop polluting /tmp during builds

Sakib Sajal (1):
  qemu: fix CVE-2021-3682

William A. Kennington III (1):
  rm_work.bbclass: Fix for files starting with -

 meta/classes/package_ipk.bbclass  |   4 +-
 meta/classes/rm_work.bbclass  |   8 +-
 meta/conf/distro/include/maintainers.inc  |   2 +-
 .../recipes-connectivity/bind/bind_9.16.16.bb |   4 +
 .../ldconfig-native-2.12.1/ldconfig.patch |   2 +-
 ...it-name-do-not-use-strdupa-on-a-path.patch |  72 ++
 meta/recipes-core/systemd/systemd_247.6.bb|   1 +
 meta/recipes-devtools/qemu/qemu.inc   |   1 +
 .../qemu/qemu/CVE-2021-3682.patch |  41 
 .../glew/glew/notempdir.patch |  19 ++
 meta/recipes-graphics/glew/glew_2.2.0.bb  |   1 +
 ...-runner_2.4.1.bb => ptest-runner_2.4.2.bb} |   2 +-
 ...1e135a16091c93f6f5f7525a5c58fb7ca9f9.patch | 207 ++
 meta/recipes-support/vim/vim.inc  |   2 +
 scripts/lib/scriptutils.py|   3 +-
 scripts/lib/wic/partition.py  |   2 +-
 16 files changed, 360 insertions(+), 11 deletions(-)
 create mode 100644 
meta/recipes-core/systemd/systemd/0001-basic-unit-name-do-not-use-strdupa-on-a-path.patch
 create mode 100644 meta/recipes-devtools/qemu/qemu/CVE-2021-3682.patch
 create mode 100644 meta/recipes-graphics/glew/glew/notempdir.patch
 rename meta/recipes-support/ptest-runner/{ptest-runner_2.4.1.bb => 
ptest-runner_2.4.2.bb} (93%)
 create mode 100644 
meta/recipes-support/vim/files/b7081e135a16091c93f6f5f7525a5c58fb7ca9f9.patch

-- 
2.31.1


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

[OE-core] just want to confirm my understanding of how systemd services get symlinks

2021-10-04 Thread Robert P. J. Day


  just had a discussion with a colleague about the proper design of a
systemd-based service, and my (admittedly sophomore) understanding is
that, in the service file, there is an [Install] section, which
contains a "WantedBy" line like:

  fubar.service:WantedBy=sysinit.target

upon installation in the image, from systemd.bbclass. the routine
systemd_postinst() is invoked which (among other things), does this:

if [ "${SYSTEMD_AUTO_ENABLE}" = "enable" ]; then
for service in ${SYSTEMD_SERVICE_ESCAPED}; do
systemctl ${OPTS} enable "$service"
done
fi

and my understanding is that it is that call to "systemctl" that
creates the appropriate symlink so that that service now exists under
the directory "sysinit.target.wants".

  my need for confirmation is based on muh collegaue's pointing at
some existing systemd-based recipes in some legacy code, which
manually do the following in do_install_append():

  ln -s ${sysconfdir}/systemd/system/fubar.service \
${D}/${sysconfdir}/systemd/system/sysinit.target.wants/

i'm pretty sure that that "ln" command is unnecessary as long as the
service file is defined properly, but now i just want to get that
confirmed.

rday

p.s. hmm ... i wonder if the original recipe designer was unaware
of systemd.bbclass and thought he had to do it all manually.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156621): 
https://lists.openembedded.org/g/openembedded-core/message/156621
Mute This Topic: https://lists.openembedded.org/mt/86073990/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] sstate.bbclass: adds warnings for the exceptions of os.utime in siginfo

2021-10-04 Thread Jose Quaresma

Richard Purdie  escreveu no dia
segunda, 4/10/2021 à(s) 15:11:

> On Sun, 2021-10-03 at 22:38 +0100, Jose Quaresma wrote:
> > When we can't update the access and modified times of sstate siginfo
> > it is useful to infor the user about that.
> >
> > Signed-off-by: Jose Quaresma 
> > ---
> >  meta/classes/sstate.bbclass | 12 ++--
> >  1 file changed, 10 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
> > index 92a73114bb..2deecd8777 100644
> > --- a/meta/classes/sstate.bbclass
> > +++ b/meta/classes/sstate.bbclass
> > @@ -720,11 +720,15 @@ def sstate_package(ss, d):
> >  try:
> >  os.utime(siginfo, None)
> >  except PermissionError:
> > +bb.warn("Cannot sets the access and modified times of
> sstate siginfo %s, no access rights" % siginfo)
> >  pass
> >  except OSError as e:
> >  # Handle read-only file systems gracefully
> >  import errno
> > -if e.errno != errno.EROFS:
> > +if e.errno == errno.EROFS:
> > +bb.warn("Cannot sets the access and modified times of
> sstate siginfo %s, read-only file system" % siginfo)
> > +pass
> > +else:
> >  raise e
> >
> >  return
> > @@ -1165,11 +1169,15 @@ python sstate_eventhandler() {
> >  try:
> >  os.utime(siginfo, None)
> >  except PermissionError:
> > +bb.warn("Cannot sets the access and modified times of
> sstate siginfo %s, no access rights" % siginfo)
> >  pass
> >  except OSError as e:
> >  # Handle read-only file systems gracefully
> >  import errno
> > -if e.errno != errno.EROFS:
> > +if e.errno == errno.EROFS:
> > +bb.warn("Cannot sets the access and modified times
> of sstate siginfo %s, read-only file system" % siginfo)
> > +pass
> > +else:
> >  raise e
> >
> >
>
> I'm not sure we can do this as there are some cases where this is
> unavoidable by
> people's state setups and we wouldn't want to show warnings in those cases.
>
> (e.g. think of a shared sstate dir between multiple users where the objects
> remain owned by the user that created them).
>
>
> Cheers,
>
> Richard
>
>
>
Hi Richard,

I understand the exception for the use cases but I think it would be useful
to show this to the users. Will it be more appropriate perhaps to log this
as debug messages?

I have a final question that I don't understand clearly.
Can the omission of these timestamps updates on siginfo invalidate the use
of the sstate-cache for that task,
bitbake-dumpsig can complain about that?

-- 
Best regards,

José Quaresma

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

[OE-core] [hardknott][PATCH 11/11] scriptutils.py: Add check before deleting path

2021-10-04 Thread Anuj Mittal

From: Chandana kalluri 

Add a check before deleting path when using recipetool commands to avoid the 
following type of errors:

Traceback (most recent call last):
   File "/sources/core/scripts/lib/scriptutils.py", line 218, in 
fetch_url
shutil.rmtree(path)
  File "/usr/local/lib/python3.7/shutil.py", line 476, in rmtree
onerror(os.lstat, path, sys.exc_info())
  File "/usr/local/lib/python3.7/shutil.py", line 474, in rmtree
orig_st = os.lstat(path)
FileNotFoundError: [Errno 2] No such file or directory: 
'/build/tmp/work/recipetool-usg7o81n/work/recipe-sysroot'
ERROR: Command 'script -e -q -c "recipetool --color=always create --devtool -o 
/tmp/devtool5sq_op37 'file:///'  -x 
/build/workspace/sources/devtoolsrcxc1b9zjq -N test" /dev/null' failed

Signed-off-by: Sai Hari Chandana Kalluri 
Signed-off-by: Richard Purdie 
(cherry picked from commit b6aa8b47e023004ffd6958d1cec18c2d9c95d77b)
Signed-off-by: Anuj Mittal 
---
 scripts/lib/scriptutils.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/lib/scriptutils.py b/scripts/lib/scriptutils.py
index f92255d8dc..3164171eb2 100644
--- a/scripts/lib/scriptutils.py
+++ b/scripts/lib/scriptutils.py
@@ -215,7 +215,8 @@ def fetch_url(tinfoil, srcuri, srcrev, destdir, logger, 
preserve_tmp=False, mirr
 pathvars = ['T', 'RECIPE_SYSROOT', 'RECIPE_SYSROOT_NATIVE']
 for pathvar in pathvars:
 path = rd.getVar(pathvar)
-shutil.rmtree(path)
+if os.path.exists(path):
+shutil.rmtree(path)
 finally:
 if fetchrecipe:
 try:
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 10/11] recipes-support/ptest-runner: Bump to v2.4.2

2021-10-04 Thread Anuj Mittal

From: Anibal Limon 

Changes,

bcb8280 utils.c: add system data collection when a test gets stuck.
c29240c utils.c: handle test timeouts directly with poll()
d6f509f tests/utils.c: fix a memory corruption in find_word
c10e747 main: Do not return number of failed tests when calling ptest-runner
4958988 utils.c: fix memory leak in run_ptests()
fcfa6a1 clang: clean-ups to avoid -Weverything warnings.
215e52d Makefile: allow using CC env var to pick compiler
fdd233d mem: Simplify memory management
e5e218a mem: Fix memleak for ptest_opts
0dc42eb git: Extend the gitignore

Signed-off-by: Aníbal Limón 
Signed-off-by: Richard Purdie 
(cherry picked from commit 710fa373375beb977af704e17a925ed41c9a858d)
Signed-off-by: Anuj Mittal 
---
 .../{ptest-runner_2.4.1.bb => ptest-runner_2.4.2.bb}| 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-support/ptest-runner/{ptest-runner_2.4.1.bb => 
ptest-runner_2.4.2.bb} (93%)

diff --git a/meta/recipes-support/ptest-runner/ptest-runner_2.4.1.bb 
b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
similarity index 93%
rename from meta/recipes-support/ptest-runner/ptest-runner_2.4.1.bb
rename to meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
index 6bd10d2fec..b14b947f97 100644
--- a/meta/recipes-support/ptest-runner/ptest-runner_2.4.1.bb
+++ b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
@@ -7,7 +7,7 @@ HOMEPAGE = 
"http://git.yoctoproject.org/cgit/cgit.cgi/ptest-runner2/about/";
 LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
 
-SRCREV = "cce0edb4282ee081d043030bfdf29f3e4052f86c"
+SRCREV = "bcb82804daa8f725b6add259dcef2067e61a75aa"
 PV .= "+git${SRCPV}"
 
 SRC_URI = "git://git.yoctoproject.org/ptest-runner2 \
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 09/11] rm_work.bbclass: Fix for files starting with -

2021-10-04 Thread Anuj Mittal

From: "William A. Kennington III" 

This makes it possible to name files starting with a hyphen in the work
directory. Without this change rm will fail due to an unexpected option
being passed.

Signed-off-by: William A. Kennington III 
Signed-off-by: Richard Purdie 
(cherry picked from commit 5f1a63e0de4921ef970114a16d0827fcddcdaa0e)
Signed-off-by: Anuj Mittal 
---
 meta/classes/rm_work.bbclass | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/classes/rm_work.bbclass b/meta/classes/rm_work.bbclass
index 01c2ab1c78..2d5a56c238 100644
--- a/meta/classes/rm_work.bbclass
+++ b/meta/classes/rm_work.bbclass
@@ -73,7 +73,7 @@ do_rm_work () {
 # sstate version since otherwise we'd need to leave 'plaindirs' 
around
 # such as 'packages' and 'packages-split' and these can be large. 
No end
 # of chain tasks depend directly on do_package anymore.
-rm -f $i;
+rm -f -- $i;
 ;;
 *_setscene*)
 # Skip stamps which are already setscene versions
@@ -90,7 +90,7 @@ do_rm_work () {
 ;;
 esac
 done
-rm -f $i
+rm -f -- $i
 esac
 done
 
@@ -100,9 +100,9 @@ do_rm_work () {
 # Retain only logs and other files in temp, safely ignore
 # failures of removing pseudo folers on NFS2/3 server.
 if [ $dir = 'pseudo' ]; then
-rm -rf $dir 2> /dev/null || true
+rm -rf -- $dir 2> /dev/null || true
 elif ! echo "$excludes" | grep -q -w "$dir"; then
-rm -rf $dir
+rm -rf -- $dir
 fi
 done
 }
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 07/11] glew: Stop polluting /tmp during builds

2021-10-04 Thread Anuj Mittal

From: Richard Purdie 

Currently the glew code creates a new directory in /tmp for each make
invocation. This is a bit ugly, don't do that. The patch does break the
dist targets but we don't use them.

Signed-off-by: Richard Purdie 
(cherry picked from commit 3a55194f90e11da5671b24391a4aaf2b86a8e1e6)
Signed-off-by: Anuj Mittal 
---
 .../glew/glew/notempdir.patch | 19 +++
 meta/recipes-graphics/glew/glew_2.2.0.bb  |  1 +
 2 files changed, 20 insertions(+)
 create mode 100644 meta/recipes-graphics/glew/glew/notempdir.patch

diff --git a/meta/recipes-graphics/glew/glew/notempdir.patch 
b/meta/recipes-graphics/glew/glew/notempdir.patch
new file mode 100644
index 00..8d79ce0cdf
--- /dev/null
+++ b/meta/recipes-graphics/glew/glew/notempdir.patch
@@ -0,0 +1,19 @@
+We don't use the dist-* targets and hence DIST_DIR isn't used. The current code
+creates a new temp directory in /tmp/ for every invocation of make. Lets
+not do that.
+
+Upstream-Status: Pending [a revised version would be needed for upstream]
+Signed-off-by: Richard Purdie 
+
+Index: glew-2.2.0/Makefile
+===
+--- glew-2.2.0.orig/Makefile
 glew-2.2.0/Makefile
+@@ -56,7 +56,6 @@ DIST_SRC_ZIP ?= $(shell pwd)/$(DIST_NAME
+ DIST_SRC_TGZ ?= $(shell pwd)/$(DIST_NAME).tgz
+ DIST_WIN32   ?= $(shell pwd)/$(DIST_NAME)-win32.zip
+ 
+-DIST_DIR := $(shell mktemp -d /tmp/glew.XX)/$(DIST_NAME)
+ 
+ # To disable stripping of linked binaries either:
+ #   - use STRIP= on gmake command-line
diff --git a/meta/recipes-graphics/glew/glew_2.2.0.bb 
b/meta/recipes-graphics/glew/glew_2.2.0.bb
index 92b6083648..d7a26a3438 100644
--- a/meta/recipes-graphics/glew/glew_2.2.0.bb
+++ b/meta/recipes-graphics/glew/glew_2.2.0.bb
@@ -7,6 +7,7 @@ LIC_FILES_CHKSUM = 
"file://LICENSE.txt;md5=2ac251558de685c6b9478d89be3149c2"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/project/glew/glew/${PV}/glew-${PV}.tgz \
file://0001-Fix-build-race-in-Makefile.patch \
+   file://notempdir.patch \
file://no-strip.patch"
 
 SRC_URI[md5sum] = "3579164bccaef09e36c0af7f4fd5c7c7"
-- 
2.31.1


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

[OE-core] [hardknott][PATCH 08/11] package_ipk: Use localdata store when signing packages

2021-10-04 Thread Anuj Mittal

From: Kenfe-Mickael Laventure 

When signing a package, we were resolving the `PKGV` version using the
original data store (`d`). However, since that store does not have the
package name in its `OVERRIDES` list the wrong version can be returned in
recipes that produce multiple packages. One such example would be
`external-arm-toolchain.bb`.

The above issue is fixed by using `localdata` instead of `d` when
resolving the needed variables.

Signed-off-by: Kenfe-Mickael Laventure 
Signed-off-by: Richard Purdie 
(cherry picked from commit 2b5f01cf5a92a873ac4c3f0ba0584cab2cc05714)
Signed-off-by: Anuj Mittal 
---
 meta/classes/package_ipk.bbclass | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/package_ipk.bbclass b/meta/classes/package_ipk.bbclass
index 600b3ac90c..67d6007e11 100644
--- a/meta/classes/package_ipk.bbclass
+++ b/meta/classes/package_ipk.bbclass
@@ -230,8 +230,8 @@ def ipk_write_pkg(pkg, d):
 shell=True)
 
 if d.getVar('IPK_SIGN_PACKAGES') == '1':
-ipkver = "%s-%s" % (d.getVar('PKGV'), d.getVar('PKGR'))
-ipk_to_sign = "%s/%s_%s_%s.ipk" % (pkgoutdir, pkgname, ipkver, 
d.getVar('PACKAGE_ARCH'))
+ipkver = "%s-%s" % (localdata.getVar('PKGV'), 
localdata.getVar('PKGR'))
+ipk_to_sign = "%s/%s_%s_%s.ipk" % (pkgoutdir, pkgname, ipkver, 
localdata.getVar('PACKAGE_ARCH'))
 sign_ipk(d, ipk_to_sign)
 
 finally:
-- 
2.31.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156616): 
https://lists.openembedded.org/g/openembedded-core/message/156616
Mute This Topic: https://lists.openembedded.org/mt/86072035/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 04/10] cmake: update 3.20.2 -> 3.20.3

2021-10-04 Thread Michael Halstead

This error was fixed upstream
https://bugzilla.redhat.com/show_bug.cgi?id=1975562 kernel 4.18.0-325.x
which has landed in CentOS8 Stream on centos8-ty-2 but not plain CentOS8
where the newest kernel is still at 4.18.0-305.x.

I'll reboot into the older 4.18.0-240.15 kernel again on centos8-ty-1 as a
fix for now.

On Tue, Sep 28, 2021 at 6:12 PM Anuj Mittal  wrote:

> Did we find out the reason why this was happening? I have started
> getting this error on Centos-8 while building hardknott.
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/63/builds/4045
>
> Thanks,
>
> Anuj
>
> On Wed, 2021-06-16 at 23:45 +0100, Richard Purdie wrote:
> > On Sun, 2021-06-06 at 21:51 +0200, Alexander Kanavin wrote:
> > > On Sun, 6 Jun 2021 at 01:10, Richard Purdie
> > >  wrote:
> > > > I tried again with the autobuilder, still fails:
> > > >
> > > >
> https://autobuilder.yoctoproject.org/typhoon/#/builders/48/builds/3516
> > > >
> > > > so whatever it is, it is still "live".
> > > >
> > >
> > >
> > > I did some digging. The issue happens when:
> > > - host is centos8
> > > - SDKMACHINE is i686 (e.g. cmake is 32 bit)
> > >
> > > Then there's a failing syscall attempting to set file times:
> > > utimensat_time64(AT_FDCWD, "../install/usr/local/lib/cmake/assimp-
> > > 4.1/assimp-config.cmake",
> > > [{tv_sec=1622966723, tv_nsec=6319439026193432576},
> > > {tv_sec=1622966579, tv_nsec=17840053692309438464}], 0) = -1
> > > EINVAL (Invalid argument)
> > >
> > > On latest Fedora, there's no issue:
> > > utimensat_time64(AT_FDCWD, "../install2/usr/local/lib/cmake/assimp-
> > > 4.1/assimp-config.cmake",
> > > [{tv_sec=1623002886, tv_nsec=6369724778172907520},
> > > {tv_sec=1623002886, tv_nsec=17839174083007217664}], 0) = 0
> > >
> > > utimensat_time64 only appeared with 5.1 kernels, however, 4.18 should
> > > be returning ENOSYS in that case
> > > probably?
> >
> > I hacked up a quick test bit of code (which makes assumptions
> > about 32 bit):
> >
> > #include 
> > #include 
> > #include 
> > #include 
> > #include 
> > #include 
> >
> > struct timespec64 {
> > long long   tv_sec; /* seconds */
> > long long   tv_nsec;/* nanoseconds */
> > };
> >
> > int main() {
> >   int fd = open("foo", O_RDWR | O_CREAT, 0644);
> >   write(fd, "foo", 3);
> >   struct timespec64 times[2] = {};
> >   times[0].tv_sec = 1622966723;
> >   times[0].tv_nsec = 631943;
> >   times[1].tv_sec = 1622966579;
> >   times[1].tv_nsec = 178400;
> >   int rc = syscall(SYS_utimensat_time64, fd, NULL, ×[0], 0);
> >   printf("rc=%d\n", rc);
> >   close(fd);
> >   return rc;
> > }
> >
> > built with "gcc -m32 test-syscall.c -o test" and run with "strace
> > ./test".
> > This works on all the systems I tried it in. As does:
> >
> >
> >   times[0].tv_sec = 1;
> >   times[0].tv_nsec = 2;
> >   times[1].tv_sec = 3;
> >   times[1].tv_nsec = 4;
> >
> > however if you set (and ignore the compiler warning):
> >
> >   times[0].tv_sec = 1622966723;
> >   times[0].tv_nsec = 6319439026193432576;
> >   times[1].tv_sec = 1622966579;
> >   times[1].tv_nsec = 17840053692309438464;
> >
> > then you see EINVAL on the centos system but not on my ubuntu one. It
> > will
> > do that until you reduce the values of tv_nsec right now. So it seems
> > most
> > systems accept large tv_nsec values but the Centos one does not.
> >
> > I think tv_nsec may be being clamped to LONG_MAX of 4 bytes but should
> > be
> > a LONG_LONG_MAX of 8 bytes on a 32 bit since the field is a 64 bit
> > long.
> >
> > Michael: Hopefully that gives you something to raise with them?
> >
> > Cheers,
> >
> > Richard
> >
> >
> >
> >
> >
> >
> >
> >
>
>
> 
>
>

-- 
Michael Halstead
Linux Foundation / Yocto Project
Systems Operations Engineer

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

Re: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS

2021-10-04 Thread Konrad Weihmann




On 04.10.21 17:29, Martin Jansa wrote:
 > Still this is very prone to errors, so I personally rather have a 
space too much than to few (esp. as the else clause doesn't add a space).


The missing space in else clause is often intentional in cases like 
this, so that the append has no impact on value when not enabled 
with GTKDOC_ENABLED. Why should it add useless space when not enabled? 
Just for the possibility of someone adding something to else clause 
without leading space there?


Partly yes, I found those conditionals rather hard to review, esp. for 
more complex scenarios, so I kind of would suggest to add the mandatory 
space for the append operation all the time and add no additional space 
in the stuff that is popping out of the conditional computation.


But I'm also fine with dropping the last 2 patches of the series.
Only the first patch solves a real world issue... the remaining two were 
just a result of a linter scan




On Mon, Oct 4, 2021 at 5:14 PM Konrad Weihmann > wrote:




On 04.10.21 17:08, Peter Kjellerstedt wrote:
 >> -Original Message-
 >> From: openembedded-core@lists.openembedded.org
 > c...@lists.openembedded.org
> On Behalf Of Konrad Weihmann
 >> Sent: den 4 oktober 2021 15:29
 >> To: openembedded-core@lists.openembedded.org

 >> Cc: Konrad Weihmann mailto:kweihm...@outlook.com>>
 >> Subject: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS
 >>
 >> fix the append with a leading blank, otherwise this messes up
 >> DEPENDS when globally inherited classes add dependecies of their own
 >>
 >> Signed-off-by: Konrad Weihmann mailto:kweihm...@outlook.com>>
 >> ---
 >>   meta/recipes-core/glib-2.0/glib.inc | 2 +-
 >>   1 file changed, 1 insertion(+), 1 deletion(-)
 >>
 >> diff --git a/meta/recipes-core/glib-2.0/glib.inc
b/meta/recipes-core/glib-
 >> 2.0/glib.inc
 >> index a34776b083..3436645f15 100644
 >> --- a/meta/recipes-core/glib-2.0/glib.inc
 >> +++ b/meta/recipes-core/glib-2.0/glib.inc
 >> @@ -30,7 +30,7 @@ LEAD_SONAME = "libglib-2.0.*"
 >>
 >>   inherit meson gettext gtk-doc pkgconfig ptest-gnome
upstream-version-is-
 >> even bash-completion gio-module-cache manpages
 >>
 >> -DEPENDS:append:class-target = "${@' gtk-doc' if
d.getVar('GTKDOC_ENABLED') == 'True' else ''}"
 >> +DEPENDS:append:class-target = " ${@' gtk-doc' if
d.getVar('GTKDOC_ENABLED') == 'True' else ''}"
 >
 > This change is not needed as there already is a space inside
 > the if statement (in ' gtk-doc').

Still this is very prone to errors, so I personally rather have a space
too much than to few (esp. as the else clause doesn't add a space).

IHMO it should be for readability purpose rather " ${@'gtk-doc' if
d.getVar('GTKDOC_ENABLED') == 'True' else ''}" than like it is now -
but
that's just cosmetics and I'm sure one could discuss about that a lot

 >
 >>
 >>   GTKDOC_MESON_OPTION = "gtk_doc"
 >>
 >> --
 >> 2.25.1
 >
 > //Peter
 >




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

Re: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS

2021-10-04 Thread Martin Jansa

> Still this is very prone to errors, so I personally rather have a
space too much than to few (esp. as the else clause doesn't add a space).

The missing space in else clause is often intentional in cases like this,
so that the append has no impact on value when not enabled
with GTKDOC_ENABLED. Why should it add useless space when not enabled? Just
for the possibility of someone adding something to else clause without
leading space there?

On Mon, Oct 4, 2021 at 5:14 PM Konrad Weihmann 
wrote:

>
>
> On 04.10.21 17:08, Peter Kjellerstedt wrote:
> >> -Original Message-
> >> From: openembedded-core@lists.openembedded.org  >> c...@lists.openembedded.org> On Behalf Of Konrad Weihmann
> >> Sent: den 4 oktober 2021 15:29
> >> To: openembedded-core@lists.openembedded.org
> >> Cc: Konrad Weihmann 
> >> Subject: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS
> >>
> >> fix the append with a leading blank, otherwise this messes up
> >> DEPENDS when globally inherited classes add dependecies of their own
> >>
> >> Signed-off-by: Konrad Weihmann 
> >> ---
> >>   meta/recipes-core/glib-2.0/glib.inc | 2 +-
> >>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/meta/recipes-core/glib-2.0/glib.inc
> b/meta/recipes-core/glib-
> >> 2.0/glib.inc
> >> index a34776b083..3436645f15 100644
> >> --- a/meta/recipes-core/glib-2.0/glib.inc
> >> +++ b/meta/recipes-core/glib-2.0/glib.inc
> >> @@ -30,7 +30,7 @@ LEAD_SONAME = "libglib-2.0.*"
> >>
> >>   inherit meson gettext gtk-doc pkgconfig ptest-gnome
> upstream-version-is-
> >> even bash-completion gio-module-cache manpages
> >>
> >> -DEPENDS:append:class-target = "${@' gtk-doc' if
> d.getVar('GTKDOC_ENABLED') == 'True' else ''}"
> >> +DEPENDS:append:class-target = " ${@' gtk-doc' if
> d.getVar('GTKDOC_ENABLED') == 'True' else ''}"
> >
> > This change is not needed as there already is a space inside
> > the if statement (in ' gtk-doc').
>
> Still this is very prone to errors, so I personally rather have a space
> too much than to few (esp. as the else clause doesn't add a space).
>
> IHMO it should be for readability purpose rather " ${@'gtk-doc' if
> d.getVar('GTKDOC_ENABLED') == 'True' else ''}" than like it is now - but
> that's just cosmetics and I'm sure one could discuss about that a lot
>
> >
> >>
> >>   GTKDOC_MESON_OPTION = "gtk_doc"
> >>
> >> --
> >> 2.25.1
> >
> > //Peter
> >
>
> 
>
>

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

Re: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS

2021-10-04 Thread Konrad Weihmann




On 04.10.21 17:08, Peter Kjellerstedt wrote:

-Original Message-
From: openembedded-core@lists.openembedded.org  On Behalf Of Konrad Weihmann
Sent: den 4 oktober 2021 15:29
To: openembedded-core@lists.openembedded.org
Cc: Konrad Weihmann 
Subject: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS

fix the append with a leading blank, otherwise this messes up
DEPENDS when globally inherited classes add dependecies of their own

Signed-off-by: Konrad Weihmann 
---
  meta/recipes-core/glib-2.0/glib.inc | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-
2.0/glib.inc
index a34776b083..3436645f15 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -30,7 +30,7 @@ LEAD_SONAME = "libglib-2.0.*"

  inherit meson gettext gtk-doc pkgconfig ptest-gnome upstream-version-is-
even bash-completion gio-module-cache manpages

-DEPENDS:append:class-target = "${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') == 
'True' else ''}"
+DEPENDS:append:class-target = " ${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') == 
'True' else ''}"


This change is not needed as there already is a space inside
the if statement (in ' gtk-doc').


Still this is very prone to errors, so I personally rather have a space 
too much than to few (esp. as the else clause doesn't add a space).


IHMO it should be for readability purpose rather " ${@'gtk-doc' if 
d.getVar('GTKDOC_ENABLED') == 'True' else ''}" than like it is now - but 
that's just cosmetics and I'm sure one could discuss about that a lot






  GTKDOC_MESON_OPTION = "gtk_doc"

--
2.25.1


//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156611): 
https://lists.openembedded.org/g/openembedded-core/message/156611
Mute This Topic: https://lists.openembedded.org/mt/86066304/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 3/3] update-rc.d: fix append in DEPENDS

2021-10-04 Thread Peter Kjellerstedt

> -Original Message-
> From: openembedded-core@lists.openembedded.org  c...@lists.openembedded.org> On Behalf Of Konrad Weihmann
> Sent: den 4 oktober 2021 15:29
> To: openembedded-core@lists.openembedded.org
> Cc: Konrad Weihmann 
> Subject: [OE-core] [PATCH 3/3] update-rc.d: fix append in DEPENDS
> 
> fix the append with a leading blank, otherwise this messes up
> DEPENDS when globally inherited classes add dependecies of their own
> 
> Signed-off-by: Konrad Weihmann 
> ---
>  meta/classes/update-rc.d.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/update-rc.d.bbclass b/meta/classes/update-
> rc.d.bbclass
> index 0a3a608662..921df98584 100644
> --- a/meta/classes/update-rc.d.bbclass
> +++ b/meta/classes/update-rc.d.bbclass
> @@ -1,6 +1,6 @@
>  UPDATERCPN ?= "${PN}"
> 
> -DEPENDS:append:class-target = "${@bb.utils.contains('DISTRO_FEATURES', 
> 'sysvinit', ' update-rc.d initscripts', '', d)}"
> +DEPENDS:append:class-target = " ${@bb.utils.contains('DISTRO_FEATURES', 
> 'sysvinit', ' update-rc.d initscripts', '', d)}"

This change is not needed as there already is a space inside 
the if statement (in ' update-rc.d initscripts').

> 
>  UPDATERCD = "update-rc.d"
>  UPDATERCD:class-cross = ""
> --
> 2.25.1

//Peter


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

Re: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS

2021-10-04 Thread Peter Kjellerstedt

> -Original Message-
> From: openembedded-core@lists.openembedded.org  c...@lists.openembedded.org> On Behalf Of Konrad Weihmann
> Sent: den 4 oktober 2021 15:29
> To: openembedded-core@lists.openembedded.org
> Cc: Konrad Weihmann 
> Subject: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS
> 
> fix the append with a leading blank, otherwise this messes up
> DEPENDS when globally inherited classes add dependecies of their own
> 
> Signed-off-by: Konrad Weihmann 
> ---
>  meta/recipes-core/glib-2.0/glib.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-
> 2.0/glib.inc
> index a34776b083..3436645f15 100644
> --- a/meta/recipes-core/glib-2.0/glib.inc
> +++ b/meta/recipes-core/glib-2.0/glib.inc
> @@ -30,7 +30,7 @@ LEAD_SONAME = "libglib-2.0.*"
> 
>  inherit meson gettext gtk-doc pkgconfig ptest-gnome upstream-version-is-
> even bash-completion gio-module-cache manpages
> 
> -DEPENDS:append:class-target = "${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') 
> == 'True' else ''}"
> +DEPENDS:append:class-target = " ${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') 
> == 'True' else ''}"

This change is not needed as there already is a space inside 
the if statement (in ' gtk-doc').

> 
>  GTKDOC_MESON_OPTION = "gtk_doc"
> 
> --
> 2.25.1

//Peter


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156609): 
https://lists.openembedded.org/g/openembedded-core/message/156609
Mute This Topic: https://lists.openembedded.org/mt/86066304/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 15/15] glew: Stop polluting /tmp during builds

2021-10-04 Thread Steve Sakoman

From: Richard Purdie 

Currently the glew code creates a new directory in /tmp for each make
invocation. This is a bit ugly, don't do that. The patch does break the
dist targets but we don't use them.

Signed-off-by: Richard Purdie 
(cherry picked from commit 3a55194f90e11da5671b24391a4aaf2b86a8e1e6)
Signed-off-by: Steve Sakoman 
---
 .../glew/glew/notempdir.patch | 19 +++
 meta/recipes-graphics/glew/glew_2.2.0.bb  |  1 +
 2 files changed, 20 insertions(+)
 create mode 100644 meta/recipes-graphics/glew/glew/notempdir.patch

diff --git a/meta/recipes-graphics/glew/glew/notempdir.patch 
b/meta/recipes-graphics/glew/glew/notempdir.patch
new file mode 100644
index 00..8d79ce0cdf
--- /dev/null
+++ b/meta/recipes-graphics/glew/glew/notempdir.patch
@@ -0,0 +1,19 @@
+We don't use the dist-* targets and hence DIST_DIR isn't used. The current code
+creates a new temp directory in /tmp/ for every invocation of make. Lets
+not do that.
+
+Upstream-Status: Pending [a revised version would be needed for upstream]
+Signed-off-by: Richard Purdie 
+
+Index: glew-2.2.0/Makefile
+===
+--- glew-2.2.0.orig/Makefile
 glew-2.2.0/Makefile
+@@ -56,7 +56,6 @@ DIST_SRC_ZIP ?= $(shell pwd)/$(DIST_NAME
+ DIST_SRC_TGZ ?= $(shell pwd)/$(DIST_NAME).tgz
+ DIST_WIN32   ?= $(shell pwd)/$(DIST_NAME)-win32.zip
+ 
+-DIST_DIR := $(shell mktemp -d /tmp/glew.XX)/$(DIST_NAME)
+ 
+ # To disable stripping of linked binaries either:
+ #   - use STRIP= on gmake command-line
diff --git a/meta/recipes-graphics/glew/glew_2.2.0.bb 
b/meta/recipes-graphics/glew/glew_2.2.0.bb
index 92b6083648..d7a26a3438 100644
--- a/meta/recipes-graphics/glew/glew_2.2.0.bb
+++ b/meta/recipes-graphics/glew/glew_2.2.0.bb
@@ -7,6 +7,7 @@ LIC_FILES_CHKSUM = 
"file://LICENSE.txt;md5=2ac251558de685c6b9478d89be3149c2"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/project/glew/glew/${PV}/glew-${PV}.tgz \
file://0001-Fix-build-race-in-Makefile.patch \
+   file://notempdir.patch \
file://no-strip.patch"
 
 SRC_URI[md5sum] = "3579164bccaef09e36c0af7f4fd5c7c7"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156608): 
https://lists.openembedded.org/g/openembedded-core/message/156608
Mute This Topic: https://lists.openembedded.org/mt/86067680/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 14/15] oeqa/buildproject: Ensure temp directories are cleaned up

2021-10-04 Thread Steve Sakoman

From: Richard Purdie 

Signed-off-by: Richard Purdie 
(cherry picked from commit d10aff865120a5feecc42c24726bd119364e0188)
Signed-off-by: Steve Sakoman 
---
 meta/lib/oeqa/utils/buildproject.py | 3 +++
 meta/lib/oeqa/utils/targetbuild.py  | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/utils/buildproject.py 
b/meta/lib/oeqa/utils/buildproject.py
index e6d80cc8dc..dfb9661868 100644
--- a/meta/lib/oeqa/utils/buildproject.py
+++ b/meta/lib/oeqa/utils/buildproject.py
@@ -18,6 +18,7 @@ class BuildProject(metaclass=ABCMeta):
 def __init__(self, uri, foldername=None, tmpdir=None, dl_dir=None):
 self.uri = uri
 self.archive = os.path.basename(uri)
+self.tempdirobj = None
 if not tmpdir:
 self.tempdirobj = 
tempfile.TemporaryDirectory(prefix='buildproject-')
 tmpdir = self.tempdirobj.name
@@ -57,6 +58,8 @@ class BuildProject(metaclass=ABCMeta):
 return self._run('cd %s; make install %s' % (self.targetdir, 
install_args))
 
 def clean(self):
+if self.tempdirobj:
+self.tempdirobj.cleanup()
 if not self.needclean:
  return
 self._run('rm -rf %s' % self.targetdir)
diff --git a/meta/lib/oeqa/utils/targetbuild.py 
b/meta/lib/oeqa/utils/targetbuild.py
index 1055810ca3..09738add1d 100644
--- a/meta/lib/oeqa/utils/targetbuild.py
+++ b/meta/lib/oeqa/utils/targetbuild.py
@@ -19,6 +19,7 @@ class BuildProject(metaclass=ABCMeta):
 self.d = d
 self.uri = uri
 self.archive = os.path.basename(uri)
+self.tempdirobj = None
 if not tmpdir:
 tmpdir = self.d.getVar('WORKDIR')
 if not tmpdir:
@@ -71,9 +72,10 @@ class BuildProject(metaclass=ABCMeta):
 return self._run('cd %s; make install %s' % (self.targetdir, 
install_args))
 
 def clean(self):
+if self.tempdirobj:
+self.tempdirobj.cleanup()
 self._run('rm -rf %s' % self.targetdir)
 subprocess.check_call('rm -f %s' % self.localarchive, shell=True)
-pass
 
 class TargetBuildProject(BuildProject):
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156607): 
https://lists.openembedded.org/g/openembedded-core/message/156607
Mute This Topic: https://lists.openembedded.org/mt/86067679/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 13/15] oeqa/selftest/gotoolchain: Fix temp file cleanup

2021-10-04 Thread Steve Sakoman

From: Richard Purdie 

The go tests leave readonly files and directories behind.
Fix this to allow cleanup.

[YOCTO #14575]

Signed-off-by: Richard Purdie 
(cherry picked from commit 5680e95d7bd9fe00a797b2d0deb8cb4790027508)
Signed-off-by: Steve Sakoman 
---
 meta/lib/oeqa/selftest/cases/gotoolchain.py | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/meta/lib/oeqa/selftest/cases/gotoolchain.py 
b/meta/lib/oeqa/selftest/cases/gotoolchain.py
index 3119520f0d..59f80aad28 100644
--- a/meta/lib/oeqa/selftest/cases/gotoolchain.py
+++ b/meta/lib/oeqa/selftest/cases/gotoolchain.py
@@ -43,6 +43,12 @@ class oeGoToolchainSelfTest(OESelftestTestCase):
 
 @classmethod
 def tearDownClass(cls):
+# Go creates file which are readonly
+for dirpath, dirnames, filenames in os.walk(cls.tmpdir_SDKQA):
+for filename in filenames + dirnames:
+f = os.path.join(dirpath, filename)
+if not os.path.islink(f):
+os.chmod(f, 0o775)
 shutil.rmtree(cls.tmpdir_SDKQA, ignore_errors=True)
 super(oeGoToolchainSelfTest, cls).tearDownClass()
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156606): 
https://lists.openembedded.org/g/openembedded-core/message/156606
Mute This Topic: https://lists.openembedded.org/mt/86067678/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 12/15] rm_work.bbclass: Fix for files starting with -

2021-10-04 Thread Steve Sakoman

From: "William A. Kennington III" 

This makes it possible to name files starting with a hyphen in the work
directory. Without this change rm will fail due to an unexpected option
being passed.

Signed-off-by: William A. Kennington III 
Signed-off-by: Richard Purdie 
(cherry picked from commit 5f1a63e0de4921ef970114a16d0827fcddcdaa0e)
Signed-off-by: Steve Sakoman 
---
 meta/classes/rm_work.bbclass | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/classes/rm_work.bbclass b/meta/classes/rm_work.bbclass
index 01c2ab1c78..2d5a56c238 100644
--- a/meta/classes/rm_work.bbclass
+++ b/meta/classes/rm_work.bbclass
@@ -73,7 +73,7 @@ do_rm_work () {
 # sstate version since otherwise we'd need to leave 'plaindirs' 
around
 # such as 'packages' and 'packages-split' and these can be large. 
No end
 # of chain tasks depend directly on do_package anymore.
-rm -f $i;
+rm -f -- $i;
 ;;
 *_setscene*)
 # Skip stamps which are already setscene versions
@@ -90,7 +90,7 @@ do_rm_work () {
 ;;
 esac
 done
-rm -f $i
+rm -f -- $i
 esac
 done
 
@@ -100,9 +100,9 @@ do_rm_work () {
 # Retain only logs and other files in temp, safely ignore
 # failures of removing pseudo folers on NFS2/3 server.
 if [ $dir = 'pseudo' ]; then
-rm -rf $dir 2> /dev/null || true
+rm -rf -- $dir 2> /dev/null || true
 elif ! echo "$excludes" | grep -q -w "$dir"; then
-rm -rf $dir
+rm -rf -- $dir
 fi
 done
 }
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156605): 
https://lists.openembedded.org/g/openembedded-core/message/156605
Mute This Topic: https://lists.openembedded.org/mt/86067677/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 11/15] libc_package/buildstats: Fix python regex quoting warnings

2021-10-04 Thread Steve Sakoman

From: Richard Purdie 

Signed-off-by: Richard Purdie 
(cherry picked from commit 6a8077317ce12e13018ec4472f728dd24880bda9)
Signed-off-by: Steve Sakoman 
---
 meta/classes/libc-package.bbclass | 2 +-
 meta/lib/buildstats.py| 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/classes/libc-package.bbclass 
b/meta/classes/libc-package.bbclass
index de3b4250c7..1143f538d6 100644
--- a/meta/classes/libc-package.bbclass
+++ b/meta/classes/libc-package.bbclass
@@ -355,7 +355,7 @@ python package_do_split_gconvs () {
 m.write("\t@echo 'Progress %d/%d'\n" % (i, total))
 m.write("\t" + makerecipe + "\n\n")
 d.setVar("EXTRA_OEMAKE", "-C %s ${PARALLEL_MAKE}" % 
(os.path.dirname(makefile)))
-d.setVarFlag("oe_runmake", "progress", "outof:Progress\s(\d+)/(\d+)")
+d.setVarFlag("oe_runmake", "progress", r"outof:Progress\s(\d+)/(\d+)")
 bb.note("Executing binary locale generation makefile")
 bb.build.exec_func("oe_runmake", d)
 bb.note("collecting binary locales from locale tree")
diff --git a/meta/lib/buildstats.py b/meta/lib/buildstats.py
index 8627ed3c31..c52b6c3b72 100644
--- a/meta/lib/buildstats.py
+++ b/meta/lib/buildstats.py
@@ -43,8 +43,8 @@ class SystemStats:
 # depends on the heartbeat event, which fires less often.
 self.min_seconds = 1
 
-self.meminfo_regex = 
re.compile(b'^(MemTotal|MemFree|Buffers|Cached|SwapTotal|SwapFree):\s*(\d+)')
-self.diskstats_regex = 
re.compile(b'^([hsv]d.|mtdblock\d|mmcblk\d|cciss/c\d+d\d+.*)$')
+self.meminfo_regex = 
re.compile(rb'^(MemTotal|MemFree|Buffers|Cached|SwapTotal|SwapFree):\s*(\d+)')
+self.diskstats_regex = 
re.compile(rb'^([hsv]d.|mtdblock\d|mmcblk\d|cciss/c\d+d\d+.*)$')
 self.diskstats_ltime = None
 self.diskstats_data = None
 self.stat_ltimes = None
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156604): 
https://lists.openembedded.org/g/openembedded-core/message/156604
Mute This Topic: https://lists.openembedded.org/mt/86067675/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 10/15] oeqa/qemurunner: Use oe._exit(), not sys.exit()

2021-10-04 Thread Steve Sakoman

From: Richard Purdie 

sys.exit will cause finally statements and other code to run at exit. Since
we're using os.fork() here, os._exit() is apprioriate in this codepath.

Signed-off-by: Richard Purdie 
(cherry picked from commit ec08498ff29de9ccd23be88b9d7af3dab6bbb81e)
Signed-off-by: Steve Sakoman 
---
 meta/lib/oeqa/utils/qemurunner.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/lib/oeqa/utils/qemurunner.py 
b/meta/lib/oeqa/utils/qemurunner.py
index f7e5a3b3a6..de0dff3ff0 100644
--- a/meta/lib/oeqa/utils/qemurunner.py
+++ b/meta/lib/oeqa/utils/qemurunner.py
@@ -234,7 +234,7 @@ class QemuRunner:
 r = os.fdopen(r)
 x = r.read()
 os.killpg(os.getpgid(self.runqemu.pid), signal.SIGTERM)
-sys.exit(0)
+os._exit(0)
 
 self.logger.debug("runqemu started, pid is %s" % self.runqemu.pid)
 self.logger.debug("waiting at most %s seconds for qemu pid (%s)" %
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156603): 
https://lists.openembedded.org/g/openembedded-core/message/156603
Mute This Topic: https://lists.openembedded.org/mt/86067674/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 09/15] pybootchart: Avoid divide by zero

2021-10-04 Thread Steve Sakoman

From: Richard Purdie 

Avoid a rare divide by zero error if there isn't data point spread.

[YOCTO #14547]

Signed-off-by: Richard Purdie 
(cherry picked from commit d7e36d01e87ddf89f76f164a0b7d98f597a53fa5)
Signed-off-by: Steve Sakoman 
---
 scripts/pybootchartgui/pybootchartgui/draw.py | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/scripts/pybootchartgui/pybootchartgui/draw.py 
b/scripts/pybootchartgui/pybootchartgui/draw.py
index 29eb7505bc..fc708b55c3 100644
--- a/scripts/pybootchartgui/pybootchartgui/draw.py
+++ b/scripts/pybootchartgui/pybootchartgui/draw.py
@@ -267,7 +267,10 @@ def draw_chart(ctx, color, fill, chart_bounds, data, 
proc_tree, data_range):
 # avoid divide by zero
 if max_y == 0:
 max_y = 1.0
-xscale = float (chart_bounds[2]) / (max_x - x_shift)
+if (max_x - x_shift):
+xscale = float (chart_bounds[2]) / (max_x - x_shift)
+else:
+xscale = float (chart_bounds[2])
 # If data_range is given, scale the chart so that the value range in
 # data_range matches the chart bounds exactly.
 # Otherwise, scale so that the actual data matches the chart bounds.
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156602): 
https://lists.openembedded.org/g/openembedded-core/message/156602
Mute This Topic: https://lists.openembedded.org/mt/86067672/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 08/15] libsamplerate0: Set correct soname for 0.1.9

2021-10-04 Thread Steve Sakoman

From: Tom Pollard 

Manually patch SHARED_VERSION_INFO, which was missed in
the 0.1.9 release and later incorrectly fixed until 0.2.1

Signed-off-by: Tom Pollard 
Signed-off-by: Richard Purdie 
(cherry picked from commit cb2e8efd316d44b9b1453882114856e0eb7b3500)
Signed-off-by: Steve Sakoman 
---
 .../libsamplerate0/shared_version_info.patch| 13 +
 .../libsamplerate/libsamplerate0_0.1.9.bb   |  1 +
 2 files changed, 14 insertions(+)
 create mode 100644 
meta/recipes-multimedia/libsamplerate/libsamplerate0/shared_version_info.patch

diff --git 
a/meta/recipes-multimedia/libsamplerate/libsamplerate0/shared_version_info.patch
 
b/meta/recipes-multimedia/libsamplerate/libsamplerate0/shared_version_info.patch
new file mode 100644
index 00..b42d564b4b
--- /dev/null
+++ 
b/meta/recipes-multimedia/libsamplerate/libsamplerate0/shared_version_info.patch
@@ -0,0 +1,13 @@
+Index: libsamplerate-0.1.8/configure.ac
+===
+--- libsamplerate-0.1.8.orig/configure.ac
 libsamplerate-0.1.8/configure.ac
+@@ -53,7 +53,7 @@ AC_PROG_LN_S
+ #  6. If any interfaces have been removed since the last public release, then 
set age
+ # to 0.
+ 
+-SHARED_VERSION_INFO="1:8:1"
++SHARED_VERSION_INFO="1:9:1"
+ 
+ 
+ 
diff --git a/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb 
b/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
index 6dfc42b436..8345d6880f 100644
--- a/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
+++ b/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
@@ -10,6 +10,7 @@ PR = "r1"
 
 SRC_URI = "http://www.mega-nerd.com/SRC/libsamplerate-${PV}.tar.gz \
file://0001-configure.ac-improve-alsa-handling.patch \
+   file://shared_version_info.patch \
 "
 
 SRC_URI[md5sum] = "2b78ae9fe63b36b9fbb6267fad93f259"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156601): 
https://lists.openembedded.org/g/openembedded-core/message/156601
Mute This Topic: https://lists.openembedded.org/mt/86067671/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 07/15] bzip2: Update soname for libbz2 1.0.8

2021-10-04 Thread Steve Sakoman

From: Tom Pollard 

Set shared library name as libbz2.so.1.0.8, version in configure.ac
already synced via do_configure PV substitution.

Signed-off-by: Tom Pollard 
Signed-off-by: Richard Purdie 
(cherry picked from commit 07e3abc9d282a54add69a6905ec4248f3104219f)
Signed-off-by: Steve Sakoman 
---
 meta/recipes-extended/bzip2/bzip2/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/bzip2/bzip2/Makefile.am 
b/meta/recipes-extended/bzip2/bzip2/Makefile.am
index dcf64584d9..adc85a62b2 100644
--- a/meta/recipes-extended/bzip2/bzip2/Makefile.am
+++ b/meta/recipes-extended/bzip2/bzip2/Makefile.am
@@ -1,6 +1,6 @@
 
 lib_LTLIBRARIES = libbz2.la
-libbz2_la_LDFLAGS = -version-info 1:6:0
+libbz2_la_LDFLAGS = -version-info 1:8:0
 
 libbz2_la_SOURCES = blocksort.c  \
 huffman.c\
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156600): 
https://lists.openembedded.org/g/openembedded-core/message/156600
Mute This Topic: https://lists.openembedded.org/mt/86067670/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 06/15] common-licenses: add "Unlicense" license file

2021-10-04 Thread Steve Sakoman

From: "Robert P. J. Day" 

Signed-off-by: Robert P. J. Day 
Signed-off-by: Richard Purdie 
(cherry picked from commit 7698f3145a9c48778d7bb79f54df169cf375349c)
Signed-off-by: Steve Sakoman 
---
 meta/files/common-licenses/Unlicense | 24 
 1 file changed, 24 insertions(+)
 create mode 100644 meta/files/common-licenses/Unlicense

diff --git a/meta/files/common-licenses/Unlicense 
b/meta/files/common-licenses/Unlicense
new file mode 100644
index 00..68a49daad8
--- /dev/null
+++ b/meta/files/common-licenses/Unlicense
@@ -0,0 +1,24 @@
+This is free and unencumbered software released into the public domain.
+
+Anyone is free to copy, modify, publish, use, compile, sell, or
+distribute this software, either in source code form or as a compiled
+binary, for any purpose, commercial or non-commercial, and by any
+means.
+
+In jurisdictions that recognize copyright laws, the author or authors
+of this software dedicate any and all copyright interest in the
+software to the public domain. We make this dedication for the benefit
+of the public at large and to the detriment of our heirs and
+successors. We intend this dedication to be an overt act of
+relinquishment in perpetuity of all present and future rights to this
+software under copyright law.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
+OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+OTHER DEALINGS IN THE SOFTWARE.
+
+For more information, please refer to 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156599): 
https://lists.openembedded.org/g/openembedded-core/message/156599
Mute This Topic: https://lists.openembedded.org/mt/86067667/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 05/15] systemd: Add fix for systemd-networkd crash during free

2021-10-04 Thread Steve Sakoman

From: Ranjitsinh Rathod 

We are observing systemd-network service crash during link down
while freeing link->ifname pointer

Backtrace:
(gdb) bt
0  __GI_abort () at abort.c:107
1  0x007f861d32b4 in __libc_message (action=action@entry=do_abort, 
fmt=fmt@entry=0x7f8628d500 "%s\n") at ../sysdeps/posix/libc_fatal.c:155
2  0x007f861da51c in malloc_printerr (str=str@entry=0x7f86289070 "free(): 
invalid next size (fast)") at malloc.c:5347
3  0x007f861dbd58 in _int_free (av=0x7f862c9a28 , 
p=0x558aa28eb0, have_lock=0) at malloc.c:4249
4  0x005569249cf0 in link_free (link=0x558aa1c0d0) at 
../git/src/network/networkd-link.c:715
5  link_unref (p=0x558aa1c0d0) at ../git/src/network/networkd-link.c:734
6  0x00556920f34c in manager_rtnl_process_link (rtnl=, 
message=0x558aa2a430, userdata=0x558a9fc630)

While checking upstream code change with regards to link->ifname
memory allocation and free, we found below PR which also fixes
random systemd-networkd crash:

https://github.com/systemd/systemd/pull/19631
https://github.com/systemd/systemd/issues/19629

Signed-off-by: Ranjitsinh Rathod 
Signed-off-by: Ranjitsinh Rathod 
Signed-off-by: Steve Sakoman 
---
 ...-info-for-ordered-set-new-and-introd.patch |  78 +
 ...dered_set_clear-free-with-destructor.patch |  35 +++
 ...etwork-add-skeleton-of-request-queue.patch | 285 ++
 ...quests-when-link-enters-linger-state.patch |  50 +++
 ...ork-fix-Link-reference-counter-issue.patch | 278 +
 ...nk_drop-and-link_detach_from_manager.patch |  67 
 meta/recipes-core/systemd/systemd_244.5.bb|   6 +
 7 files changed, 799 insertions(+)
 create mode 100644 
meta/recipes-core/systemd/systemd/basic-pass-allocation-info-for-ordered-set-new-and-introd.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/introduce-ordered_set_clear-free-with-destructor.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-add-skeleton-of-request-queue.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-also-drop-requests-when-link-enters-linger-state.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-fix-Link-reference-counter-issue.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-merge-link_drop-and-link_detach_from_manager.patch

diff --git 
a/meta/recipes-core/systemd/systemd/basic-pass-allocation-info-for-ordered-set-new-and-introd.patch
 
b/meta/recipes-core/systemd/systemd/basic-pass-allocation-info-for-ordered-set-new-and-introd.patch
new file mode 100644
index 00..86d9b0499a
--- /dev/null
+++ 
b/meta/recipes-core/systemd/systemd/basic-pass-allocation-info-for-ordered-set-new-and-introd.patch
@@ -0,0 +1,78 @@
+From 1f25c71d9d0b5fe6cf383c347dcebc2443a99fe1 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= 
+Date: Tue, 1 Sep 2020 12:42:35 +0200
+Subject: [PATCH] basic: pass allocation info for ordered_set_new() and
+ introduce ordered_set_ensure_put()
+
+Upstream-Status: Backport 
[https://github.com/systemd/systemd-stable/commit/1f25c71d9d0b5fe6cf383c347dcebc2443a99fe1]
+Signed-off-by: Ranjitsinh Rathod 
+
+---
+ src/basic/ordered-set.c | 21 +
+ src/basic/ordered-set.h | 18 +++---
+ 2 files changed, 28 insertions(+), 11 deletions(-)
+
+diff --git a/src/basic/ordered-set.c b/src/basic/ordered-set.c
+index 7fdb47e064..fb82c17b5a 100644
+--- a/src/basic/ordered-set.c
 b/src/basic/ordered-set.c
+@@ -4,6 +4,27 @@
+ #include "ordered-set.h"
+ #include "strv.h"
+ 
++int _ordered_set_ensure_allocated(OrderedSet **s, const struct hash_ops *ops  
HASHMAP_DEBUG_PARAMS) {
++if (*s)
++return 0;
++
++*s = _ordered_set_new(ops  HASHMAP_DEBUG_PASS_ARGS);
++if (!*s)
++return -ENOMEM;
++
++return 0;
++}
++
++int _ordered_set_ensure_put(OrderedSet **s, const struct hash_ops *ops, void 
*p  HASHMAP_DEBUG_PARAMS) {
++int r;
++
++r = _ordered_set_ensure_allocated(s, ops  HASHMAP_DEBUG_PASS_ARGS);
++if (r < 0)
++return r;
++
++return ordered_set_put(*s, p);
++}
++
+ int ordered_set_consume(OrderedSet *s, void *p) {
+ int r;
+ 
+diff --git a/src/basic/ordered-set.h b/src/basic/ordered-set.h
+index a42a57eb49..2c241a808b 100644
+--- a/src/basic/ordered-set.h
 b/src/basic/ordered-set.h
+@@ -7,20 +7,16 @@
+ 
+ typedef struct OrderedSet OrderedSet;
+ 
+-static inline OrderedSet* ordered_set_new(const struct hash_ops *ops) {
+-return (OrderedSet*) ordered_hashmap_new(ops);
++static inline OrderedSet* _ordered_set_new(const struct hash_ops *ops  
HASHMAP_DEBUG_PARAMS) {
++return (OrderedSet*) internal_ordered_hashmap_new(ops  
HASHMAP_DEBUG_PASS_ARGS);
+ }
++#define ordered_set_new(ops) _ordered_set_new(ops  HASHMAP_DEBUG_SRC_ARGS)
+ 
+-static inline int ordered_set_ensure_allocated(OrderedSet **s, const struct 
hash_ops *ops) {
+-if (*s)
+-return 0;
++int _orde

[OE-core][dunfell 04/15] mtd-utils: upgrade 2.1.2 -> 2.1.3

2021-10-04 Thread Steve Sakoman

From: Stefano Babic 

Drop also --enable-install-tests from configuration options because this
was removed in 2.1.3.

(cherry picked from commit c95c852b84f02f5e2ad5c575ab683bba0471f221)
Signed-off-by: Stefano Babic 
CC: David Oberhollenzer 
CC: Alexandre Belloni 
Signed-off-by: Alexandre Belloni 
Signed-off-by: Richard Purdie 
Signed-off-by: Steve Sakoman 
---
 meta/recipes-devtools/mtd/mtd-utils_git.bb | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/mtd/mtd-utils_git.bb 
b/meta/recipes-devtools/mtd/mtd-utils_git.bb
index 55f05dc8f9..d8355bbad3 100644
--- a/meta/recipes-devtools/mtd/mtd-utils_git.bb
+++ b/meta/recipes-devtools/mtd/mtd-utils_git.bb
@@ -11,17 +11,15 @@ inherit autotools pkgconfig update-alternatives
 DEPENDS = "zlib e2fsprogs util-linux"
 RDEPENDS_mtd-utils-tests += "bash"
 
-PV = "2.1.2"
+PV = "2.1.3"
 
-SRCREV = "7b986779342021bda87c04da3bf729718736d8ab"
+SRCREV = "42ea7cd48d2b3c306d59bb6c530d79f8c25bf9f5"
 SRC_URI = "git://git.infradead.org/mtd-utils.git \
file://add-exclusion-to-mkfs-jffs2-git-2.patch \
"
 
 S = "${WORKDIR}/git/"
 
-EXTRA_OECONF += "--enable-install-tests"
-
 # xattr support creates an additional compile-time dependency on acl because
 # the sys/acl.h header is needed. libacl is not needed and thus enabling xattr
 # regardless whether acl is enabled or disabled in the distro should be okay.
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156597): 
https://lists.openembedded.org/g/openembedded-core/message/156597
Mute This Topic: https://lists.openembedded.org/mt/86067661/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 03/15] mtd-utils: upgrade 2.1.1 -> 2.1.2

2021-10-04 Thread Steve Sakoman

From: Richard Purdie 

Drop backported patch.

(cherry picked from commit e38fd1ac331d824b2db94a7ae46026b111257e83)
Signed-off-by: Richard Purdie 
Signed-off-by: Steve Sakoman 
---
 ...-utils-Fix-return-value-of-ubiformat.patch | 62 ---
 meta/recipes-devtools/mtd/mtd-utils_git.bb|  7 +--
 2 files changed, 3 insertions(+), 66 deletions(-)
 delete mode 100644 
meta/recipes-devtools/mtd/mtd-utils/0001-mtd-utils-Fix-return-value-of-ubiformat.patch

diff --git 
a/meta/recipes-devtools/mtd/mtd-utils/0001-mtd-utils-Fix-return-value-of-ubiformat.patch
 
b/meta/recipes-devtools/mtd/mtd-utils/0001-mtd-utils-Fix-return-value-of-ubiformat.patch
deleted file mode 100644
index d43f7e1a7a..00
--- 
a/meta/recipes-devtools/mtd/mtd-utils/0001-mtd-utils-Fix-return-value-of-ubiformat.patch
+++ /dev/null
@@ -1,62 +0,0 @@
-From 4d19bffcfd66e25d3ee74536ae2d2da7ad52e8e2 Mon Sep 17 00:00:00 2001
-From: Barry Grussling 
-Date: Sun, 12 Jan 2020 12:33:32 -0800
-Subject: [PATCH] mtd-utils: Fix return value of ubiformat
-Organization: O.S. Systems Software LTDA.
-
-This changeset fixes a feature regression in ubiformat.  Older versions of
-ubiformat, when invoked with a flash-image, would return 0 in the case no error
-was encountered.  Upon upgrading to latest, it was discovered that ubiformat
-returned 255 even without encountering an error condition.
-
-This changeset corrects the above issue and causes ubiformat, when given an
-image file, to return 0 when no errors are detected.
-
-Tested by running through my loading scripts and verifying ubiformat returned
-0.
-
-Upstream-Status: Backport [2.1.2]
-
-Signed-off-by: Barry Grussling 
-Signed-off-by: David Oberhollenzer 
-Signed-off-by: Otavio Salvador 

- ubi-utils/ubiformat.c | 7 +--
- 1 file changed, 5 insertions(+), 2 deletions(-)
-
-diff --git a/ubi-utils/ubiformat.c b/ubi-utils/ubiformat.c
-index a90627c..5377b12 100644
 a/ubi-utils/ubiformat.c
-+++ b/ubi-utils/ubiformat.c
-@@ -550,6 +550,7 @@ static int format(libmtd_t libmtd, const struct 
mtd_dev_info *mtd,
-   struct ubi_vtbl_record *vtbl;
-   int eb1 = -1, eb2 = -1;
-   long long ec1 = -1, ec2 = -1;
-+  int ret = -1;
- 
-   write_size = UBI_EC_HDR_SIZE + mtd->subpage_size - 1;
-   write_size /= mtd->subpage_size;
-@@ -643,8 +644,10 @@ static int format(libmtd_t libmtd, const struct 
mtd_dev_info *mtd,
-   if (!args.quiet && !args.verbose)
-   printf("\n");
- 
--  if (novtbl)
-+  if (novtbl) {
-+  ret = 0;
-   goto out_free;
-+  }
- 
-   if (eb1 == -1 || eb2 == -1) {
-   errmsg("no eraseblocks for volume table");
-@@ -669,7 +672,7 @@ static int format(libmtd_t libmtd, const struct 
mtd_dev_info *mtd,
- 
- out_free:
-   free(hdr);
--  return -1;
-+  return ret;
- }
- 
- int main(int argc, char * const argv[])
--- 
-2.27.0
-
diff --git a/meta/recipes-devtools/mtd/mtd-utils_git.bb 
b/meta/recipes-devtools/mtd/mtd-utils_git.bb
index 9c05dc03dc..55f05dc8f9 100644
--- a/meta/recipes-devtools/mtd/mtd-utils_git.bb
+++ b/meta/recipes-devtools/mtd/mtd-utils_git.bb
@@ -11,13 +11,12 @@ inherit autotools pkgconfig update-alternatives
 DEPENDS = "zlib e2fsprogs util-linux"
 RDEPENDS_mtd-utils-tests += "bash"
 
-PV = "2.1.1"
+PV = "2.1.2"
 
-SRCREV = "4443221ce9b88440cd9f5bb78e6fe95621d36c8a"
+SRCREV = "7b986779342021bda87c04da3bf729718736d8ab"
 SRC_URI = "git://git.infradead.org/mtd-utils.git \
file://add-exclusion-to-mkfs-jffs2-git-2.patch \
-   file://0001-mtd-utils-Fix-return-value-of-ubiformat.patch \
-"
+   "
 
 S = "${WORKDIR}/git/"
 
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156596): 
https://lists.openembedded.org/g/openembedded-core/message/156596
Mute This Topic: https://lists.openembedded.org/mt/86067660/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 02/15] openssh: Fix CVE-2021-28041

2021-10-04 Thread Steve Sakoman

From: sana kazi 

Added patch to fix CVE-2021-28041.

Link:
http://archive.ubuntu.com/ubuntu/pool/main/o/openssh/openssh_8.2p1-4ubuntu0.3.debian.tar.xz

Signed-off-by: Sana Kazi 
Signed-off-by: Sana Kazi 
Signed-off-by: Steve Sakoman 
---
 .../openssh/openssh/CVE-2021-28041.patch  | 20 +++
 .../openssh/openssh_8.2p1.bb  |  1 +
 2 files changed, 21 insertions(+)
 create mode 100644 
meta/recipes-connectivity/openssh/openssh/CVE-2021-28041.patch

diff --git a/meta/recipes-connectivity/openssh/openssh/CVE-2021-28041.patch 
b/meta/recipes-connectivity/openssh/openssh/CVE-2021-28041.patch
new file mode 100644
index 00..9fd7e932d1
--- /dev/null
+++ b/meta/recipes-connectivity/openssh/openssh/CVE-2021-28041.patch
@@ -0,0 +1,20 @@
+Description: fix double-free memory corruption in ssh-agent
+Author: Marc Deslauriers 
+Origin: minimal fix for 
https://github.com/openssh/openssh-portable/commit/e04fd6dde16de1cdc5a4d9946397ff60d96568db
+
+Signed-off-by: Sana Kazi 
+
+CVE: CVE-2021-28041
+Upstream-Status: Backport 
[http://archive.ubuntu.com/ubuntu/pool/main/o/openssh/openssh_8.2p1-4ubuntu0.3.debian.tar.xz]
+Comment: No change in any hunk
+
+--- a/ssh-agent.c
 b/ssh-agent.c
+@@ -496,6 +496,7 @@ process_add_identity(SocketEntry *e)
+   goto err;
+   }
+   free(ext_name);
++  ext_name = NULL;
+   break;
+   default:
+   error("%s: Unknown constraint %d", __func__, ctype);
diff --git a/meta/recipes-connectivity/openssh/openssh_8.2p1.bb 
b/meta/recipes-connectivity/openssh/openssh_8.2p1.bb
index 64a0a72a8f..5f03bdc877 100644
--- a/meta/recipes-connectivity/openssh/openssh_8.2p1.bb
+++ b/meta/recipes-connectivity/openssh/openssh_8.2p1.bb
@@ -25,6 +25,7 @@ SRC_URI = 
"http://ftp.openbsd.org/pub/OpenBSD/OpenSSH/portable/openssh-${PV}.tar
file://sshd_check_keys \
file://add-test-support-for-busybox.patch \
file://CVE-2020-14145.patch \
+   file://CVE-2021-28041.patch \
"
 SRC_URI[md5sum] = "3076e6413e8dbe56d33848c1054ac091"
 SRC_URI[sha256sum] = 
"43925151e6cf6cee1450190c0e9af4dc36b41c12737619edff8bcebdff64e671"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156595): 
https://lists.openembedded.org/g/openembedded-core/message/156595
Mute This Topic: https://lists.openembedded.org/mt/86067657/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 01/15] vim: fix CVE-2021-3778

2021-10-04 Thread Steve Sakoman

From: Minjae Kim 

vim is vulnerable to Heap-based Buffer Overflow

reference:
https://github.com/vim/vim/commit/65b605665997fad54ef39a93199e305af2fe4d7f
Signed-off-by: Steve Sakoman 
---
 .../vim/files/CVE-2021-3778.patch | 49 +++
 meta/recipes-support/vim/vim.inc  |  1 +
 2 files changed, 50 insertions(+)
 create mode 100644 meta/recipes-support/vim/files/CVE-2021-3778.patch

diff --git a/meta/recipes-support/vim/files/CVE-2021-3778.patch 
b/meta/recipes-support/vim/files/CVE-2021-3778.patch
new file mode 100644
index 00..9e1c4d29be
--- /dev/null
+++ b/meta/recipes-support/vim/files/CVE-2021-3778.patch
@@ -0,0 +1,49 @@
+From eb41373c8c88b0789e5cf04669d6116f9a199264 Mon Sep 17 00:00:00 2001
+From: Minjae Kim 
+Date: Sun, 26 Sep 2021 23:48:00 +
+Subject: [PATCH] patch 8.2.3409: reading beyond end of line with invalid utf-8
+ character
+
+Problem: Reading beyond end of line with invalid utf-8 character.
+Solution: Check for NUL when advancing.
+
+Upstream-Status: Accepted 
[https://github.com/vim/vim/commit/65b605665997fad54ef39a93199e305af2fe4d7f]
+CVE: CVE-2021-3778
+Signed-off-by: Minjae Kim 
+---
+ src/regexp_nfa.c | 3 ++-
+ src/testdir/test_regexp_utf8.vim | 7 +++
+ 2 files changed, 9 insertions(+), 1 deletion(-)
+
+diff --git a/src/regexp_nfa.c b/src/regexp_nfa.c
+index fb512f961..4d337f1f1 100644
+--- a/src/regexp_nfa.c
 b/src/regexp_nfa.c
+@@ -5455,7 +5455,8 @@ find_match_text(colnr_T startcol, int regstart, char_u 
*match_text)
+   match = FALSE;
+   break;
+   }
+-  len2 += MB_CHAR2LEN(c2);
++  len2 += enc_utf8 ? utf_ptr2len(rex.line + col + len2)
++   : MB_CHAR2LEN(c2);
+   }
+   if (match
+   // check that no composing char follows
+diff --git a/src/testdir/test_regexp_utf8.vim 
b/src/testdir/test_regexp_utf8.vim
+index 19ff882be..e0665818b 100644
+--- a/src/testdir/test_regexp_utf8.vim
 b/src/testdir/test_regexp_utf8.vim
+@@ -215,3 +215,10 @@ func Test_optmatch_toolong()
+   set re=0
+ endfunc
+ 
++func Test_match_invalid_byte()
++  call writefile(0z630a.765d30aa0a.2e0a.790a.4030, 'Xinvalid')
++  new
++  source Xinvalid
++  bwipe!
++  call delete('Xinvalid')
++endfunc
+-- 
+2.17.1
+
diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes-support/vim/vim.inc
index ecaba7107e..fc4e205b74 100644
--- a/meta/recipes-support/vim/vim.inc
+++ b/meta/recipes-support/vim/vim.inc
@@ -18,6 +18,7 @@ SRC_URI = "git://github.com/vim/vim.git \
file://no-path-adjust.patch \
file://racefix.patch \
file://b7081e135a16091c93f6f5f7525a5c58fb7ca9f9.patch \
+  file://CVE-2021-3778.patch \
 "
 
 SRCREV = "98056533b96b6b5d8849641de93185dd7bcadc44"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156594): 
https://lists.openembedded.org/g/openembedded-core/message/156594
Mute This Topic: https://lists.openembedded.org/mt/86067655/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 00/15] Patch review

2021-10-04 Thread Steve Sakoman

Please review this next set of patches for dunfell and have comments back by end
of day Wednesday.

Passed a-full on autobuilder:

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

The following changes since commit 8e7c8e43260682efafabc50c757b9c2daff98f13:

  connman: add CVE_PRODUCT (2021-09-24 04:27:46 -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

Minjae Kim (1):
  vim: fix CVE-2021-3778

Ranjitsinh Rathod (1):
  systemd: Add fix for systemd-networkd crash during free

Richard Purdie (7):
  mtd-utils: upgrade 2.1.1 -> 2.1.2
  pybootchart: Avoid divide by zero
  oeqa/qemurunner: Use oe._exit(), not sys.exit()
  libc_package/buildstats: Fix python regex quoting warnings
  oeqa/selftest/gotoolchain: Fix temp file cleanup
  oeqa/buildproject: Ensure temp directories are cleaned up
  glew: Stop polluting /tmp during builds

Robert P. J. Day (1):
  common-licenses: add "Unlicense" license file

Stefano Babic (1):
  mtd-utils: upgrade 2.1.2 -> 2.1.3

Tom Pollard (2):
  bzip2: Update soname for libbz2 1.0.8
  libsamplerate0: Set correct soname for 0.1.9

William A. Kennington III (1):
  rm_work.bbclass: Fix for files starting with -

sana kazi (1):
  openssh: Fix CVE-2021-28041

 meta/classes/libc-package.bbclass |   2 +-
 meta/classes/rm_work.bbclass  |   8 +-
 meta/files/common-licenses/Unlicense  |  24 ++
 meta/lib/buildstats.py|   4 +-
 meta/lib/oeqa/selftest/cases/gotoolchain.py   |   6 +
 meta/lib/oeqa/utils/buildproject.py   |   3 +
 meta/lib/oeqa/utils/qemurunner.py |   2 +-
 meta/lib/oeqa/utils/targetbuild.py|   4 +-
 .../openssh/openssh/CVE-2021-28041.patch  |  20 ++
 .../openssh/openssh_8.2p1.bb  |   1 +
 ...-info-for-ordered-set-new-and-introd.patch |  78 +
 ...dered_set_clear-free-with-destructor.patch |  35 +++
 ...etwork-add-skeleton-of-request-queue.patch | 285 ++
 ...quests-when-link-enters-linger-state.patch |  50 +++
 ...ork-fix-Link-reference-counter-issue.patch | 278 +
 ...nk_drop-and-link_detach_from_manager.patch |  67 
 meta/recipes-core/systemd/systemd_244.5.bb|   6 +
 ...-utils-Fix-return-value-of-ubiformat.patch |  62 
 meta/recipes-devtools/mtd/mtd-utils_git.bb|   9 +-
 meta/recipes-extended/bzip2/bzip2/Makefile.am |   2 +-
 .../glew/glew/notempdir.patch |  19 ++
 meta/recipes-graphics/glew/glew_2.2.0.bb  |   1 +
 .../libsamplerate0/shared_version_info.patch  |  13 +
 .../libsamplerate/libsamplerate0_0.1.9.bb |   1 +
 .../vim/files/CVE-2021-3778.patch |  49 +++
 meta/recipes-support/vim/vim.inc  |   1 +
 scripts/pybootchartgui/pybootchartgui/draw.py |   5 +-
 27 files changed, 956 insertions(+), 79 deletions(-)
 create mode 100644 meta/files/common-licenses/Unlicense
 create mode 100644 
meta/recipes-connectivity/openssh/openssh/CVE-2021-28041.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/basic-pass-allocation-info-for-ordered-set-new-and-introd.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/introduce-ordered_set_clear-free-with-destructor.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-add-skeleton-of-request-queue.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-also-drop-requests-when-link-enters-linger-state.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-fix-Link-reference-counter-issue.patch
 create mode 100644 
meta/recipes-core/systemd/systemd/network-merge-link_drop-and-link_detach_from_manager.patch
 delete mode 100644 
meta/recipes-devtools/mtd/mtd-utils/0001-mtd-utils-Fix-return-value-of-ubiformat.patch
 create mode 100644 meta/recipes-graphics/glew/glew/notempdir.patch
 create mode 100644 
meta/recipes-multimedia/libsamplerate/libsamplerate0/shared_version_info.patch
 create mode 100644 meta/recipes-support/vim/files/CVE-2021-3778.patch

-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156593): 
https://lists.openembedded.org/g/openembedded-core/message/156593
Mute This Topic: https://lists.openembedded.org/mt/86067650/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] sstate.bbclass: adds warnings for the exceptions of os.utime in siginfo

2021-10-04 Thread Richard Purdie

On Sun, 2021-10-03 at 22:38 +0100, Jose Quaresma wrote:
> When we can't update the access and modified times of sstate siginfo
> it is useful to infor the user about that.
> 
> Signed-off-by: Jose Quaresma 
> ---
>  meta/classes/sstate.bbclass | 12 ++--
>  1 file changed, 10 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
> index 92a73114bb..2deecd8777 100644
> --- a/meta/classes/sstate.bbclass
> +++ b/meta/classes/sstate.bbclass
> @@ -720,11 +720,15 @@ def sstate_package(ss, d):
>  try:
>  os.utime(siginfo, None)
>  except PermissionError:
> +bb.warn("Cannot sets the access and modified times of sstate 
> siginfo %s, no access rights" % siginfo)
>  pass
>  except OSError as e:
>  # Handle read-only file systems gracefully
>  import errno
> -if e.errno != errno.EROFS:
> +if e.errno == errno.EROFS:
> +bb.warn("Cannot sets the access and modified times of sstate 
> siginfo %s, read-only file system" % siginfo)
> +pass
> +else:
>  raise e
>  
>  return
> @@ -1165,11 +1169,15 @@ python sstate_eventhandler() {
>  try:
>  os.utime(siginfo, None)
>  except PermissionError:
> +bb.warn("Cannot sets the access and modified times of sstate 
> siginfo %s, no access rights" % siginfo)
>  pass
>  except OSError as e:
>  # Handle read-only file systems gracefully
>  import errno
> -if e.errno != errno.EROFS:
> +if e.errno == errno.EROFS:
> +bb.warn("Cannot sets the access and modified times of 
> sstate siginfo %s, read-only file system" % siginfo)
> +pass
> +else:
>  raise e
>  
> 

I'm not sure we can do this as there are some cases where this is unavoidable by
people's state setups and we wouldn't want to show warnings in those cases.

(e.g. think of a shared sstate dir between multiple users where the objects
remain owned by the user that created them).


Cheers,

Richard



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156592): 
https://lists.openembedded.org/g/openembedded-core/message/156592
Mute This Topic: https://lists.openembedded.org/mt/86052256/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] [poky] [PATCH] sstate: Switch to ZStandard compressor support

2021-10-04 Thread Alexander Kanavin

Yes, it's here:
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=abelloni/kirkstone-next

Alex

On Mon, 4 Oct 2021 at 14:26, Henry Kleynhans  wrote:

> Hi Alex,
>
> I cannot see abelloni/kirkstone-next here:
> http://git.yoctoproject.org/cgit/cgit.cgi/poky/refs/heads
>
> Do I need to get them somewhere else?
>
> Kind regards, Henry
> --
> *From:* p...@lists.yoctoproject.org  on
> behalf of Alexander Kanavin 
> *Sent:* 04 October 2021 12:10
> *To:* Henry Kleynhans 
> *Cc:* OE-core ;
> p...@lists.yoctoproject.org ; Michael van
> der Westhuizen 
> *Subject:* Re: [poky] [PATCH] sstate: Switch to ZStandard compressor
> support
>
> ZSTD_THREADS is already defined in bitbake.conf, you do not need to
> redefine it.
> (please use abelloni/kirkstone-next for testing, it is there)
>
> Alex
>
> On Mon, 4 Oct 2021 at 13:00, Henry Kleynhans via lists.yoctoproject.org
>  wrote:
>
> This patch switches the compressor from Gzip to ZStandard for ssate cache
> files.
>
> Zstandard compression provides a significant improvement in
> decompression speed as well as improvement in compression speed and disk
> usage over the 'tgz' format in use.  Furthermore, its configurable
> compression level offers a trade-off between time spent compressing
> sstate cache files and disk space used by those files.  The reduced disk
> usage also contributes to saving network traffic for those sharing their
> sstate cache with others.
>
> Zstandard should therefore be a good choice when:
> * disk space is at a premium
> * network speed / resources are limited
> * the CI server can sstate packages can be created at high compression
> * less CPU on the build server should be used for sstate decompression
>
> Signed-off-by: Henry Kleynhans 
> ---
>  meta/classes/sstate.bbclass| 30 ++
>  meta/conf/bitbake.conf |  2 +-
>  scripts/sstate-cache-management.sh | 40 +++---
>  3 files changed, 41 insertions(+), 31 deletions(-)
>
> diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
> index 92a73114bb..9d40c8ebe4 100644
> --- a/meta/classes/sstate.bbclass
> +++ b/meta/classes/sstate.bbclass
> @@ -1,17 +1,20 @@
>  SSTATE_VERSION = "3"
>
> +ZSTD_THREADS ?= "${BB_NUMBER_THREADS}"
> +SSTATE_ZSTD_CLEVEL = "8"
> +
>  SSTATE_MANIFESTS ?= "${TMPDIR}/sstate-control"
>  SSTATE_MANFILEPREFIX =
> "${SSTATE_MANIFESTS}/manifest-${SSTATE_MANMACH}-${PN}"
>
>  def generate_sstatefn(spec, hash, taskname, siginfo, d):
>  if taskname is None:
> return ""
> -extension = ".tgz"
> +extension = ".tar.zst"
>  # 8 chars reserved for siginfo
>  limit = 254 - 8
>  if siginfo:
>  limit = 254
> -extension = ".tgz.siginfo"
> +extension = ".tar.zst.siginfo"
>  if not hash:
>  hash = "INVALID"
>  fn = spec + hash + "_" + taskname + extension
> @@ -37,7 +40,7 @@ SSTATE_PKGNAME=
> "${SSTATE_EXTRAPATH}${@generate_sstatefn(d.getVar('SSTATE_PK
>  SSTATE_PKG= "${SSTATE_DIR}/${SSTATE_PKGNAME}"
>  SSTATE_EXTRAPATH   = ""
>  SSTATE_EXTRAPATHWILDCARD = ""
> -SSTATE_PATHSPEC   =
> "${SSTATE_DIR}/${SSTATE_EXTRAPATHWILDCARD}*/*/${SSTATE_PKGSPEC}*_${SSTATE_PATH_CURRTASK}.tgz*"
> +SSTATE_PATHSPEC   =
> "${SSTATE_DIR}/${SSTATE_EXTRAPATHWILDCARD}*/*/${SSTATE_PKGSPEC}*_${SSTATE_PATH_CURRTASK}.tar.zst*"
>
>  # explicitly make PV to depend on evaluated value of PV variable
>  PV[vardepvalue] = "${PV}"
> @@ -825,23 +828,24 @@ sstate_create_package () {
> mkdir --mode=0775 -p `dirname ${SSTATE_PKG}`
> TFILE=`mktemp ${SSTATE_PKG}.`
>
> -   # Use pigz if available
> -   OPT="-czS"
> -   if [ -x "$(command -v pigz)" ]; then
> -   OPT="-I pigz -cS"
> +   OPT="-cS"
> +   ZSTD="zstd -${SSTATE_ZSTD_CLEVEL} -T${ZSTD_THREADS}"
> +   # Use pzstd if available
> +   if [ -x "$(command -v pzstd)" ]; then
> +   ZSTD="pzstd -${SSTATE_ZSTD_CLEVEL} -p ${ZSTD_THREADS}"
> fi
>
> # Need to handle empty directories
> if [ "$(ls -A)" ]; then
> set +e
> -   tar $OPT -f $TFILE *
> +   tar -I "$ZSTD" $OPT -f $TFILE *
> ret=$?
> if [ $ret -ne 0 ] && [ $ret -ne 1 ]; then
> exit 1
> fi
> set -e
> else
> -   tar $OPT --file=$TFILE --files-from=/dev/null
> +   tar -I "$ZSTD" $OPT --file=$TFILE --files-from=/dev/null
> fi
> chmod 0664 $TFILE
> # Skip if it was already created by some other process
> @@ -880,7 +884,13 @@ python sstate_report_unihash() {
>  # Will be run from within SSTATE_INSTDIR.
>  #
>  sstate_unpack_package () {
> -   tar -xvzf ${SSTATE_PKG}
> +   ZSTD="zstd -T${ZSTD_THREADS}"
> +   # Use pzstd if available
> +   if [ -x "$(command -v pzstd)" ]; then
> +   ZSTD="pzstd -p ${ZSTD_THREADS}"
> +   fi
> +
> +   tar

[OE-core] [PATCH] sstate: Switch to ZStandard compressor support

2021-10-04 Thread Henry Kleynhans

From: Henry Kleynhans 

This patch switches the compressor from Gzip to ZStandard for ssate cache
files.

Zstandard compression provides a significant improvement in
decompression speed as well as improvement in compression speed and disk
usage over the 'tgz' format in use.  Furthermore, its configurable
compression level offers a trade-off between time spent compressing
sstate cache files and disk space used by those files.  The reduced disk
usage also contributes to saving network traffic for those sharing their
sstate cache with others.

Zstandard should therefore be a good choice when:
* disk space is at a premium
* network speed / resources are limited
* the CI server can sstate packages can be created at high compression
* less CPU on the build server should be used for sstate decompression

Signed-off-by: Henry Kleynhans 
---
 meta/classes/sstate.bbclass| 29 ++
 scripts/sstate-cache-management.sh | 40 +++---
 2 files changed, 39 insertions(+), 30 deletions(-)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index 92a73114bb..3a67aaba19 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -1,17 +1,19 @@
 SSTATE_VERSION = "3"
 
+SSTATE_ZSTD_CLEVEL = "8"
+
 SSTATE_MANIFESTS ?= "${TMPDIR}/sstate-control"
 SSTATE_MANFILEPREFIX = "${SSTATE_MANIFESTS}/manifest-${SSTATE_MANMACH}-${PN}"
 
 def generate_sstatefn(spec, hash, taskname, siginfo, d):
 if taskname is None:
return ""
-extension = ".tgz"
+extension = ".tar.zst"
 # 8 chars reserved for siginfo
 limit = 254 - 8
 if siginfo:
 limit = 254
-extension = ".tgz.siginfo"
+extension = ".tar.zst.siginfo"
 if not hash:
 hash = "INVALID"
 fn = spec + hash + "_" + taskname + extension
@@ -37,7 +39,7 @@ SSTATE_PKGNAME= 
"${SSTATE_EXTRAPATH}${@generate_sstatefn(d.getVar('SSTATE_PK
 SSTATE_PKG= "${SSTATE_DIR}/${SSTATE_PKGNAME}"
 SSTATE_EXTRAPATH   = ""
 SSTATE_EXTRAPATHWILDCARD = ""
-SSTATE_PATHSPEC   = 
"${SSTATE_DIR}/${SSTATE_EXTRAPATHWILDCARD}*/*/${SSTATE_PKGSPEC}*_${SSTATE_PATH_CURRTASK}.tgz*"
+SSTATE_PATHSPEC   = 
"${SSTATE_DIR}/${SSTATE_EXTRAPATHWILDCARD}*/*/${SSTATE_PKGSPEC}*_${SSTATE_PATH_CURRTASK}.tar.zst*"
 
 # explicitly make PV to depend on evaluated value of PV variable
 PV[vardepvalue] = "${PV}"
@@ -825,23 +827,24 @@ sstate_create_package () {
mkdir --mode=0775 -p `dirname ${SSTATE_PKG}`
TFILE=`mktemp ${SSTATE_PKG}.`
 
-   # Use pigz if available
-   OPT="-czS"
-   if [ -x "$(command -v pigz)" ]; then
-   OPT="-I pigz -cS"
+   OPT="-cS"
+   ZSTD="zstd -${SSTATE_ZSTD_CLEVEL} -T${ZSTD_THREADS}"
+   # Use pzstd if available
+   if [ -x "$(command -v pzstd)" ]; then
+   ZSTD="pzstd -${SSTATE_ZSTD_CLEVEL} -p ${ZSTD_THREADS}"
fi
 
# Need to handle empty directories
if [ "$(ls -A)" ]; then
set +e
-   tar $OPT -f $TFILE *
+   tar -I "$ZSTD" $OPT -f $TFILE *
ret=$?
if [ $ret -ne 0 ] && [ $ret -ne 1 ]; then
exit 1
fi
set -e
else
-   tar $OPT --file=$TFILE --files-from=/dev/null
+   tar -I "$ZSTD" $OPT --file=$TFILE --files-from=/dev/null
fi
chmod 0664 $TFILE
# Skip if it was already created by some other process
@@ -880,7 +883,13 @@ python sstate_report_unihash() {
 # Will be run from within SSTATE_INSTDIR.
 #
 sstate_unpack_package () {
-   tar -xvzf ${SSTATE_PKG}
+   ZSTD="zstd -T${ZSTD_THREADS}"
+   # Use pzstd if available
+   if [ -x "$(command -v pzstd)" ]; then
+   ZSTD="pzstd -p ${ZSTD_THREADS}"
+   fi
+
+   tar -I "$ZSTD" -xvf ${SSTATE_PKG}
# update .siginfo atime on local/NFS mirror
[ -O ${SSTATE_PKG}.siginfo ] && [ -w ${SSTATE_PKG}.siginfo ] && [ -h 
${SSTATE_PKG}.siginfo ] && touch -a ${SSTATE_PKG}.siginfo
# Use "! -w ||" to return true for read only files
diff --git a/scripts/sstate-cache-management.sh 
b/scripts/sstate-cache-management.sh
index f1706a2229..d39671f7c6 100755
--- a/scripts/sstate-cache-management.sh
+++ b/scripts/sstate-cache-management.sh
@@ -114,7 +114,7 @@ echo_error () {
 # * Add .done/.siginfo to the remove list
 # * Add destination of symlink to the remove list
 #
-# $1: output file, others: sstate cache file (.tgz)
+# $1: output file, others: sstate cache file (.tar.zst)
 gen_rmlist (){
   local rmlist_file="$1"
   shift
@@ -131,13 +131,13 @@ gen_rmlist (){
   dest="`readlink -e $i`"
   if [ -n "$dest" ]; then
   echo $dest >> $rmlist_file
-  # Remove the .siginfo when .tgz is removed
+  # Remove the .siginfo when .tar.zst is removed
   if [ -f "$dest.siginfo" ]; then
   echo $dest.siginfo >> $rmlist_fi

[OE-core] [PATCH 2/3] glibc: fix append in DEPENDS

2021-10-04 Thread Konrad Weihmann

fix the append with a leading blank, otherwise this messes up
DEPENDS when globally inherited classes add dependecies of their own

Signed-off-by: Konrad Weihmann 
---
 meta/recipes-core/glib-2.0/glib.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/glib-2.0/glib.inc 
b/meta/recipes-core/glib-2.0/glib.inc
index a34776b083..3436645f15 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -30,7 +30,7 @@ LEAD_SONAME = "libglib-2.0.*"
 
 inherit meson gettext gtk-doc pkgconfig ptest-gnome upstream-version-is-even 
bash-completion gio-module-cache manpages
 
-DEPENDS:append:class-target = "${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') == 
'True' else ''}"
+DEPENDS:append:class-target = " ${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') == 
'True' else ''}"
 
 GTKDOC_MESON_OPTION = "gtk_doc"
 
-- 
2.25.1


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

[OE-core] [PATCH 3/3] update-rc.d: fix append in DEPENDS

2021-10-04 Thread Konrad Weihmann

fix the append with a leading blank, otherwise this messes up
DEPENDS when globally inherited classes add dependecies of their own

Signed-off-by: Konrad Weihmann 
---
 meta/classes/update-rc.d.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/update-rc.d.bbclass b/meta/classes/update-rc.d.bbclass
index 0a3a608662..921df98584 100644
--- a/meta/classes/update-rc.d.bbclass
+++ b/meta/classes/update-rc.d.bbclass
@@ -1,6 +1,6 @@
 UPDATERCPN ?= "${PN}"
 
-DEPENDS:append:class-target = "${@bb.utils.contains('DISTRO_FEATURES', 
'sysvinit', ' update-rc.d initscripts', '', d)}"
+DEPENDS:append:class-target = " ${@bb.utils.contains('DISTRO_FEATURES', 
'sysvinit', ' update-rc.d initscripts', '', d)}"
 
 UPDATERCD = "update-rc.d"
 UPDATERCD:class-cross = ""
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156589): 
https://lists.openembedded.org/g/openembedded-core/message/156589
Mute This Topic: https://lists.openembedded.org/mt/86066318/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/3] libical: fix append in DEPENDS

2021-10-04 Thread Konrad Weihmann

fix the append with a leading blank, otherwise this messes up
DEPENDS when globally inherited classes add dependecies of their own

Signed-off-by: Konrad Weihmann 
---
 meta/recipes-support/libical/libical_3.0.10.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-support/libical/libical_3.0.10.bb 
b/meta/recipes-support/libical/libical_3.0.10.bb
index aa5f11e817..209a50217c 100644
--- a/meta/recipes-support/libical/libical_3.0.10.bb
+++ b/meta/recipes-support/libical/libical_3.0.10.bb
@@ -19,7 +19,7 @@ UPSTREAM_CHECK_URI = 
"https://github.com/libical/libical/releases";
 
 inherit cmake pkgconfig
 
-DEPENDS:append:class-target = "libical-native"
+DEPENDS:append:class-target = " libical-native"
 
 PACKAGECONFIG ??= "icu glib"
 PACKAGECONFIG[bdb] = ",-DCMAKE_DISABLE_FIND_PACKAGE_BDB=True,db"
-- 
2.25.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156587): 
https://lists.openembedded.org/g/openembedded-core/message/156587
Mute This Topic: https://lists.openembedded.org/mt/86066302/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/1] ltp: Update to 20210927

2021-10-04 Thread Naresh Kamboju

Hi Daniel and Petr,

> FWIW, we've build-tested it (against Sumo :(, changing the override
> separator), but have yet to execute the actual tests.

Thanks Daniel for creating the latest LTP builds for arm, arm64, i386
and x86_64.
LTP syscall tested on these architectures and no regression found.

15 new test cases pass on arm and arm64.
17 new test cases pass on i386 and x86_64.

- Naresh

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

[OE-core] [PATCH 3/3] reproducible_build: Work around caching issues

2021-10-04 Thread Richard Purdie

SOURCE_DATE_EPOCH can be expanded early in the parsing process before
the class extensions are applied. This can mean the directory pointed
to for the SDE can be incorrect until later in parsing. Cache the file
name in the cached value and allow it to dynamically update.

This isn't ideal but avoding expansion of the variable likely isn't
possible and I'm not sure how else to handle this. This works around
the issue until a better solution can be found.

Signed-off-by: Richard Purdie 
---
 meta/classes/reproducible_build.bbclass | 11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/meta/classes/reproducible_build.bbclass 
b/meta/classes/reproducible_build.bbclass
index 89f645b8584..7571c116c82 100644
--- a/meta/classes/reproducible_build.bbclass
+++ b/meta/classes/reproducible_build.bbclass
@@ -115,11 +115,14 @@ EPOCHTASK = "do_deploy_source_date_epoch"
 do_unpack[postfuncs] += "create_source_date_epoch_stamp"
 
 def get_source_date_epoch_value(d):
-cached = d.getVar('__CACHED_SOURCE_DATE_EPOCH')
-if cached:
+epochfile = d.getVar('SDE_FILE')
+cached, efile = d.getVar('__CACHED_SOURCE_DATE_EPOCH') or (None, None)
+if cached and efile == epochfile:
 return cached
 
-epochfile = d.getVar('SDE_FILE')
+if cached and epochfile != efile:
+bb.debug(1, "Epoch file changed from %s to %s" % (efile, epochfile))
+
 source_date_epoch = int(d.getVar('SOURCE_DATE_EPOCH_FALLBACK'))
 try:
 with open(epochfile, 'r') as f:
@@ -137,7 +140,7 @@ def get_source_date_epoch_value(d):
 except FileNotFoundError:
 bb.debug(1, "Cannot find %s. SOURCE_DATE_EPOCH will default to %d" % 
(epochfile, source_date_epoch))
 
-d.setVar('__CACHED_SOURCE_DATE_EPOCH', str(source_date_epoch))
+d.setVar('__CACHED_SOURCE_DATE_EPOCH', (str(source_date_epoch), epochfile))
 return str(source_date_epoch)
 
 export SOURCE_DATE_EPOCH ?= "${@get_source_date_epoch_value(d)}"
-- 
2.32.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156585): 
https://lists.openembedded.org/g/openembedded-core/message/156585
Mute This Topic: https://lists.openembedded.org/mt/86065580/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/3 v2] image-artifact-names: Use SOURCE_DATE_EPOCH when making reproducible builds for deploy

2021-10-04 Thread Richard Purdie

Using DATETIME means the do_deploy output is not deterministic. Use SDE
when using reproducible builds since it is consistent for given input data.

Signed-off-by: Richard Purdie 
---
 meta/classes/image-artifact-names.bbclass | 9 -
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/meta/classes/image-artifact-names.bbclass 
b/meta/classes/image-artifact-names.bbclass
index 3ac8dd731a6..67f04e81658 100644
--- a/meta/classes/image-artifact-names.bbclass
+++ b/meta/classes/image-artifact-names.bbclass
@@ -4,7 +4,7 @@
 
 IMAGE_BASENAME ?= "${PN}"
 IMAGE_VERSION_SUFFIX ?= "-${DATETIME}"
-IMAGE_VERSION_SUFFIX[vardepsexclude] += "DATETIME"
+IMAGE_VERSION_SUFFIX[vardepsexclude] += "DATETIME SOURCE_DATE_EPOCH"
 IMAGE_NAME ?= "${IMAGE_BASENAME}-${MACHINE}${IMAGE_VERSION_SUFFIX}"
 IMAGE_LINK_NAME ?= "${IMAGE_BASENAME}-${MACHINE}"
 
@@ -13,3 +13,10 @@ IMAGE_LINK_NAME ?= "${IMAGE_BASENAME}-${MACHINE}"
 # by default) followed by additional suffices which describe the format (.ext4,
 # .ext4.xz, etc.).
 IMAGE_NAME_SUFFIX ??= ".rootfs"
+
+python () {
+if bb.data.inherits_class('reproducible_build', d) and 
bb.data.inherits_class('deploy', d) and d.getVar("IMAGE_VERSION_SUFFIX") == 
"-${DATETIME}":
+import datetime
+d.setVar("IMAGE_VERSION_SUFFIX", "-" + 
datetime.datetime.fromtimestamp(int(d.getVar("SOURCE_DATE_EPOCH")), 
datetime.timezone.utc).strftime('%Y%m%d%H%M%S'))
+d.setVarFlag("IMAGE_VERSION_SUFFIX", "vardepvalue", "")
+}
-- 
2.32.0


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

[OE-core] [PATCH 2/3 v2] abi_version/sstate: Bump HASH_VERSION and SSTATE_VERSION

2021-10-04 Thread Richard Purdie

At this point the hash equivalence and sstate is 'junk' on the autobuilder
unforuntately due to the volume of fixes and also the volume of slighly
not quite right patches tested during the development of the fixes.

In order to try and help any remaining sanity I might have, bump the
version numbers to start with a clean slate so we're working from a known
good baseline rather than risk chasing phantom issues. For those
upgrading, there wouldn't be much reuse anyway after the changes.

Signed-off-by: Richard Purdie 
---
 meta/classes/sstate.bbclass  | 2 +-
 meta/conf/abi_version.conf   | 2 +-
 meta/recipes-devtools/elfutils/elfutils_0.185.bb | 1 -
 meta/recipes-devtools/syslinux/syslinux_6.04-pre2.bb | 1 -
 4 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index 89e9f561787..7f4b1f6804c 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -1,4 +1,4 @@
-SSTATE_VERSION = "3"
+SSTATE_VERSION = "5"
 
 SSTATE_MANIFESTS ?= "${TMPDIR}/sstate-control"
 SSTATE_MANFILEPREFIX = "${SSTATE_MANIFESTS}/manifest-${SSTATE_MANMACH}-${PN}"
diff --git a/meta/conf/abi_version.conf b/meta/conf/abi_version.conf
index 35faef9a368..e84cad1019a 100644
--- a/meta/conf/abi_version.conf
+++ b/meta/conf/abi_version.conf
@@ -12,4 +12,4 @@ OELAYOUT_ABI = "14"
 # a reset of the equivalence, for example when reproducibility issues break the
 # existing match data. Distros can also append to this value for the same 
effect.
 #
-HASHEQUIV_HASH_VERSION  = "5"
+HASHEQUIV_HASH_VERSION  = "8"
diff --git a/meta/recipes-devtools/elfutils/elfutils_0.185.bb 
b/meta/recipes-devtools/elfutils/elfutils_0.185.bb
index 9ea4de8e407..f4769e36328 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.185.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.185.bb
@@ -34,7 +34,6 @@ SRC_URI[sha256sum] = 
"dc8d3e74ab209465e7f568e1b3bb9a5a142f8656e2b57d10049a73da2a
 
 # remove at next version upgrade or when output changes
 PR = "r1"
-HASHEQUIV_HASH_VERSION .= ".2"
 
 inherit autotools gettext ptest pkgconfig
 
diff --git a/meta/recipes-devtools/syslinux/syslinux_6.04-pre2.bb 
b/meta/recipes-devtools/syslinux/syslinux_6.04-pre2.bb
index 0dd18d7a068..62aa1b0e877 100644
--- a/meta/recipes-devtools/syslinux/syslinux_6.04-pre2.bb
+++ b/meta/recipes-devtools/syslinux/syslinux_6.04-pre2.bb
@@ -28,7 +28,6 @@ SRC_URI[sha256sum] = 
"4441a5d593f85bb6e8d578cf6653fb4ec30f9e8f4a2315a3d8f2d0a8b3
 
 # remove at next version upgrade or when output changes
 PR = "r1"
-HASHEQUIV_HASH_VERSION .= ".1"
 
 RECIPE_NO_UPDATE_REASON = "6.04-pre3 is broken"
 UPSTREAM_CHECK_URI = "https://www.zytor.com/pub/syslinux/";
-- 
2.32.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156584): 
https://lists.openembedded.org/g/openembedded-core/message/156584
Mute This Topic: https://lists.openembedded.org/mt/86065578/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] [poky] [PATCH] sstate: Switch to ZStandard compressor support

2021-10-04 Thread Alexander Kanavin

ZSTD_THREADS is already defined in bitbake.conf, you do not need to
redefine it.
(please use abelloni/kirkstone-next for testing, it is there)

Alex

On Mon, 4 Oct 2021 at 13:00, Henry Kleynhans via lists.yoctoproject.org
 wrote:

> This patch switches the compressor from Gzip to ZStandard for ssate cache
> files.
>
> Zstandard compression provides a significant improvement in
> decompression speed as well as improvement in compression speed and disk
> usage over the 'tgz' format in use.  Furthermore, its configurable
> compression level offers a trade-off between time spent compressing
> sstate cache files and disk space used by those files.  The reduced disk
> usage also contributes to saving network traffic for those sharing their
> sstate cache with others.
>
> Zstandard should therefore be a good choice when:
> * disk space is at a premium
> * network speed / resources are limited
> * the CI server can sstate packages can be created at high compression
> * less CPU on the build server should be used for sstate decompression
>
> Signed-off-by: Henry Kleynhans 
> ---
>  meta/classes/sstate.bbclass| 30 ++
>  meta/conf/bitbake.conf |  2 +-
>  scripts/sstate-cache-management.sh | 40 +++---
>  3 files changed, 41 insertions(+), 31 deletions(-)
>
> diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
> index 92a73114bb..9d40c8ebe4 100644
> --- a/meta/classes/sstate.bbclass
> +++ b/meta/classes/sstate.bbclass
> @@ -1,17 +1,20 @@
>  SSTATE_VERSION = "3"
>
> +ZSTD_THREADS ?= "${BB_NUMBER_THREADS}"
> +SSTATE_ZSTD_CLEVEL = "8"
> +
>  SSTATE_MANIFESTS ?= "${TMPDIR}/sstate-control"
>  SSTATE_MANFILEPREFIX =
> "${SSTATE_MANIFESTS}/manifest-${SSTATE_MANMACH}-${PN}"
>
>  def generate_sstatefn(spec, hash, taskname, siginfo, d):
>  if taskname is None:
> return ""
> -extension = ".tgz"
> +extension = ".tar.zst"
>  # 8 chars reserved for siginfo
>  limit = 254 - 8
>  if siginfo:
>  limit = 254
> -extension = ".tgz.siginfo"
> +extension = ".tar.zst.siginfo"
>  if not hash:
>  hash = "INVALID"
>  fn = spec + hash + "_" + taskname + extension
> @@ -37,7 +40,7 @@ SSTATE_PKGNAME=
> "${SSTATE_EXTRAPATH}${@generate_sstatefn(d.getVar('SSTATE_PK
>  SSTATE_PKG= "${SSTATE_DIR}/${SSTATE_PKGNAME}"
>  SSTATE_EXTRAPATH   = ""
>  SSTATE_EXTRAPATHWILDCARD = ""
> -SSTATE_PATHSPEC   =
> "${SSTATE_DIR}/${SSTATE_EXTRAPATHWILDCARD}*/*/${SSTATE_PKGSPEC}*_${SSTATE_PATH_CURRTASK}.tgz*"
> +SSTATE_PATHSPEC   =
> "${SSTATE_DIR}/${SSTATE_EXTRAPATHWILDCARD}*/*/${SSTATE_PKGSPEC}*_${SSTATE_PATH_CURRTASK}.tar.zst*"
>
>  # explicitly make PV to depend on evaluated value of PV variable
>  PV[vardepvalue] = "${PV}"
> @@ -825,23 +828,24 @@ sstate_create_package () {
> mkdir --mode=0775 -p `dirname ${SSTATE_PKG}`
> TFILE=`mktemp ${SSTATE_PKG}.`
>
> -   # Use pigz if available
> -   OPT="-czS"
> -   if [ -x "$(command -v pigz)" ]; then
> -   OPT="-I pigz -cS"
> +   OPT="-cS"
> +   ZSTD="zstd -${SSTATE_ZSTD_CLEVEL} -T${ZSTD_THREADS}"
> +   # Use pzstd if available
> +   if [ -x "$(command -v pzstd)" ]; then
> +   ZSTD="pzstd -${SSTATE_ZSTD_CLEVEL} -p ${ZSTD_THREADS}"
> fi
>
> # Need to handle empty directories
> if [ "$(ls -A)" ]; then
> set +e
> -   tar $OPT -f $TFILE *
> +   tar -I "$ZSTD" $OPT -f $TFILE *
> ret=$?
> if [ $ret -ne 0 ] && [ $ret -ne 1 ]; then
> exit 1
> fi
> set -e
> else
> -   tar $OPT --file=$TFILE --files-from=/dev/null
> +   tar -I "$ZSTD" $OPT --file=$TFILE --files-from=/dev/null
> fi
> chmod 0664 $TFILE
> # Skip if it was already created by some other process
> @@ -880,7 +884,13 @@ python sstate_report_unihash() {
>  # Will be run from within SSTATE_INSTDIR.
>  #
>  sstate_unpack_package () {
> -   tar -xvzf ${SSTATE_PKG}
> +   ZSTD="zstd -T${ZSTD_THREADS}"
> +   # Use pzstd if available
> +   if [ -x "$(command -v pzstd)" ]; then
> +   ZSTD="pzstd -p ${ZSTD_THREADS}"
> +   fi
> +
> +   tar -I "$ZSTD" -xvf ${SSTATE_PKG}
> # update .siginfo atime on local/NFS mirror
> [ -O ${SSTATE_PKG}.siginfo ] && [ -w ${SSTATE_PKG}.siginfo ] && [
> -h ${SSTATE_PKG}.siginfo ] && touch -a ${SSTATE_PKG}.siginfo
> # Use "! -w ||" to return true for read only files
> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
> index e25d632dc1..ed746bdf99 100644
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -498,7 +498,7 @@ HOSTTOOLS += " \
>  HOSTTOOLS += "${@'ip ping ps scp ssh stty' if
> (bb.utils.contains_any('IMAGE_CLASSES', 'testimage testsdk', True, False,
> d) or any(x in (d.getVar("BBINCLUDED") or "") for x in
> ["testimage.

Re: [OE-core] [PATCH RESEND 2/2] glibc: ptest: Add support for running glibc test suite with ptest

2021-10-04 Thread Alexandre Belloni

Hello Lukasz,


On 30/09/2021 12:43:57+0200, ?ukasz Majewski wrote:
> This patch introduces new recipe - namely 'glibc-tests', which
> builds and installs glibc test suite to OE/Yocto built image.
> 
> It reuses code from already available 'glibc-testsuite' recipe,
> which is run with 'bitbake glibc-testsuite -c check' and uses qemu
> to execute remotely (via SSH) tests on some emulated machine.
> 
> This recipe installs eligible tests on some rootfs image.
> Afterwards, either all of them or only time related subset, those
> tests can be executed on the real hardware, to facilitate validation
> of this platform with for example Y2038 problem compliance.
> 
> By default all tests are executed, with 'ptest-runner glibc-tests'.
> To test only time related subset - one needs to call:
> cd /usr/lib/glibc-tests/ptest/; rm run-ptest; \
> ln -s run-ptest-time run-ptest; cd -; ptest-runner glibc-tests
> 
> To facilitate debugging, source files are provided by default with
> the unstripped debugging symbols. Such approach would reduce the
> already complex recipe (as it inherits base glibc one), so there
> is no need to also install *-dbg and *-src packages.
> 

This ends up with:

ERROR: Nothing RPROVIDES 'glibc-tests-ptest' (but 
/home/pokybuild/yocto-worker/qemuarm64-ptest/build/meta/recipes-core/images/core-image-ptest-all.bb
 RDEPENDS on or otherwise requires it)

https://autobuilder.yoctoproject.org/typhoon/#/builders/82/builds/2341/steps/11/logs/stdio

> Signed-off-by: Lukasz Majewski 
> ---
>  .../distro/include/ptest-packagelists.inc |   1 +
>  meta/recipes-core/glibc/glibc-tests_2.34.bb   | 101 ++
>  meta/recipes-core/glibc/glibc/run-ptest-all   |  25 +
>  meta/recipes-core/glibc/glibc/run-ptest-time  |  30 ++
>  4 files changed, 157 insertions(+)
>  create mode 100644 meta/recipes-core/glibc/glibc-tests_2.34.bb
>  create mode 100755 meta/recipes-core/glibc/glibc/run-ptest-all
>  create mode 100755 meta/recipes-core/glibc/glibc/run-ptest-time
> 
> diff --git a/meta/conf/distro/include/ptest-packagelists.inc 
> b/meta/conf/distro/include/ptest-packagelists.inc
> index e0ae1dd971..2031c8730a 100644
> --- a/meta/conf/distro/include/ptest-packagelists.inc
> +++ b/meta/conf/distro/include/ptest-packagelists.inc
> @@ -60,6 +60,7 @@ PTESTS_FAST = "\
>  slang-ptest \
>  wayland-ptest \
>  zlib-ptest \
> +glibc-tests-ptest \
>  "
>  PTESTS_FAST:remove:mips64 = "qemu-ptest"
>  PTESTS_PROBLEMS:append:mips64 = "qemu-ptest"
> diff --git a/meta/recipes-core/glibc/glibc-tests_2.34.bb 
> b/meta/recipes-core/glibc/glibc-tests_2.34.bb
> new file mode 100644
> index 00..896809f700
> --- /dev/null
> +++ b/meta/recipes-core/glibc/glibc-tests_2.34.bb
> @@ -0,0 +1,101 @@
> +require glibc_${PV}.bb
> +require glibc-tests.inc
> +
> +inherit ptest
> +
> +SRC_URI:append = " \
> + file://run-ptest-all file://run-ptest-time \
> +"
> +
> +SUMMARY = "glibc tests to be run with ptest"
> +
> +# Erase some variables already set by glibc_${PV}
> +python __anonymous() {
> +   # Remove packages provided by glibc build, we only need a subset of 
> them
> +   d.setVar("PACKAGES", "${PN}")
> +
> +   d.setVar("PROVIDES", "${PN}")
> +   d.setVar("RPROVIDES", "${PN} glibc-ptest")
> +
> +   d.setVar("RRECOMMENDS", "")
> +}
> +
> +# Remove any lefovers from original glibc recipe
> +RPROVIDES:${PN} = "${PN}"
> +RRECOMMENDS:${PN} = ""
> +RDEPENDS:${PN} = "sed"
> +
> +# Just build tests for target - do not run them
> +do_check:append () {
> + oe_runmake -i check run-built-tests=no
> +}
> +addtask do_check after do_compile before do_install_ptest_base
> +
> +glibc_strip_build_directory () {
> + # Delete all non executable files from build directory
> + find ${B} ! -executable -type f -delete
> +
> + # Remove build dynamic libraries and links to them as
> + # those are already installed in the target device
> + find ${B} -type f -name "*.so" -delete
> + find ${B} -type l -name "*.so*" -delete
> +
> + # Remove headers (installed with glibc)
> + find ${B} -type f -name "*.h" -delete
> +
> + find ${B} -type f -name "isomac" -delete
> + find ${B} -type f -name "annexc" -delete
> +}
> +
> +do_install_ptest_base () {
> + glibc_strip_build_directory
> +
> + # Install build test programs to the image
> + install -d ${D}${PTEST_PATH}/tests/glibc-ptest/
> + cp -r ${B}/* ${D}${PTEST_PATH}/tests/glibc-ptest/
> +
> + install -d ${D}${PTEST_PATH}
> + cp ${WORKDIR}/run-ptest-* ${D}${PTEST_PATH}/
> +
> + # By default run all glibc tests
> + cd ${D}${PTEST_PATH}/
> + ln -s run-ptest-all run-ptest
> +}
> +
> +# The datadir directory is required to allow core (and reused)
> +# glibc cleanup function to finish correctly, as this directory
> +# is not created for ptests
> +stash_locale_package_cleanup:prepend () {
> + mkdir -p ${PKGD}${datadir}
> +}
> +
> +stash_locale_sysroot_cleanup:prepend () {
> + mkdir -p ${SY

[OE-core][PATCH] busybox: 1.34.0 -> 1.34.1

2021-10-04 Thread Andrej Valek

- update to next stable version 1.34.1

Signed-off-by: Andrej Valek 
---
 .../{busybox-inittab_1.34.0.bb => busybox-inittab_1.34.1.bb}| 0
 .../busybox/{busybox_1.34.0.bb => busybox_1.34.1.bb}| 2 +-
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename meta/recipes-core/busybox/{busybox-inittab_1.34.0.bb => 
busybox-inittab_1.34.1.bb} (100%)
 rename meta/recipes-core/busybox/{busybox_1.34.0.bb => busybox_1.34.1.bb} (95%)

diff --git a/meta/recipes-core/busybox/busybox-inittab_1.34.0.bb 
b/meta/recipes-core/busybox/busybox-inittab_1.34.1.bb
similarity index 100%
rename from meta/recipes-core/busybox/busybox-inittab_1.34.0.bb
rename to meta/recipes-core/busybox/busybox-inittab_1.34.1.bb
diff --git a/meta/recipes-core/busybox/busybox_1.34.0.bb 
b/meta/recipes-core/busybox/busybox_1.34.1.bb
similarity index 95%
rename from meta/recipes-core/busybox/busybox_1.34.0.bb
rename to meta/recipes-core/busybox/busybox_1.34.1.bb
index 51df1df05f..6aed0f0476 100644
--- a/meta/recipes-core/busybox/busybox_1.34.0.bb
+++ b/meta/recipes-core/busybox/busybox_1.34.1.bb
@@ -51,4 +51,4 @@ SRC_URI = 
"https://busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
"
 SRC_URI:append:libc-musl = " file://musl.cfg "
 
-SRC_URI[tarball.sha256sum] = 
"ec8d1615edb045b83b81966604759c4d4ac921434ab4011da604f629c06074ce"
+SRC_URI[tarball.sha256sum] = 
"415fbd89e5344c96acf449d94a6f956dbed62e18e835fc83e064db33a34bd549"
-- 
2.31.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156580): 
https://lists.openembedded.org/g/openembedded-core/message/156580
Mute This Topic: https://lists.openembedded.org/mt/86062952/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] inetutils: update 2.1 -> 2.2

2021-10-04 Thread Alexander Kanavin

From: Alexander Kanavin 

Signed-off-by: Alexander Kanavin 
---
 .../inetutils/inetutils/CVE-2021-40491.patch  | 88 ---
 ...ly-check-pam_appl.h-when-pam-enabled.patch | 12 +--
 .../{inetutils_2.1.bb => inetutils_2.2.bb}|  3 +-
 3 files changed, 7 insertions(+), 96 deletions(-)
 delete mode 100644 
meta/recipes-connectivity/inetutils/inetutils/CVE-2021-40491.patch
 rename meta/recipes-connectivity/inetutils/{inetutils_2.1.bb => 
inetutils_2.2.bb} (98%)

diff --git a/meta/recipes-connectivity/inetutils/inetutils/CVE-2021-40491.patch 
b/meta/recipes-connectivity/inetutils/inetutils/CVE-2021-40491.patch
deleted file mode 100644
index 202488f75c..00
--- a/meta/recipes-connectivity/inetutils/inetutils/CVE-2021-40491.patch
+++ /dev/null
@@ -1,88 +0,0 @@
-From 98ccabf68e5b3f0a177bd1925581753d10041448 Mon Sep 17 00:00:00 2001
-From: Simon Josefsson 
-Date: Wed, 1 Sep 2021 09:09:50 +0200
-Subject: [PATCH] ftp: check that PASV/LSPV addresses match.
-
-* NEWS: Mention change.
-* ftp/ftp.c (initconn): Validate returned addresses.
-
-CVE: CVE-2021-40491
-
-Upstream-Status: Backport
-[https://git.savannah.gnu.org/cgit/inetutils.git/commit/?id=58cb043b190fd04effdaea7c9403416b436e50dd]
-
-Signed-off-by: Yi Zhao 

- NEWS  |  9 +
- ftp/ftp.c | 21 +
- 2 files changed, 30 insertions(+)
-
-diff --git a/NEWS b/NEWS
-index 7c5e62c..bd9a4da 100644
 a/NEWS
-+++ b/NEWS
-@@ -4,6 +4,15 @@ GNU inetutils NEWS -- history of user-visible changes.
- 
- ** ftp
- 
-+The ftp client now validate addresses returned by PASV/LSPV responses,
-+to make sure they match the server address.  Reported by ZeddYu Lu in
-+.
-+
-+Thanks to Luke Mewburn  for discussion and fix to
-+NetBSD code, we used a similar solution.
-+
-+** ftp
-+
- Disable use of readline when environment variable TERM is unset or set
- to "dumb" (caused problems with Emacs AngeFTP on MacOS).  Thanks to
- Alex Bochannek for report, debugging and patch.
-diff --git a/ftp/ftp.c b/ftp/ftp.c
-index d21dbdd..7513539 100644
 a/ftp/ftp.c
-+++ b/ftp/ftp.c
-@@ -1365,6 +1365,13 @@ initconn (void)
- uint32_t *pu32 = (uint32_t *) &data_addr_sa4->sin_addr.s_addr;
- pu32[0] = htonl ( (h[0] << 24) | (h[1] << 16) | (h[2] << 8) | 
h[3]);
-   }
-+  if (data_addr_sa4->sin_addr.s_addr
-+  != ((struct sockaddr_in *) &hisctladdr)->sin_addr.s_addr)
-+{
-+  printf ("Passive mode address mismatch.\n");
-+  (void) command ("ABOR");/* Cancel any open connection.  
*/
-+  goto bad;
-+}
-   } /* LPSV IPv4 */
- else /* IPv6 */
-   {
-@@ -1395,6 +1402,13 @@ initconn (void)
- pu32[2] = htonl ( (h[8] << 24) | (h[9] << 16) | (h[10] << 8) 
| h[11]);
- pu32[3] = htonl ( (h[12] << 24) | (h[13] << 16) | (h[14] << 
8) | h[15]);
-   }
-+  if (data_addr_sa6->sin6_addr.s6_addr
-+  != ((struct sockaddr_in6 *) &hisctladdr)->sin6_addr.s6_addr)
-+{
-+  printf ("Passive mode address mismatch.\n");
-+  (void) command ("ABOR");/* Cancel any open connection.  
*/
-+  goto bad;
-+}
-   } /* LPSV IPv6 */
-   }
-   else /* !EPSV && !LPSV */
-@@ -1415,6 +1429,13 @@ initconn (void)
-| ((a2 & 0xff) << 8) | (a3 & 0xff) );
- data_addr_sa4->sin_port =
- htons (((p0 & 0xff) << 8) | (p1 & 0xff));
-+if (data_addr_sa4->sin_addr.s_addr
-+!= ((struct sockaddr_in *) &hisctladdr)->sin_addr.s_addr)
-+  {
-+printf ("Passive mode address mismatch.\n");
-+(void) command ("ABOR");  /* Cancel any open connection.  
*/
-+goto bad;
-+  }
-   } /* PASV */
- else
-   {
--- 
-2.17.1
-
diff --git 
a/meta/recipes-connectivity/inetutils/inetutils/inetutils-only-check-pam_appl.h-when-pam-enabled.patch
 
b/meta/recipes-connectivity/inetutils/inetutils/inetutils-only-check-pam_appl.h-when-pam-enabled.patch
index 7d5c087c9d..2343c03cb4 100644
--- 
a/meta/recipes-connectivity/inetutils/inetutils/inetutils-only-check-pam_appl.h-when-pam-enabled.patch
+++ 
b/meta/recipes-connectivity/inetutils/inetutils/inetutils-only-check-pam_appl.h-when-pam-enabled.patch
@@ -1,4 +1,4 @@
-From 684e45b34a33186bb17bcee0b01814c549a60bf6 Mon Sep 17 00:00:00 2001
+From cc66e842e037fba9f06761f942abe5c4856492b8 Mon Sep 17 00:00:00 2001
 From: Kai Kang 
 Date: Wed, 6 Mar 2019 09:36:11 -0500
 Subject: [PATCH] inetutils: Import version 1.9.4
@@ -15,10 +15,10 @@ Signed-off-by: Kai Kang 
  1 file changed, 14 insertions(+), 1 deletion(-)
 
 diff --git a/configure.ac b/configure.ac
-index 86136fb..b220319 100644
+index 5e16c3a..18510a8

[OE-core] [PATCH] package_rpm: use zstd instead of xz

2021-10-04 Thread Alexander Kanavin

zstd has similar time and space performance in compression but is
vastly faster in decompression, which benefits rootfs creation
(especially when installing very large packages) and on-target
package installation.

Also, ensure ZSTD_THREADS doesn't change sstate checksums. The
detailed explanation is in the commit making similar change for
XZ_THREADS.

Signed-off-by: Alexander Kanavin 
---
 meta/classes/package_rpm.bbclass | 4 ++--
 meta/conf/bitbake.conf   | 4 
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/meta/classes/package_rpm.bbclass b/meta/classes/package_rpm.bbclass
index 1c62d8283b..e738806fd7 100644
--- a/meta/classes/package_rpm.bbclass
+++ b/meta/classes/package_rpm.bbclass
@@ -684,8 +684,8 @@ python do_package_rpm () {
 cmd = cmd + " --define '_use_internal_dependency_generator 0'"
 cmd = cmd + " --define '_binaries_in_noarch_packages_terminate_build 0'"
 cmd = cmd + " --define '_build_id_links none'"
-cmd = cmd + " --define '_binary_payload w6T%d.xzdio'" % 
int(d.getVar("XZ_THREADS"))
-cmd = cmd + " --define '_source_payload w6T%d.xzdio'" % 
int(d.getVar("XZ_THREADS"))
+cmd = cmd + " --define '_binary_payload w19T%d.zstdio'" % 
int(d.getVar("ZSTD_THREADS"))
+cmd = cmd + " --define '_source_payload w19T%d.zstdio'" % 
int(d.getVar("ZSTD_THREADS"))
 cmd = cmd + " --define 'clamp_mtime_to_source_date_epoch 1'"
 cmd = cmd + " --define 'use_source_date_epoch_as_buildtime 1'"
 cmd = cmd + " --define '_buildhost reproducible'"
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index e25d632dc1..1ae377750b 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -815,6 +815,10 @@ XZ_THREADS[vardepvalue] = "1"
 XZ_DEFAULTS ?= "--memlimit=${XZ_MEMLIMIT} --threads=${XZ_THREADS}"
 XZ_DEFAULTS[vardepsexclude] += "XZ_MEMLIMIT XZ_THREADS"
 
+# Default parallelism for zstd
+ZSTD_THREADS ?= "${@oe.utils.cpu_count(at_least=2)}"
+ZSTD_THREADS[vardepvalue] = "1"
+
 # Limit the number of threads that OpenMP libraries will use. Otherwise they
 # may fallback to using all CPUs
 export OMP_NUM_THREADS = "${BB_NUMBER_THREADS}"
-- 
2.31.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156578): 
https://lists.openembedded.org/g/openembedded-core/message/156578
Mute This Topic: https://lists.openembedded.org/mt/86062347/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] bitbake.conf: Set vardepvalue for PARALLEL_MAKEINST

2021-10-04 Thread Richard Purdie

If you leave PARALLEL_MAKEINST at its default from PARALLEL_MAKE,
sstate checksums are fine and don't change as the number of cores do.
If you override it to a specific value, this does the cause the sstate
checksums to change. We don't want the checksums to change if the
value of this variable differs.

Therefore set a vardepvalue so a specific value is used for checksum
purposes.

Signed-off-by: Richard Purdie 
---
 meta/conf/bitbake.conf | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index a207cbb33d2..1239206645d 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -604,6 +604,7 @@ LINKER_HASH_STYLE:mipsarch:libc-musl = "sysv"
 # Pass parallel make options to the compile task
 EXTRA_OEMAKE:prepend:task-compile = "${PARALLEL_MAKE} "
 PARALLEL_MAKEINST ??= "${PARALLEL_MAKE}"
+PARALLEL_MAKEINST[vardepvalue] = "1"
 # Pass parallel make options to the install task
 EXTRA_OEMAKE:prepend:task-install = "${PARALLEL_MAKEINST} "
 
-- 
2.32.0


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

60 matches

Mail list logo