[OE-core] [PATCH 0/1] uboot-sign.bbclass: fix signature and deployment
The following changes since commit 19c9842644b7b0a599437fd394111f161c8b1f1e: apt: Improve SRC_URI (2018-11-20 10:11:24 +) are available in the git repository at: git://git.openembedded.org/openembedded-core-contrib rbt/uboot http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/uboot Robert Yang (1): uboot-sign.bbclass: fix signature and deployment meta/classes/kernel-fitimage.bbclass | 17 ++- meta/classes/uboot-sign.bbclass | 95 meta/recipes-bsp/u-boot/u-boot.inc | 2 +- 3 files changed, 69 insertions(+), 45 deletions(-) -- 2.7.4 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 1/1] uboot-sign.bbclass: fix signature and deployment
Fixed: MACHINE = "beaglebone-yocto" KERNEL_CLASSES += "kernel-fitimage" KERNEL_IMAGETYPE_beaglebone-yocto = "fitImage" UBOOT_MACHINE_beaglebone-yocto = "am335x_boneblack_vboot_config" UBOOT_MKIMAGE_DTCOPTS = "-I dts -O dtb -p 2000" UBOOT_SIGN_KEYDIR = "${TOPDIR}/conf" UBOOT_SIGN_KEYNAME = "dev" UBOOT_SIGN_ENABLE = "1" IMAGE_INSTALL_remove = "kernel-image-zimage" $ cd conf $ openssl genrsa -F4 -out dev.key 2048 $ openssl req -batch -new -x509 -key dev.key -out dev.crt $ cd ../ $ bitbake u-boot linux-yocto $ grep signature tmp/deploy/images/beaglebone-yocto/*.dtb Binary file tmp/deploy/images/beaglebone-yocto/u-boot-beaglebone-yocto-2018.07-r0.dtb matches Binary file tmp/deploy/images/beaglebone-yocto/u-boot-beaglebone-yocto.dtb matches Binary file tmp/deploy/images/beaglebone-yocto/u-boot.dtb matches And there would be no signature info when rebuild from sstate: $ bitbake u-boot linux-yocto -cclean $ bitbake u-boot linux-yocto $ grep signature tmp/deploy/images/beaglebone-yocto/*.dtb No result This s because kernel directly edit ${DEPLOY_DIR_IMAGE}/u-boot.dtb, (Note, it is global ${DEPLOY_DIR_IMAGE}, not recipe's DEPLOYDIR), so that the modified info is not in sstate, and would be lost when rebuild from sstate. There are other problems in previouse code: - The u-boot.dtb is provided by u-boot, but edited by kernel during signing, so it should be deployed by kernel rather than u-boot. - The u-boot.do_concat_dtb directly install files to global ${DEPLOY_DIR_IMAGE}, this is incorrect, the ${DEPLOY_DIR_IMAGE} should be installed by do_deploy. - It seems that it assumes do_deploy depends on do_install according the comments, but they have no relationships: # do_concat_dtb is scheduled _before_ do_install as it overwrite the # u-boot.bin in both DEPLOYDIR and DEPLOY_IMAGE_DIR. - The do_concat_dtb should be run after do_compile, but it doesn't have this dependency. Make u-boot install u-boot.dtb to ${datadir}, kernel copies u-boot.dtb from ${STAGING_DATADIR} to ${B} and deploy it can fix the problem. [YOCTO #12112] Reported-by: Christian Andersen Signed-off-by: Robert Yang --- meta/classes/kernel-fitimage.bbclass | 17 ++- meta/classes/uboot-sign.bbclass | 95 meta/recipes-bsp/u-boot/u-boot.inc | 2 +- 3 files changed, 69 insertions(+), 45 deletions(-) diff --git a/meta/classes/kernel-fitimage.bbclass b/meta/classes/kernel-fitimage.bbclass index 4c4fd99..7c5bcd0 100644 --- a/meta/classes/kernel-fitimage.bbclass +++ b/meta/classes/kernel-fitimage.bbclass @@ -35,7 +35,7 @@ python __anonymous () { # the fitImage: if d.getVar('UBOOT_SIGN_ENABLE') == "1": uboot_pn = d.getVar('PREFERRED_PROVIDER_u-boot') or 'u-boot' -d.appendVarFlag('do_assemble_fitimage', 'depends', ' %s:do_deploy' % uboot_pn) +d.appendVarFlag('do_assemble_fitimage', 'depends', ' %s:do_populate_sysroot' % uboot_pn) } # Options for the device tree compiler passed to mkimage '-D' feature: @@ -456,10 +456,17 @@ fitimage_assemble() { # Step 7: Sign the image and add public key to U-Boot dtb # if [ "x${UBOOT_SIGN_ENABLE}" = "x1" ] ; then + add_key_to_u_boot="" + if [ -n "${UBOOT_DTB_BINARY}" ]; then + # The u-boot.dtb is a symlink to UBOOT_DTB_IMAGE, so we need copy + # both of them, and don't dereference the symlink. + cp -P ${STAGING_DATADIR}/u-boot*.dtb ${B} + add_key_to_u_boot="-K ${B}/${UBOOT_DTB_BINARY}" + fi uboot-mkimage \ ${@'-D "${UBOOT_MKIMAGE_DTCOPTS}"' if len('${UBOOT_MKIMAGE_DTCOPTS}') else ''} \ -F -k "${UBOOT_SIGN_KEYDIR}" \ - ${@'-K "${DEPLOY_DIR_IMAGE}/${UBOOT_DTB_BINARY}"' if len('${UBOOT_DTB_BINARY}') else ''} \ + $add_key_to_u_boot \ -r arch/${ARCH}/boot/${2} fi } @@ -505,5 +512,11 @@ kernel_do_deploy_append() { install -m 0644 ${B}/arch/${ARCH}/boot/fitImage-${INITRAMFS_IMAGE} ${DEPLOYDIR}/fitImage-${INITRAMFS_IMAGE_NAME}-${KERNEL_FIT_NAME}.bin ln -snf fitImage-${INITRAMFS_IMAGE_NAME}-${KERNEL_FIT_NAME}.bin ${DEPLOYDIR}/fitImage-${INITRAMFS_IMAGE_NAME}-${KERNEL_FIT_LINK_NAME} fi + if [ "${UBOOT_SIGN_ENABLE}" = "1" -a -n "${UBOOT_DTB_BINARY}" ] ; then + # UBOOT_DTB_IMAGE is a realfile, but we can't use + # ${UBOOT_DTB_IMAGE} since it contains ${PV} which is aimed + # for u-boot, but we are in kernel env now. + install -m 0644 ${B}/u-boot-${MACHINE}*.dtb ${DEPLOYDIR}/ + fi fi } diff --git a/meta/classes/uboot-sign.bbclass b/meta/classes/uboot-sign.bbclass index 8ee904e..0e5e1b1 100644 --- a/meta/classes/uboot-sign.
[OE-core] [PATCH 8/8] busybox: check CONFIG_DESKTOP before using 'od -t' in test case
The '-t' option support for 'od' is enabled by CONFIG_DESKTOP. So check it before using it in test cases. Signed-off-by: Chen Qi --- ...te-check-CONFIG_DESKTOP-before-using-od-t.patch | 83 ++ meta/recipes-core/busybox/busybox_1.29.2.bb| 1 + 2 files changed, 84 insertions(+) create mode 100644 meta/recipes-core/busybox/busybox/0001-testsuite-check-CONFIG_DESKTOP-before-using-od-t.patch diff --git a/meta/recipes-core/busybox/busybox/0001-testsuite-check-CONFIG_DESKTOP-before-using-od-t.patch b/meta/recipes-core/busybox/busybox/0001-testsuite-check-CONFIG_DESKTOP-before-using-od-t.patch new file mode 100644 index 000..9b96eee --- /dev/null +++ b/meta/recipes-core/busybox/busybox/0001-testsuite-check-CONFIG_DESKTOP-before-using-od-t.patch @@ -0,0 +1,83 @@ +From b09db631e77c63ebe733c69b00108f043f1f1d85 Mon Sep 17 00:00:00 2001 +From: Chen Qi +Date: Wed, 21 Nov 2018 10:00:29 +0800 +Subject: [PATCH] testsuite: check CONFIG_DESKTOP before using 'od -t' + +The '-t' option for od is enabled by CONFIG_DESKTOP. So +check this config before using 'od -t' in test cases. + +Upstream-Status: Submitted [http://lists.busybox.net/pipermail/busybox/2018-November/086895.html] + +Signed-off-by: Chen Qi +--- + testsuite/echo/echo-prints-dash| 1 + + testsuite/echo/echo-prints-non-opts| 1 + + testsuite/echo/echo-prints-slash-zero | 2 +- + testsuite/echo/echo-prints-slash_00041 | 2 +- + testsuite/echo/echo-prints-slash_0041 | 2 +- + testsuite/echo/echo-prints-slash_041 | 2 +- + testsuite/echo/echo-prints-slash_41| 2 +- + 7 files changed, 7 insertions(+), 5 deletions(-) + +diff --git a/testsuite/echo/echo-prints-dash b/testsuite/echo/echo-prints-dash +index ddcdbad..f1f31a0 100644 +--- a/testsuite/echo/echo-prints-dash b/testsuite/echo/echo-prints-dash +@@ -1 +1,2 @@ ++# FEATURE: CONFIG_DESKTOP + test "`busybox echo - | od -t x1 | head -n 1`" = "000 2d 0a" +diff --git a/testsuite/echo/echo-prints-non-opts b/testsuite/echo/echo-prints-non-opts +index c7d1e20..4e642a1 100644 +--- a/testsuite/echo/echo-prints-non-opts b/testsuite/echo/echo-prints-non-opts +@@ -1 +1,2 @@ ++# FEATURE: CONFIG_DESKTOP + test "`busybox echo -neEZ | od -t x1 | head -n 1`" = "000 2d 6e 65 45 5a 0a" +diff --git a/testsuite/echo/echo-prints-slash-zero b/testsuite/echo/echo-prints-slash-zero +index d97ed8e..948f899 100644 +--- a/testsuite/echo/echo-prints-slash-zero b/testsuite/echo/echo-prints-slash-zero +@@ -1,3 +1,3 @@ +-# FEATURE: CONFIG_FEATURE_FANCY_ECHO ++# FEATURE: CONFIG_FEATURE_FANCY_ECHO CONFIG_DESKTOP + + test "`busybox echo -e -n 'msg\n\0' | od -t x1 | head -n 1`" = "000 6d 73 67 0a 00" +diff --git a/testsuite/echo/echo-prints-slash_00041 b/testsuite/echo/echo-prints-slash_00041 +index 9cffabd..7ea217b 100644 +--- a/testsuite/echo/echo-prints-slash_00041 b/testsuite/echo/echo-prints-slash_00041 +@@ -1,3 +1,3 @@ +-# FEATURE: CONFIG_FEATURE_FANCY_ECHO ++# FEATURE: CONFIG_FEATURE_FANCY_ECHO CONFIG_DESKTOP + + test "`busybox echo -ne '\00041z' | od -t x1 | head -n 1`" = "000 04 31 7a" +diff --git a/testsuite/echo/echo-prints-slash_0041 b/testsuite/echo/echo-prints-slash_0041 +index b07429d..941faca 100644 +--- a/testsuite/echo/echo-prints-slash_0041 b/testsuite/echo/echo-prints-slash_0041 +@@ -1,3 +1,3 @@ +-# FEATURE: CONFIG_FEATURE_FANCY_ECHO ++# FEATURE: CONFIG_FEATURE_FANCY_ECHO CONFIG_DESKTOP + + test "`busybox echo -ne '\0041z' | od -t x1 | head -n 1`" = "000 21 7a" +diff --git a/testsuite/echo/echo-prints-slash_041 b/testsuite/echo/echo-prints-slash_041 +index 1d70cec..60cbfca 100644 +--- a/testsuite/echo/echo-prints-slash_041 b/testsuite/echo/echo-prints-slash_041 +@@ -1,3 +1,3 @@ +-# FEATURE: CONFIG_FEATURE_FANCY_ECHO ++# FEATURE: CONFIG_FEATURE_FANCY_ECHO CONFIG_DESKTOP + + test "`busybox echo -ne '\041z' | od -t x1 | head -n 1`" = "000 21 7a" +diff --git a/testsuite/echo/echo-prints-slash_41 b/testsuite/echo/echo-prints-slash_41 +index 6d8999b..afd7c36 100644 +--- a/testsuite/echo/echo-prints-slash_41 b/testsuite/echo/echo-prints-slash_41 +@@ -1,3 +1,3 @@ +-# FEATURE: CONFIG_FEATURE_FANCY_ECHO ++# FEATURE: CONFIG_FEATURE_FANCY_ECHO CONFIG_DESKTOP + + test "`busybox echo -ne '\41z' | od -t x1 | head -n 1`" = "000 21 7a" +-- +2.7.4 + diff --git a/meta/recipes-core/busybox/busybox_1.29.2.bb b/meta/recipes-core/busybox/busybox_1.29.2.bb index f38db2a..66049bd 100644 --- a/meta/recipes-core/busybox/busybox_1.29.2.bb +++ b/meta/recipes-core/busybox/busybox_1.29.2.bb @@ -44,6 +44,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \ file://0001-testsuite-check-uudecode-before-using-it.patch \ file://0001-testsuite-use-www.example.org-for-wget-test-cases.patch \ file://0001-du-l-works-fix-to-use-145-instead-of-144.patch \ + file://0001-testsuite-check-CONFIG_DESKTOP-before-using-od-t.patch \ " SRC_URI_append_libc-musl = " file
[OE-core] [PATCH 5/8] busybox: use example.org instead of google.com in wget test case
Use example.org to ensure it's always reachible. Signed-off-by: Chen Qi --- ...e-use-www.example.org-for-wget-test-cases.patch | 62 ++ meta/recipes-core/busybox/busybox_1.29.2.bb| 1 + 2 files changed, 63 insertions(+) create mode 100644 meta/recipes-core/busybox/busybox/0001-testsuite-use-www.example.org-for-wget-test-cases.patch diff --git a/meta/recipes-core/busybox/busybox/0001-testsuite-use-www.example.org-for-wget-test-cases.patch b/meta/recipes-core/busybox/busybox/0001-testsuite-use-www.example.org-for-wget-test-cases.patch new file mode 100644 index 000..7ba0f77 --- /dev/null +++ b/meta/recipes-core/busybox/busybox/0001-testsuite-use-www.example.org-for-wget-test-cases.patch @@ -0,0 +1,62 @@ +From 03acf460990dfcf6e52b298301b348b9504460a8 Mon Sep 17 00:00:00 2001 +From: Chen Qi +Date: Tue, 13 Nov 2018 13:51:42 +0800 +Subject: [PATCH] testsuite: use www.example.org for wget test cases + +Use example.org instead of google.com to make sure it's +always reachible. + +Upstream-Status: Submitted [http://lists.busybox.net/pipermail/busybox/2018-November/086835.html] + +Signed-off-by: Chen Qi +--- + testsuite/wget/wget--O-overrides--P| 2 +- + testsuite/wget/wget-handles-empty-path | 2 +- + testsuite/wget/wget-retrieves-google-index | 2 +- + testsuite/wget/wget-supports--P| 2 +- + 4 files changed, 4 insertions(+), 4 deletions(-) + +diff --git a/testsuite/wget/wget--O-overrides--P b/testsuite/wget/wget--O-overrides--P +index 40a3a96..490d743 100644 +--- a/testsuite/wget/wget--O-overrides--P b/testsuite/wget/wget--O-overrides--P +@@ -1,5 +1,5 @@ + test x"$SKIP_INTERNET_TESTS" != x"" && exit + + mkdir foo +-busybox wget -q -O index.html -P foo http://www.google.com/ ++busybox wget -q -O index.html -P foo http://www.example.org/ + test -s index.html +diff --git a/testsuite/wget/wget-handles-empty-path b/testsuite/wget/wget-handles-empty-path +index 01d60bd..1802db6 100644 +--- a/testsuite/wget/wget-handles-empty-path b/testsuite/wget/wget-handles-empty-path +@@ -1,3 +1,3 @@ + test x"$SKIP_INTERNET_TESTS" != x"" && exit + +-busybox wget http://www.google.com ++busybox wget http://www.example.org +diff --git a/testsuite/wget/wget-retrieves-google-index b/testsuite/wget/wget-retrieves-google-index +index f9dbb8b..fc801d4 100644 +--- a/testsuite/wget/wget-retrieves-google-index b/testsuite/wget/wget-retrieves-google-index +@@ -1,4 +1,4 @@ + test x"$SKIP_INTERNET_TESTS" != x"" && exit + +-busybox wget -q -O foo http://www.google.com/ ++busybox wget -q -O foo http://www.example.org/ + test -s foo +diff --git a/testsuite/wget/wget-supports--P b/testsuite/wget/wget-supports--P +index bfe4ac4..4ff2fe0 100644 +--- a/testsuite/wget/wget-supports--P b/testsuite/wget/wget-supports--P +@@ -1,5 +1,5 @@ + test x"$SKIP_INTERNET_TESTS" != x"" && exit + + mkdir foo +-busybox wget -q -P foo http://www.google.com/ ++busybox wget -q -P foo http://www.example.org/ + test -s foo/index.html +-- +2.7.4 + diff --git a/meta/recipes-core/busybox/busybox_1.29.2.bb b/meta/recipes-core/busybox/busybox_1.29.2.bb index a63fa82..98767f0 100644 --- a/meta/recipes-core/busybox/busybox_1.29.2.bb +++ b/meta/recipes-core/busybox/busybox_1.29.2.bb @@ -42,6 +42,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \ file://rcK \ file://makefile-libbb-race.patch \ file://0001-testsuite-check-uudecode-before-using-it.patch \ + file://0001-testsuite-use-www.example.org-for-wget-test-cases.patch \ " SRC_URI_append_libc-musl = " file://musl.cfg " -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 7/8] busybox: fix du-l-works test case
64 + 64 + 16 = 144K 144 + sizeof_a_directory >= 145 So fix to use 145 instead of 144. Signed-off-by: Chen Qi --- ...-du-l-works-fix-to-use-145-instead-of-144.patch | 32 ++ meta/recipes-core/busybox/busybox_1.29.2.bb| 1 + 2 files changed, 33 insertions(+) create mode 100644 meta/recipes-core/busybox/busybox/0001-du-l-works-fix-to-use-145-instead-of-144.patch diff --git a/meta/recipes-core/busybox/busybox/0001-du-l-works-fix-to-use-145-instead-of-144.patch b/meta/recipes-core/busybox/busybox/0001-du-l-works-fix-to-use-145-instead-of-144.patch new file mode 100644 index 000..bb5f753 --- /dev/null +++ b/meta/recipes-core/busybox/busybox/0001-du-l-works-fix-to-use-145-instead-of-144.patch @@ -0,0 +1,32 @@ +From c69feb2fdb755d6db34d0eb0a9e9994386c15e3a Mon Sep 17 00:00:00 2001 +From: Chen Qi +Date: Wed, 14 Nov 2018 15:19:57 +0800 +Subject: [PATCH] du-l-works: fix to use 145 instead of 144 + +64 + 64 + 16 = 144 +144 + size_of_directory >= 145 +So fix to use 145 instead of 144. + +Upstream-Status: Submitted [http://lists.busybox.net/pipermail/busybox/2018-November/086836.html] + +Signed-off-by: Chen Qi +--- + testsuite/du/du-l-works | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/testsuite/du/du-l-works b/testsuite/du/du-l-works +index af87345..575064a 100644 +--- a/testsuite/du/du-l-works b/testsuite/du/du-l-works +@@ -5,7 +5,7 @@ cd du.testdir + dd if=/dev/zero of=file1 bs=1k count=64 2>/dev/null + ln file1 file1.1 + dd if=/dev/zero of=file2 bs=1k count=16 2>/dev/null +-test x"`busybox du -l .`" = x"144 ." \ ++test x"`busybox du -l .`" = x"145 ." \ + -o x"`busybox du -l .`" = x"146 ." \ + -o x"`busybox du -l .`" = x"148 ." \ + -o x"`busybox du -l .`" = x"152 ." \ +-- +2.7.4 + diff --git a/meta/recipes-core/busybox/busybox_1.29.2.bb b/meta/recipes-core/busybox/busybox_1.29.2.bb index 98767f0..f38db2a 100644 --- a/meta/recipes-core/busybox/busybox_1.29.2.bb +++ b/meta/recipes-core/busybox/busybox_1.29.2.bb @@ -43,6 +43,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \ file://makefile-libbb-race.patch \ file://0001-testsuite-check-uudecode-before-using-it.patch \ file://0001-testsuite-use-www.example.org-for-wget-test-cases.patch \ + file://0001-du-l-works-fix-to-use-145-instead-of-144.patch \ " SRC_URI_append_libc-musl = " file://musl.cfg " -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 1/8] busybox: enable bzip2 by default
bunzip2 is enabled by default, but bzip2 is not. This is kind of strange, and it also causes busybox's ptest failure regarding bunzip2, as bunzip2's test case needs bzip2 command. Signed-off-by: Chen Qi --- meta/recipes-core/busybox/busybox/defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-core/busybox/busybox/defconfig b/meta/recipes-core/busybox/busybox/defconfig index 32213c0..0c40990 100644 --- a/meta/recipes-core/busybox/busybox/defconfig +++ b/meta/recipes-core/busybox/busybox/defconfig @@ -150,7 +150,7 @@ CONFIG_LZCAT=y # CONFIG_UNXZ is not set CONFIG_XZCAT=y # CONFIG_XZ is not set -# CONFIG_BZIP2 is not set +CONFIG_BZIP2=y CONFIG_FEATURE_BZIP2_DECOMPRESS=y CONFIG_CPIO=y # CONFIG_FEATURE_CPIO_O is not set -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 4/8] busybox: check uudecode before using it in test case
Check uudecode before using it in test case to avoid unexpected failure. Signed-off-by: Chen Qi --- ...-testsuite-check-uudecode-before-using-it.patch | 45 ++ meta/recipes-core/busybox/busybox_1.29.2.bb| 1 + 2 files changed, 46 insertions(+) create mode 100644 meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch diff --git a/meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch b/meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch new file mode 100644 index 000..2d9856f --- /dev/null +++ b/meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch @@ -0,0 +1,45 @@ +From 85f164ec427fecbae24140b3c93acbbe2ba1c817 Mon Sep 17 00:00:00 2001 +From: Chen Qi +Date: Tue, 13 Nov 2018 11:27:01 +0800 +Subject: [PATCH] testsuite: check uudecode before using it + +The unzip and tar test cases use uudecode command, so also +check it via optional function. + +Upstream-Status: Submitted [http://lists.busybox.net/pipermail/busybox/2018-November/086833.html] + +Signed-off-by: Chen Qi +--- + testsuite/tar.tests | 2 +- + testsuite/unzip.tests | 2 +- + 2 files changed, 2 insertions(+), 2 deletions(-) + +diff --git a/testsuite/tar.tests b/testsuite/tar.tests +index d71a349..8c88567 100755 +--- a/testsuite/tar.tests b/testsuite/tar.tests +@@ -336,7 +336,7 @@ SKIP= + cd .. || exit 1; rm -rf tar.tempdir 2>/dev/null + + mkdir tar.tempdir && cd tar.tempdir || exit 1 +-optional FEATURE_SEAMLESS_BZ2 FEATURE_TAR_AUTODETECT LS ++optional FEATURE_SEAMLESS_BZ2 FEATURE_TAR_AUTODETECT LS UUDECODE + testing "Symlink attack: create symlink and then write through it" '\ + exec 2>&1 + uudecode -o input && tar xvf input; echo $? +diff --git a/testsuite/unzip.tests b/testsuite/unzip.tests +index af53de9..3b45ece 100755 +--- a/testsuite/unzip.tests b/testsuite/unzip.tests +@@ -31,7 +31,7 @@ rmdir foo + rm foo.zip + + # File containing some damaged encrypted stream +-optional FEATURE_UNZIP_CDF ++optional FEATURE_UNZIP_CDF UUDECODE + testing "unzip (bad archive)" "uudecode; unzip bad.zip 2>&1; echo \$?" \ + "Archive: bad.zip + inflating: ]3j½r«IK-%Ix +-- +2.7.4 + diff --git a/meta/recipes-core/busybox/busybox_1.29.2.bb b/meta/recipes-core/busybox/busybox_1.29.2.bb index df3ea59..a63fa82 100644 --- a/meta/recipes-core/busybox/busybox_1.29.2.bb +++ b/meta/recipes-core/busybox/busybox_1.29.2.bb @@ -41,6 +41,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \ file://rcS \ file://rcK \ file://makefile-libbb-race.patch \ + file://0001-testsuite-check-uudecode-before-using-it.patch \ " SRC_URI_append_libc-musl = " file://musl.cfg " -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 3/8] busybox: add zip to RDEPENDS of ptest package
busybox's unzip test case requires zip command. However, busybox itself does not provide one. So add zip as a runtime dependency. Signed-off-by: Chen Qi --- meta/recipes-core/busybox/busybox.inc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc index b8d8858..b45d2b6 100644 --- a/meta/recipes-core/busybox/busybox.inc +++ b/meta/recipes-core/busybox/busybox.inc @@ -53,6 +53,9 @@ RDEPENDS_${PN} = "${@["", "busybox-inittab"][(d.getVar('VIRTUAL-RUNTIME_init_man inherit cml1 systemd update-rc.d ptest +# busybox's unzip test case needs zip command, which busybox itself does not provide +RDEPENDS_${PN}-ptest = "zip" + # internal helper def busybox_cfg(feature, tokens, cnf, rem): if type(tokens) == type(""): -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 6/8] busybox: remove three hostname test cases
These three test case are removed because of two reasons. 1) net-tools' hostname have different behaviour regarding some options So when net-tools is installed, test cases may fail. e.g. test x$(hostname -i) = x$(busybox hostname -i) 2) hostname may set to some value, e.g. qemux86-64, which is not an entry in /etc/hosts This results in lookup failure when using `hostname -f' as there's no entry in /etc/hosts. Signed-off-by: Chen Qi --- meta/recipes-core/busybox/busybox.inc | 6 ++ 1 file changed, 6 insertions(+) diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc index b45d2b6..cc7246f 100644 --- a/meta/recipes-core/busybox/busybox.inc +++ b/meta/recipes-core/busybox/busybox.inc @@ -352,6 +352,12 @@ do_install_ptest () { cp -r ${B}/testsuite ${D}${PTEST_PATH}/ cp ${B}/.config ${D}${PTEST_PATH}/ ln -s /bin/busybox ${D}${PTEST_PATH}/busybox + # Remove some hostname test cases as they may fail because of two reasons + # 1) net-tools' hostname have different behaviour regarding some options + # 2) hostname may set to some value, e.g. qemux86-64, which is not an entry in /etc/hosts + rm ${D}${PTEST_PATH}/testsuite/hostname/hostname-i-works + rm ${D}${PTEST_PATH}/testsuite/hostname/hostname-d-works + rm ${D}${PTEST_PATH}/testsuite/hostname/hostname-s-works } inherit update-alternatives -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 2/8] bzip2: extend alternatives list to include bzip2
bzip2 could also be provided by busybox. So extend the alternatives list to include bzip2. Signed-off-by: Chen Qi --- meta/recipes-extended/bzip2/bzip2_1.0.6.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-extended/bzip2/bzip2_1.0.6.bb b/meta/recipes-extended/bzip2/bzip2_1.0.6.bb index 025f45c..7d04e30 100644 --- a/meta/recipes-extended/bzip2/bzip2_1.0.6.bb +++ b/meta/recipes-extended/bzip2/bzip2_1.0.6.bb @@ -29,7 +29,7 @@ CFLAGS_append = " -fPIC -fpic -Winline -fno-strength-reduce -D_FILE_OFFSET_BITS= inherit autotools update-alternatives ptest relative_symlinks ALTERNATIVE_PRIORITY = "100" -ALTERNATIVE_${PN} = "bunzip2 bzcat" +ALTERNATIVE_${PN} = "bunzip2 bzcat bzip2" #install binaries to bzip2-native under sysroot for replacement-native EXTRA_OECONF_append_class-native = " --bindir=${STAGING_BINDIR_NATIVE}/${PN}" -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH V2 0/8] Fixes for busybox ptest
Changes in V2: * fix bzip2 recipe to extend its alternatives list * check CONFIG_DESKTOP beforing using 'od -t' in test cases The following changes since commit bdffa0ce10870f70e0a7ab7636ff0d797df43e3a: apt: Improve SRC_URI (2018-11-20 10:32:16 +) are available in the git repository at: git://git.pokylinux.org/poky-contrib ChenQi/busybox-ptest http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/busybox-ptest Chen Qi (8): busybox: enable bzip2 by default bzip2: extend alternatives list to include bzip2 busybox: add zip to RDEPENDS of ptest package busybox: check uudecode before using it in test case busybox: use example.org instead of google.com in wget test case busybox: remove three hostname test cases busybox: fix du-l-works test case busybox: check CONFIG_DESKTOP before using 'od -t' in test case meta/recipes-core/busybox/busybox.inc | 9 +++ ...-du-l-works-fix-to-use-145-instead-of-144.patch | 32 + ...te-check-CONFIG_DESKTOP-before-using-od-t.patch | 83 ++ ...-testsuite-check-uudecode-before-using-it.patch | 45 ...e-use-www.example.org-for-wget-test-cases.patch | 62 meta/recipes-core/busybox/busybox/defconfig| 2 +- meta/recipes-core/busybox/busybox_1.29.2.bb| 4 ++ meta/recipes-extended/bzip2/bzip2_1.0.6.bb | 2 +- 8 files changed, 237 insertions(+), 2 deletions(-) create mode 100644 meta/recipes-core/busybox/busybox/0001-du-l-works-fix-to-use-145-instead-of-144.patch create mode 100644 meta/recipes-core/busybox/busybox/0001-testsuite-check-CONFIG_DESKTOP-before-using-od-t.patch create mode 100644 meta/recipes-core/busybox/busybox/0001-testsuite-check-uudecode-before-using-it.patch create mode 100644 meta/recipes-core/busybox/busybox/0001-testsuite-use-www.example.org-for-wget-test-cases.patch -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] apt: update SRC_URI
On 11/20/18 10:10 PM, Richard Purdie wrote: On Mon, 2018-11-19 at 21:54 +1100, Jonathan Liu wrote: -SRC_URI = " http://archive.ubuntu.com/ubuntu/pool/main/a/${BPN}/${BPN}_${PV}.tar.xz \ +SRC_URI = " https://launchpad.net/ubuntu/+archive/primary/+sourcefiles/apt/1.2.24/${BPN}_${PV}.tar.xz \ Why don't you replace apt with ${BPN} and 1.2.24 with ${PV} in SRC_URI? I tweaked this since it was easier than trying to remember to track this :) Cheers, Richard Thanks. :) -- BRs Sandy(Li Changqing) -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v4] u-boot: Add mkenvimage tool
On Tue, 2018-11-20 at 23:21 +0300, Alexey Brodkin wrote: > This utility is used for creation of images containing > usable in run-time U-Boot environment. > > As of today this utility is added per-board like here [1] > for Intel Edison board. > > [1] > http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb > > Given there're quite some U-Boot tools that we may want to add later > this recipe name switch from "u-boot-mkimage" to generic "u-boot- > tools" > still for compatibility we provide "u-boot-mkimage" with help > of PROVIDES as well as proposed "u-boot-mkenvimage". > > Signed-off-by: Alexey Brodkin > Cc: Richard Purdie > Cc: Otavio Salvador > Cc: Martin Jansa > Cc: Ross Burton > Cc: Marek Vasut > --- > > I may "git grep u-boot-mkimage" in more places: > 1. meta/classes/image_types.bbclass:311:CONVERSION_DEPENDS_u-boot = > "u-boot-mkimage-native" > 2. meta/classes/kernel-fitimage.bbclass:7:depends = "%s u- > boot-mkimage-native dtc-native" % depends > 3. meta/classes/kernel-uimage.bbclass:6:depends = "%s u- > boot-mkimage-native" % depends > 4. meta/conf/distro/include/distro_alias.inc:357:DISTRO_PN_ALIAS_pn- > u-boot-mkimage = "Ubuntu=uboot-mkimage Debian=uboot-mkimage" > > But that's not clear for me if it's required to swap "u-boot-mkimage" > to "u-boot-tools" see: > 1. "u-boot-mkimage" is still a valid recipe as we export ot via > PROVIDES > (or I'm mixing package name with recipe name which very well > might be > the case) > 2. I don't fully understand a mening of "DISTRO_PN_ALIAS". > If that's a name of a package from some big Linux distro or how > target > utility is named in that distro. For example in Debian Sid > there's > a package named "u-boot-tools" which includes "mkimage", > "mkenvimage", > "fw_printenv", "fw_setenv" and many more while in > "distro_alias.inc" > we still have: > >8 > DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage > Debian=uboot-mkimage" > >8 I meant to say, for cleanliness, all the references you mentioned should also be converted in this patch. Most will work with PROVIDES mapping but we should improve them. Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v4] u-boot: Add mkenvimage tool
On Tue, 2018-11-20 at 23:21 +0300, Alexey Brodkin wrote: > This utility is used for creation of images containing > usable in run-time U-Boot environment. > > As of today this utility is added per-board like here [1] > for Intel Edison board. > > [1] > http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb > > Given there're quite some U-Boot tools that we may want to add later > this recipe name switch from "u-boot-mkimage" to generic "u-boot- > tools" > still for compatibility we provide "u-boot-mkimage" with help > of PROVIDES as well as proposed "u-boot-mkenvimage". > > Signed-off-by: Alexey Brodkin > Cc: Richard Purdie > Cc: Otavio Salvador > Cc: Martin Jansa > Cc: Ross Burton > Cc: Marek Vasut > --- > > I may "git grep u-boot-mkimage" in more places: > 1. meta/classes/image_types.bbclass:311:CONVERSION_DEPENDS_u-boot = > "u-boot-mkimage-native" > 2. meta/classes/kernel-fitimage.bbclass:7:depends = "%s u- > boot-mkimage-native dtc-native" % depends > 3. meta/classes/kernel-uimage.bbclass:6:depends = "%s u- > boot-mkimage-native" % depends > 4. meta/conf/distro/include/distro_alias.inc:357:DISTRO_PN_ALIAS_pn- > u-boot-mkimage = "Ubuntu=uboot-mkimage Debian=uboot-mkimage" > > But that's not clear for me if it's required to swap "u-boot-mkimage" > to "u-boot-tools" see: > 1. "u-boot-mkimage" is still a valid recipe as we export ot via > PROVIDES > (or I'm mixing package name with recipe name which very well > might be > the case) > 2. I don't fully understand a mening of "DISTRO_PN_ALIAS". > If that's a name of a package from some big Linux distro or how > target > utility is named in that distro. For example in Debian Sid > there's > a package named "u-boot-tools" which includes "mkimage", > "mkenvimage", > "fw_printenv", "fw_setenv" and many more while in > "distro_alias.inc" > we still have: > >8 > DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage > Debian=uboot-mkimage" > >8 Sorry, the tests in meta-oe have exposed another issue: https://8n1.org/14048/3f99 which means the PROVIDES isn't quite right. The reason is there is magic in the recipe which extends it to the native and nativesdk cases but its not quite working. I think the easiest way to address this may be PROVIDES = "${MLPREFIX}u-boot-mkimage ${MLPREFIX}u-boot-mkenvimage" PROVIDES_class-native = "u-boot-mkimage-native u-boot-mkenvimage- native" since in the nativesdk case, MLPREFIX expands to nativesdk-. The second class-native line may not be needed. You could check with: bitbake u-boot-tools -e | grep ^PROVIDES= and see if it computes the value without manually overriding it. Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [[thud][PATCH] meta/classes/qemu.bbclass: set -cpu of e500v2 to e500mc
Hello , Please apply this patch in thud branch . Now build gcc failed for e500v2 Thanks Chunrong -Original Message- From: C.r. Guo Sent: 2018年11月15日 11:37 Cc: C.r. Guo Subject: [[thud][PATCH] meta/classes/qemu.bbclass: set -cpu of e500v2 to e500mc From: Chunrong Guo *fix the below error |The postinstall intercept hook 'update_font_cache' failed |qemu: uncaught target signal 4 (Illegal instruction) - core dumped This workaround will need to be reverted once the proper fix is found. Signed-off-by: Chunrong Guo --- meta/classes/qemu.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/classes/qemu.bbclass b/meta/classes/qemu.bbclass index f5c5780..e8545f1 100644 --- a/meta/classes/qemu.bbclass +++ b/meta/classes/qemu.bbclass @@ -55,7 +55,7 @@ def qemu_run_binary(data, rootfs_path, binary): QEMU_OPTIONS = "-r ${OLDEST_KERNEL} ${@d.getVar("QEMU_EXTRAOPTIONS_%s" % d.getVar('PACKAGE_ARCH')) or ""}" QEMU_OPTIONS[vardeps] += "QEMU_EXTRAOPTIONS_${PACKAGE_ARCH}" -QEMU_EXTRAOPTIONS_ppce500v2 = " -cpu e500v2" +QEMU_EXTRAOPTIONS_ppce500v2 = " -cpu e500mc" QEMU_EXTRAOPTIONS_ppce500mc = " -cpu e500mc" QEMU_EXTRAOPTIONS_ppce5500 = " -cpu e500mc" QEMU_EXTRAOPTIONS_ppc64e5500 = " -cpu e500mc" -- 2.7.4 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v4] u-boot: Add mkenvimage tool
Hi Marek, > -Original Message- > From: Marek Vasut [mailto:ma...@denx.de] > Sent: Tuesday, November 20, 2018 11:23 PM > To: Alexey Brodkin ; > openembedded-core@lists.openembedded.org > Cc: Ross Burton ; Khem Raj ; > linux-snps-...@lists.infradead.org; Richard Purdie > ; Otavio Salvador > ; Martin Jansa > Subject: Re: [PATCH v4] u-boot: Add mkenvimage tool > > On 11/20/2018 09:21 PM, Alexey Brodkin wrote: > > This utility is used for creation of images containing > > usable in run-time U-Boot environment. > > > > As of today this utility is added per-board like here [1] > > for Intel Edison board. > > > > [1] > > https://urldefense.proofpoint.com/v2/url?u=http-3A__git.yoctoproject.org_cgit_cgit.cgi_meta-2Dintel-2Dedison_tree_meta- > 2Dintel-2Dedison-2Dbsp_recipes-2Dbsp_u-2Dboot_u-2Dboot-2Dtools- > 5F2014.04.bb&d=DwICaQ&c=DPL6_X_6JkXFx7AXWqB0tg&r=lqdeeSSEes0GFDDl656eViXO7breS55ytWkhpk5R81I&m=nj37jjvGHLwnN1 > dG2F_IclB3hs-QGaHNS6Z30MOVzGg&s=ReoeMoga0WNrEzYviKd1GBckgNDpZ1E6fs9XguZmy7k&e= > > > > Given there're quite some U-Boot tools that we may want to add later > > this recipe name switch from "u-boot-mkimage" to generic "u-boot-tools" > > still for compatibility we provide "u-boot-mkimage" with help > > of PROVIDES as well as proposed "u-boot-mkenvimage". > > > > Signed-off-by: Alexey Brodkin > > Cc: Richard Purdie > > Cc: Otavio Salvador > > Cc: Martin Jansa > > Cc: Ross Burton > > Cc: Marek Vasut > > --- > > > > I may "git grep u-boot-mkimage" in more places: > > 1. meta/classes/image_types.bbclass:311:CONVERSION_DEPENDS_u-boot = > > "u-boot-mkimage-native" > > 2. meta/classes/kernel-fitimage.bbclass:7:depends = "%s > > u-boot-mkimage-native dtc-native" % depends > > 3. meta/classes/kernel-uimage.bbclass:6:depends = "%s > > u-boot-mkimage-native" % depends > > 4. > > meta/conf/distro/include/distro_alias.inc:357:DISTRO_PN_ALIAS_pn-u-boot-mkimage > > = "Ubuntu=uboot-mkimage > Debian=uboot-mkimage" > > > > But that's not clear for me if it's required to swap "u-boot-mkimage" > > to "u-boot-tools" see: > > 1. "u-boot-mkimage" is still a valid recipe as we export ot via PROVIDES > > (or I'm mixing package name with recipe name which very well might be > > the case) > > 2. I don't fully understand a mening of "DISTRO_PN_ALIAS". > > If that's a name of a package from some big Linux distro or how target > > utility is named in that distro. For example in Debian Sid there's > > a package named "u-boot-tools" which includes "mkimage", "mkenvimage", > > "fw_printenv", "fw_setenv" and many more while in "distro_alias.inc" > > we still have: > > >8 > > DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage > > Debian=uboot-mkimage" > > >8 > > Just make u-boot-tools RPROVIDE the u-boot-mkimage, and make it generate > a u-boot-mkimage package , no ? Sorry I'm still a bit confused with all those complexities of OE so pls pardon my silly questions. What are we going to solve with RPROVIDE and how it differs from PROVIDES I already added in the recipe? -Alexey -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v4] u-boot: Add mkenvimage tool
On 11/20/2018 09:21 PM, Alexey Brodkin wrote: > This utility is used for creation of images containing > usable in run-time U-Boot environment. > > As of today this utility is added per-board like here [1] > for Intel Edison board. > > [1] > http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb > > Given there're quite some U-Boot tools that we may want to add later > this recipe name switch from "u-boot-mkimage" to generic "u-boot-tools" > still for compatibility we provide "u-boot-mkimage" with help > of PROVIDES as well as proposed "u-boot-mkenvimage". > > Signed-off-by: Alexey Brodkin > Cc: Richard Purdie > Cc: Otavio Salvador > Cc: Martin Jansa > Cc: Ross Burton > Cc: Marek Vasut > --- > > I may "git grep u-boot-mkimage" in more places: > 1. meta/classes/image_types.bbclass:311:CONVERSION_DEPENDS_u-boot = > "u-boot-mkimage-native" > 2. meta/classes/kernel-fitimage.bbclass:7:depends = "%s > u-boot-mkimage-native dtc-native" % depends > 3. meta/classes/kernel-uimage.bbclass:6:depends = "%s > u-boot-mkimage-native" % depends > 4. > meta/conf/distro/include/distro_alias.inc:357:DISTRO_PN_ALIAS_pn-u-boot-mkimage > = "Ubuntu=uboot-mkimage Debian=uboot-mkimage" > > But that's not clear for me if it's required to swap "u-boot-mkimage" > to "u-boot-tools" see: > 1. "u-boot-mkimage" is still a valid recipe as we export ot via PROVIDES > (or I'm mixing package name with recipe name which very well might be > the case) > 2. I don't fully understand a mening of "DISTRO_PN_ALIAS". > If that's a name of a package from some big Linux distro or how target > utility is named in that distro. For example in Debian Sid there's > a package named "u-boot-tools" which includes "mkimage", "mkenvimage", > "fw_printenv", "fw_setenv" and many more while in "distro_alias.inc" > we still have: > >8 > DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage > Debian=uboot-mkimage" > >8 Just make u-boot-tools RPROVIDE the u-boot-mkimage, and make it generate a u-boot-mkimage package , no ? -- Best regards, Marek Vasut -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH v4] u-boot: Add mkenvimage tool
This utility is used for creation of images containing usable in run-time U-Boot environment. As of today this utility is added per-board like here [1] for Intel Edison board. [1] http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb Given there're quite some U-Boot tools that we may want to add later this recipe name switch from "u-boot-mkimage" to generic "u-boot-tools" still for compatibility we provide "u-boot-mkimage" with help of PROVIDES as well as proposed "u-boot-mkenvimage". Signed-off-by: Alexey Brodkin Cc: Richard Purdie Cc: Otavio Salvador Cc: Martin Jansa Cc: Ross Burton Cc: Marek Vasut --- I may "git grep u-boot-mkimage" in more places: 1. meta/classes/image_types.bbclass:311:CONVERSION_DEPENDS_u-boot = "u-boot-mkimage-native" 2. meta/classes/kernel-fitimage.bbclass:7:depends = "%s u-boot-mkimage-native dtc-native" % depends 3. meta/classes/kernel-uimage.bbclass:6:depends = "%s u-boot-mkimage-native" % depends 4. meta/conf/distro/include/distro_alias.inc:357:DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage Debian=uboot-mkimage" But that's not clear for me if it's required to swap "u-boot-mkimage" to "u-boot-tools" see: 1. "u-boot-mkimage" is still a valid recipe as we export ot via PROVIDES (or I'm mixing package name with recipe name which very well might be the case) 2. I don't fully understand a mening of "DISTRO_PN_ALIAS". If that's a name of a package from some big Linux distro or how target utility is named in that distro. For example in Debian Sid there's a package named "u-boot-tools" which includes "mkimage", "mkenvimage", "fw_printenv", "fw_setenv" and many more while in "distro_alias.inc" we still have: >8 DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage Debian=uboot-mkimage" >8 Changes v3 -> v4: * Updated maintainer info to make Yocto autobuilder happy again Changes v2 -> v3: * Recipe renamed to "u-boot-tools" to better match its contents (especially in the future as we add more tools there) * Existing "u-boot-mkimage" and newly introduced "u-boot-mkenvimage" made avaialble via PROVIDES Changes v1 -> v2: * Got rid of a separate recipe and instead added mkenvimage to u-boot mkimage recipe meta/conf/distro/include/maintainers.inc | 2 +- .../{u-boot-mkimage_2018.07.bb => u-boot-tools_2018.07.bb} | 9 - 2 files changed, 9 insertions(+), 2 deletions(-) rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2018.07.bb => u-boot-tools_2018.07.bb} (83%) diff --git a/meta/conf/distro/include/maintainers.inc b/meta/conf/distro/include/maintainers.inc index 0889c1c429..825c24a3f8 100644 --- a/meta/conf/distro/include/maintainers.inc +++ b/meta/conf/distro/include/maintainers.inc @@ -654,7 +654,7 @@ RECIPE_MAINTAINER_pn-tzcode-native = "Armin Kuster " RECIPE_MAINTAINER_pn-tzdata = "Armin Kuster " RECIPE_MAINTAINER_pn-u-boot = "Marek Vasut " RECIPE_MAINTAINER_pn-u-boot-fw-utils = "Marek Vasut " -RECIPE_MAINTAINER_pn-u-boot-mkimage = "Marek Vasut " +RECIPE_MAINTAINER_pn-u-boot-tools = "Marek Vasut " RECIPE_MAINTAINER_pn-udev-extraconf = "Ross Burton " RECIPE_MAINTAINER_pn-unfs3 = "Ross Burton " RECIPE_MAINTAINER_pn-unifdef = "Ross Burton " diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2018.07.bb b/meta/recipes-bsp/u-boot/u-boot-tools_2018.07.bb similarity index 83% rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2018.07.bb rename to meta/recipes-bsp/u-boot/u-boot-tools_2018.07.bb index 08bff1d161..140afa8dfe 100644 --- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2018.07.bb +++ b/meta/recipes-bsp/u-boot/u-boot-tools_2018.07.bb @@ -1,7 +1,8 @@ require u-boot-common_${PV}.inc -SUMMARY = "U-Boot bootloader image creation tool" +SUMMARY = "U-Boot bootloader tools" DEPENDS += "openssl" +PROVIDES = "u-boot-mkimage u-boot-mkenvimage" EXTRA_OEMAKE_class-target = 'CROSS_COMPILE="${TARGET_PREFIX}" CC="${CC} ${CFLAGS} ${LDFLAGS}" HOSTCC="${BUILD_CC} ${BUILD_CFLAGS} ${BUILD_LDFLAGS}" STRIP=true V=1' EXTRA_OEMAKE_class-native = 'CC="${BUILD_CC} ${BUILD_CFLAGS} ${BUILD_LDFLAGS}" HOSTCC="${BUILD_CC} ${BUILD_CFLAGS} ${BUILD_LDFLAGS}" STRIP=true V=1' @@ -20,8 +21,14 @@ do_compile () { do_install () { install -d ${D}${bindir} + + # mkimage install -m 0755 tools/mkimage ${D}${bindir}/uboot-mkimage ln -sf uboot-mkimage ${D}${bindir}/mkimage + + # mkenvimage + install -m 0755 tools/mkenvimage ${D}${bindir}/uboot-mkenvimage + ln -sf uboot-mkenvimage ${D}${bindir}/mkenvimage } RDEPENDS_${PN} += "dtc" -- 2.16.2 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 1/2] meta/icecc.bbclass: Move system blacklist to variables
The system blacklists are moved to variables which are ignore when hashing. This prevents changes to the blacklists from causing all taskhashes to change (and thus rebuild). Signed-off-by: Joshua Watt --- meta/classes/icecc.bbclass | 31 --- 1 file changed, 20 insertions(+), 11 deletions(-) diff --git a/meta/classes/icecc.bbclass b/meta/classes/icecc.bbclass index 7597e4c0c93..037cce1e964 100644 --- a/meta/classes/icecc.bbclass +++ b/meta/classes/icecc.bbclass @@ -33,7 +33,7 @@ BB_HASHBASE_WHITELIST += "ICECC_PARALLEL_MAKE ICECC_DISABLED ICECC_USER_PACKAGE_ ICECC_CARET_WORKAROUND ICECC_CFLAGS ICECC_ENV_VERSION \ ICECC_DEBUG ICECC_LOGFILE ICECC_REPEAT_RATE ICECC_PREFERRED_HOST \ ICECC_CLANG_REMOTE_CPP ICECC_IGNORE_UNVERIFIED ICECC_TEST_SOCKET \ -ICECC_ENV_DEBUG \ +ICECC_ENV_DEBUG ICECC_SYSTEM_PACKAGE_BL ICECC_SYSTEM_CLASS_BL \ " ICECC_ENV_EXEC ?= "${STAGING_BINDIR_NATIVE}/icecc-create-env" @@ -61,6 +61,23 @@ CXXFLAGS += "${ICECC_CFLAGS}" # Debug flags when generating environments ICECC_ENV_DEBUG ??= "" +# "system" recipe blacklist contains a list of packages that can not distribute +# compile tasks for one reason or the other. When adding new entry, please +# document why (how it failed) so that we can re-evaluate it later e.g. when +# there is new version +# +# libgcc-initial - fails with CPP sanity check error if host sysroot contains +# cross gcc built for another target tune/variant +ICECC_SYSTEM_PACKAGE_BL += "\ +libgcc-initial \ +" + +# "system" classes that should be blacklisted. When adding new entry, please +# document why (how it failed) so that we can re-evaluate it later +# +ICECC_SYSTEM_CLASS_BL += "\ +" + def icecc_dep_prepend(d): # INHIBIT_DEFAULT_DEPS doesn't apply to the patch command. Whether or not # we need that built is the responsibility of the patch function / class, not @@ -134,7 +151,7 @@ def use_icecc(bb,d): pn = d.getVar('PN') -system_class_blacklist = [] +system_class_blacklist = (d.getVar('ICECC_SYSTEM_CLASS_BL') or "").split() user_class_blacklist = (d.getVar('ICECC_USER_CLASS_BL') or "none").split() package_class_blacklist = system_class_blacklist + user_class_blacklist @@ -143,15 +160,7 @@ def use_icecc(bb,d): bb.debug(1, "%s: class %s found in blacklist, disable icecc" % (pn, black)) return "no" -# "system" recipe blacklist contains a list of packages that can not distribute compile tasks -# for one reason or the other -# this is the old list (which doesn't seem to be valid anymore, because I was able to build -# all these with icecc enabled) -# system_package_blacklist = [ "glibc", "gcc", "bind", "u-boot", "dhcp-forwarder", "enchant", "connman", "orbit2" ] -# when adding new entry, please document why (how it failed) so that we can re-evaluate it later -# e.g. when there is new version -# building libgcc-initial with icecc fails with CPP sanity check error if host sysroot contains cross gcc built for another target tune/variant -system_package_blacklist = ["libgcc-initial"] +system_package_blacklist = (d.getVar('ICECC_SYSTEM_PACKAGE_BL') or "").split() user_package_blacklist = (d.getVar('ICECC_USER_PACKAGE_BL') or "").split() user_package_whitelist = (d.getVar('ICECC_USER_PACKAGE_WL') or "").split() package_blacklist = system_package_blacklist + user_package_blacklist -- 2.19.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 2/2] meta/icecc.bbclass: Update system blacklists
Updates the system blacklists to include packages that are known to have problems compiling under icecream Signed-off-by: Joshua Watt --- meta/classes/icecc.bbclass | 8 1 file changed, 8 insertions(+) diff --git a/meta/classes/icecc.bbclass b/meta/classes/icecc.bbclass index 037cce1e964..2b189232cb7 100644 --- a/meta/classes/icecc.bbclass +++ b/meta/classes/icecc.bbclass @@ -68,14 +68,22 @@ ICECC_ENV_DEBUG ??= "" # # libgcc-initial - fails with CPP sanity check error if host sysroot contains # cross gcc built for another target tune/variant +# target-sdk-provides-dummy - ${HOST_PREFIX} is empty which triggers the "NULL +# prefix" error. ICECC_SYSTEM_PACKAGE_BL += "\ libgcc-initial \ +target-sdk-provides-dummy \ " # "system" classes that should be blacklisted. When adding new entry, please # document why (how it failed) so that we can re-evaluate it later # +# image - Image aren't compiling, but the testing framework for images captures +# PARALLEL_MAKE as part of the test environment. Many tests won't use +# icecream, but leaving the high level of parallelism can cause them to +# consume an unnecessary amount of resources. ICECC_SYSTEM_CLASS_BL += "\ +image \ " def icecc_dep_prepend(d): -- 2.19.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 0/2] Update icecream blacklists
Updates the icecream blacklists and makes some changes to prevent such updates from causing world rebuilds in the future. Joshua Watt (2): meta/icecc.bbclass: Move system blacklist to variables meta/icecc.bbclass: Update system blacklists meta/classes/icecc.bbclass | 39 +++--- 1 file changed, 28 insertions(+), 11 deletions(-) -- 2.19.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [oe-commits] [openembedded-core] 10/35: tzcode: update to 2018f
On Fri, Nov 16, 2018 at 04:33:46PM +, g...@git.openembedded.org wrote: > This is an automated email from the git hooks/post-receive script. > > rpurdie pushed a commit to branch sumo > in repository openembedded-core. > > commit 9702041a732ca062a9899543d05e585c930407f3 > Author: Armin Kuster > AuthorDate: Mon Oct 22 15:05:11 2018 +0100 > > tzcode: update to 2018f This breaks postgresql build in sumo: | postgresql/9.4.17-r0/recipe-sysroot-native/usr/bin/zic -d 'postgresql/9.4.17-r0/image/usr/share/postgresql/timezone' -p 'US/Eastern' postgresql/9.4.17-r0/build/../postgresql-9.4.17/src/timezone/data/tzdata.zi | "postgresql/9.4.17-r0/build/../postgresql-9.4.17/src/timezone/data/tzdata.zi", line 1595: Invalid rule name "+" | "postgresql/9.4.17-r0/build/../postgresql-9.4.17/src/timezone/data/tzdata.zi", line 1596: Invalid rule name "+" | "postgresql/9.4.17-r0/build/../postgresql-9.4.17/src/timezone/data/tzdata.zi", line 1597: Invalid rule name "+" | "postgresql/9.4.17-r0/build/../postgresql-9.4.17/src/timezone/data/tzdata.zi", line 1598: Invalid rule name "+" | "postgresql/9.4.17-r0/build/../postgresql-9.4.17/src/timezone/data/tzdata.zi", line 1599: Invalid rule name "+" | Makefile:52: recipe for target 'install' failed | make[2]: *** [install] Error 1 | make[2]: Leaving directory 'postgresql/9.4.17-r0/build/src/timezone' | Makefile:35: recipe for target 'install-timezone-recurse' failed | make[1]: *** [install-timezone-recurse] Error 2 Are you going to backport: commit 805e74d00cff3a7566335150f7e3b41972265cc1 Author: Armin Kuster Date: Mon Oct 22 16:47:55 2018 +0100 postgresql: Fix build issue with timezone 2018f update Sync our copy of the timezone library with IANA release tzcode2018f. to meta-oe/sumo? Thanks > > Changes to code > > zic now always generates TZif files where time type 0 is used for > timestamps before the first transition. This simplifies the > reading of TZif files and should not affect behavior of existing > TZif readers because the same set of time types is used; only > their internal indexes may have changed. This affects only the > legacy zones EST5EDT, CST6CDT, MST7MDT, PST8PDT, CET, MET, and > EET, which previously used nonzero types for these timestamps. > > Because of the type 0 change, zic no longer outputs a dummy > transition at time -2**59 (before the Big Bang), as clients should > no longer need this to handle historical timestamps correctly. > This reverts a change introduced in 2013d and shrinks most TZif > files by a few bytes. > > zic now supports negative time-of-day in Rule and Leap lines, e.g., > "Rule X min max - Apr lastSun -6:00 1:00 -" means the transition > occurs at 18:00 on the Saturday before the last Sunday in April. > This behavior was documented in 2018a but the code did not > entirely match the documentation. > > localtime.c no longer requires at least one time type in TZif > files that lack transitions or have a POSIX-style TZ string. This > future-proofs the code against possible future extensions to the > format that would allow TZif files with POSIX-style TZ strings and > without transitions or time types. > > A read-access subscript error in localtime.c has been fixed. > It could occur only in TZif files with timecnt == 0, something that > does not happen in practice now but could happen in future versions. > > localtime.c no longer ignores TZif POSIX-style TZ strings that > specify only standard time. Instead, these TZ strings now > override the default time type for timestamps after the last > transition (or for all time stamps if there are no transitions), > just as DST strings specifying DST have always done. > > leapseconds.awk now outputs "#updated" and "#expires" comments, > and supports leap seconds at the ends of months other than June > and December. (Inspired by suggestions from Chris Woodbury.) > (From OE-Core rev: 4670dcdb6e2504469c30ebed828d4702d8c0003c) > > Signed-off-by: Armin Kuster > Signed-off-by: Richard Purdie > Signed-off-by: Armin Kuster > --- > .../tzcode/{tzcode-native_2018e.bb => tzcode-native_2018f.bb} | 8 > > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/meta/recipes-extended/tzcode/tzcode-native_2018e.bb > b/meta/recipes-extended/tzcode/tzcode-native_2018f.bb > similarity index 70% > rename from meta/recipes-extended/tzcode/tzcode-native_2018e.bb > rename to meta/recipes-extended/tzcode/tzcode-native_2018f.bb > index 9a4e465..816e34d 100644 > --- a/meta/recipes-extended/tzcode/tzcode-native_2018e.bb > +++ b/meta/recipes-extended/tzcode/tzcode-native_2018f.bb > @@ -11,10 +11,10 @@ SRC_URI =" > http://www.iana.o
Re: [OE-core] [PATCH 1/6] libmodulemd: add a new recipe
On Mon, 2018-11-19 at 16:08 +0100, Alexander Kanavin wrote: > This is a hard requirement of the new libdnf versions. > > Signed-off-by: Alexander Kanavin > --- > ...ec_tmpl.sh-use-bin-sh-not-usr-bin-sh.patch | 21 ++ > ...n.build-do-not-generate-gir-or-gtkdo.patch | 65 > +++ > .../libmodulemd/libmodulemd_git.bb| 21 ++ > 3 files changed, 107 insertions(+) > create mode 100644 meta/recipes- > devtools/libmodulemd/libmodulemd/0001-spec_tmpl.sh-use-bin-sh-not- > usr-bin-sh.patch > create mode 100644 meta/recipes- > devtools/libmodulemd/libmodulemd/0002-modulemd-v1-meson.build-do-not- > generate-gir-or-gtkdo.patch > create mode 100644 meta/recipes- > devtools/libmodulemd/libmodulemd_git.bb I think something in this series may have caused: https://autobuilder.yoctoproject.org/typhoon/#/builders/11/builds/230/steps/7/logs/step1c https://autobuilder.yoctoproject.org/typhoon/#/builders/27/builds/230/steps/7/logs/step1c and maybe some (but not all) of: https://autobuilder.yoctoproject.org/typhoon/#/builders/28/builds/242/steps/7/logs/step2d Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] systemd: fix PN-container package splitting
On Tue, 2018-11-20 at 17:39 +, Richard Purdie wrote: > On Mon, 2018-11-19 at 23:33 +0100, Adrian Freihofer wrote: > > From: Adrian Freihofer > > > > - Fix package splitting for services installed with > > systemd-container. Some files have been installed with systemd > > package instead of systemd-container package. (*.service, > > *.socket, *.html...) > > - Split journal-upload, journal-remote, journal-gatewayd into > > separate packages. These services are not strictly related to > > nspawn containers. Remote logging is now possible without > > installing all the container stuff. > > - systemd-container package rdepends on the new packages to > > maintain 100% backwards compatibility. But nspawn does not > > really depend on them. rrecommends would be fine as well. > > > > Signed-off-by: Adrian Freihofer > > --- > > meta/recipes-core/systemd/systemd_239.bb | 56 > > > > 1 file changed, 48 insertions(+), 8 deletions(-) > > This causes failures in oe-selftest: > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/28/builds/242/steps/7/logs/step2d > > (you can run individual tests with oe-selftest -r Also: https://autobuilder.yoctoproject.org/typhoon/#/builders/35/builds/192/steps/7/logs/step5b https://autobuilder.yoctoproject.org/typhoon/#/builders/35/builds/192/steps/7/logs/step7b Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
On Mon, 2018-11-19 at 13:04 +0300, Alexey Brodkin wrote: > This utility is used for creation of images containing > usable in run-time U-Boot environment. > > As of today this utility is added per-board like here [1] > for Intel Edison board. > > [1] > http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb > > Given there're quite some U-Boot tools that we may want to add later > this recipe name switch from "u-boot-mkimage" to generic "u-boot- > tools" > still for compatibility we provide "u-boot-mkimage" with help > of PROVIDES as well as proposed "u-boot-mkenvimage". > > Signed-off-by: Alexey Brodkin > Cc: Richard Purdie > Cc: Otavio Salvador > Cc: Martin Jansa > Cc: Ross Burton > Cc: Marek Vasut > --- I know others are talking about splitting the packaging and I'm ok with deferring that to a separate patch. This patch did break the build in oe-selftest as you renamed a recipe and didn't update conf/distro/include/maintainers.inc. Could you send a new version with that tweak please? (grep https://autobuilder.yoctoproject.org/typhoon/#/builders/28/builds/242/steps/7/logs/step2d for the recipe name to see the test that failed) Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] systemd: fix PN-container package splitting
On Mon, 2018-11-19 at 23:33 +0100, Adrian Freihofer wrote: > From: Adrian Freihofer > > - Fix package splitting for services installed with > systemd-container. Some files have been installed with systemd > package instead of systemd-container package. (*.service, > *.socket, *.html...) > - Split journal-upload, journal-remote, journal-gatewayd into > separate packages. These services are not strictly related to > nspawn containers. Remote logging is now possible without > installing all the container stuff. > - systemd-container package rdepends on the new packages to > maintain 100% backwards compatibility. But nspawn does not > really depend on them. rrecommends would be fine as well. > > Signed-off-by: Adrian Freihofer > --- > meta/recipes-core/systemd/systemd_239.bb | 56 > > 1 file changed, 48 insertions(+), 8 deletions(-) This causes failures in oe-selftest: https://autobuilder.yoctoproject.org/typhoon/#/builders/28/builds/242/steps/7/logs/step2d (you can run individual tests with oe-selftest -r Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [sumo] [PATCH v1] python: CVE-2018-1061
It looks like this patch was not applied. https://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg116341.html Any reasons, why? -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
On Tue, Nov 20, 2018 at 1:52 PM Alexey Brodkin wrote: > > -Original Message- > > From: Otavio Salvador [mailto:otavio.salva...@ossystems.com.br] > > Sent: Tuesday, November 20, 2018 6:49 PM > > To: Burton, Ross > > Cc: Peter Kjellerstedt ; Marek Vasut > > ; Tom Rini ; Otavio > > Salvador ; Alexey Brodkin > > ; Patches and discussions about the oe-core > > layer ; > > linux-snps-...@lists.infradead.org > > Subject: Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool > > > > On Tue, Nov 20, 2018 at 1:40 PM Burton, Ross wrote: > > > > > > (hit sent too early) > > > > > > Of course how big are these tools, and is it really useful splitting > > > them up? If we're talking a 10k binary, is putting them in separate > > > packages worth the complication? > > > > I think it makes sense to split; we are building an embedded system > > build system and granular installation is a must have. Using PN to > > install all tools is a win win combination. > > Please note we're discussing _host_ tools here! > So do we really care if there's one more utility installed on development > host? > > And if someone installs those U-Boot host utilities on the target then most > probably > this target is not super embedded any longer. The tools are commonly used on host but this is not a limitation and there are use cases where we may want to include it on target. Also, it can be installed on SDK (nativesdk packages) and being able to install what will be used is a must for me. I am not concerned about the space used by it but for the choice of me pick the right pieces I need, and not putting it all together. -- Otavio Salvador O.S. Systems http://www.ossystems.com.brhttp://code.ossystems.com.br Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
On 11/20/2018 04:52 PM, Alexey Brodkin wrote: > Hi Otavio, > >> -Original Message- >> From: Otavio Salvador [mailto:otavio.salva...@ossystems.com.br] >> Sent: Tuesday, November 20, 2018 6:49 PM >> To: Burton, Ross >> Cc: Peter Kjellerstedt ; Marek Vasut >> ; Tom Rini ; Otavio >> Salvador ; Alexey Brodkin >> ; Patches and discussions about the oe-core >> layer ; >> linux-snps-...@lists.infradead.org >> Subject: Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool >> >> On Tue, Nov 20, 2018 at 1:40 PM Burton, Ross wrote: >>> >>> (hit sent too early) >>> >>> Of course how big are these tools, and is it really useful splitting >>> them up? If we're talking a 10k binary, is putting them in separate >>> packages worth the complication? >> >> I think it makes sense to split; we are building an embedded system >> build system and granular installation is a must have. Using PN to >> install all tools is a win win combination. > > Please note we're discussing _host_ tools here! > So do we really care if there's one more utility installed on development > host? > > And if someone installs those U-Boot host utilities on the target then most > probably > this target is not super embedded any longer. You can use mkenvimage on target, I had to do that a few times. -- Best regards, Marek Vasut -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
Hi Otavio, > -Original Message- > From: Otavio Salvador [mailto:otavio.salva...@ossystems.com.br] > Sent: Tuesday, November 20, 2018 6:49 PM > To: Burton, Ross > Cc: Peter Kjellerstedt ; Marek Vasut > ; Tom Rini ; Otavio > Salvador ; Alexey Brodkin > ; Patches and discussions about the oe-core > layer ; > linux-snps-...@lists.infradead.org > Subject: Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool > > On Tue, Nov 20, 2018 at 1:40 PM Burton, Ross wrote: > > > > (hit sent too early) > > > > Of course how big are these tools, and is it really useful splitting > > them up? If we're talking a 10k binary, is putting them in separate > > packages worth the complication? > > I think it makes sense to split; we are building an embedded system > build system and granular installation is a must have. Using PN to > install all tools is a win win combination. Please note we're discussing _host_ tools here! So do we really care if there's one more utility installed on development host? And if someone installs those U-Boot host utilities on the target then most probably this target is not super embedded any longer. -Alexey -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
On Tue, Nov 20, 2018 at 01:48:40PM -0200, Otavio Salvador wrote: > On Tue, Nov 20, 2018 at 1:40 PM Burton, Ross wrote: > > > > (hit sent too early) > > > > Of course how big are these tools, and is it really useful splitting > > them up? If we're talking a 10k binary, is putting them in separate > > packages worth the complication? > > I think it makes sense to split; we are building an embedded system > build system and granular installation is a must have. Using PN to > install all tools is a win win combination. Agreed. The common use cases won't need all of the possible tools so lets not grow sizes we don't need to grow. -- Tom signature.asc Description: PGP signature -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] Yocto Project Status WW47’18
Current Dev Position: YP 2.7 M1. Next Deadline: YP 2.7 M1 Cutoff is Dec. 10, 2018 SWAT Team Rotation: · SWAT lead is currently: Paul · SWAT team rotation: Paul -> Ross on Nov. 23, 2018 · SWAT team rotation: Ross -> Amanda on Nov. 30, 2018 · https://wiki.yoctoproject.org/wiki/Yocto_Build_Failure_Swat_Team Key Status/Updates: · YP 2.6 (thud) was released based on rc1. · We’d like to thank everyone who was involved in 2.6 for making what is shaping up to be a great project release. This applied to everyone involved, be it through patches, docs, bug triage, autobuilder, QA, programme management, bug reporting, sysadmin or anything else, it all goes to helping the releases (and hence the project) being successful. · YP 2.4.4 (Rocko) is in QA and 93% complete. See: https://wiki.yoctoproject.org/wiki/2.4_QA_Status · Master has opened for new changes for 2.7 M1 and patches are being tested and merged as usual. · The next release will be ‘warrior’ followed by ‘zeus’. · The AUH has sent out patches for various upgrades recently which is timely for merging into 2.7 so lets collect those up and get them merged! · There are some autobuilder changes happening at the moment to assist us with the QA changes happening in 2.7. We updated to the recent buildbot 1.6.0 release which contained bug fixes for some UI glitches we were seeing (was good to be able to work with the latest code!). Planned Releases for YP 2.7: · YP 2.7 M1 Cutoff is Dec. 10, 2018 · YP 2.7 M1 Release Target is Dec. 21, 2018 · YP 2.7 M2 Cutoff is Jan. 21, 2019 · YP 2.7 M2 Release Target is Feb. 1, 2019 · YP 2.7 M3 Cutoff is Feb. 25, 2019 · YP 2.7 M3 Release Target is Mar. 8, 2019 · YP 2.7 M4 Cutoff is Apr. 1, 2019 · YP 2.7 M4 Release Target is Apr. 26, 2019 Planned upcoming dot releases: · YP 2.4.4 (Rocko) is in QA. See: https://wiki.yoctoproject.org/wiki/2.4_QA_Status · YP 2.5.2 (Sumo) will be targeted after YP 2.4.4 is done. · YP 2.6.1 (Thud) will be targeted after YP 2.7 M1 is done. · YP 2.5.3 (Sumo) will be targeted after YP 2.7 M4 is done. · YP 2.6.2 (Thud) will be targeted after YP 2.5.3 is done. Tracking Metrics: · WDD 2418 (last week 2402) (https://wiki.yoctoproject.org/charts/combo.html) · Poky Patch Metrics oTotal patches found: 1711 (last week 1716) oPercentage of patches in the Pending State: 738 (43%) [last week 739 (43%)] Key Status Links for YP: https://wiki.yoctoproject.org/wiki/Yocto_Project_v2.7_Status https://wiki.yoctoproject.org/wiki/Yocto_2.7_Schedule https://wiki.yoctoproject.org/wiki/Yocto_2.7_Features The Status reports are now stored on the wiki at: https://wiki.yoctoproject.org/wiki/Weekly_Status [If anyone has suggestions for other information you’d like to see on this weekly status update, let us know!] Thanks, Stephen K. Jolley Yocto Project Program Manager INTEL, MS JF1-255, 2111 N.E. 25th Avenue, Hillsboro, OR 97124 •Cell:(208) 244-4460 • Email: stephen.k.jol...@intel.com -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
On Tue, Nov 20, 2018 at 1:40 PM Burton, Ross wrote: > > (hit sent too early) > > Of course how big are these tools, and is it really useful splitting > them up? If we're talking a 10k binary, is putting them in separate > packages worth the complication? I think it makes sense to split; we are building an embedded system build system and granular installation is a must have. Using PN to install all tools is a win win combination. -- Otavio Salvador O.S. Systems http://www.ossystems.com.brhttp://code.ossystems.com.br Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [sumo] cherry-pick request to fix bison on glibc 2.28
Tim Orling writes: > http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=5e441f3f8aa896e264f44ba8b699e5841cceade4 > > or > > http://git.openembedded.org/openembedded-core/commit/?id=741415c58b356576428c26efd081212eb989l +Armin I was going to send a patch with the backport but noticed this email. Would you still prefer a patch anyway? -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
(hit sent too early) Of course how big are these tools, and is it really useful splitting them up? If we're talking a 10k binary, is putting them in separate packages worth the complication? Ross -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
On Tue, 20 Nov 2018 at 14:29, Peter Kjellerstedt wrote: > Actually, why mess with an unnecessary meta package. Just use the > empty ${PN} package for that. That works for the GStreamer packages > as well these days (since commit 8e1009a3), even though they do it > by depending on the meta packages for backwards compatibility. Yes, please. Having a u-boot-tools-meta will mean we're constantly explaining why installing u-boot-tools either doesn't install anything, or produces an error. Ross -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [rocko][PATCH 3/3] elfutils: Fix gcc compile time alignment errors
From: Khem Raj Allow devtool to organize the SRC_URI (From OE-Core rev: 49aae1d75ff1c6a9643c30a8cc5776a2ffa83dd3) Signed-off-by: Khem Raj Signed-off-by: Ross Burton --- .../elfutils/elfutils_0.170.bb| 40 +++-- ...ed-structs-follow-the-gcc-memory-lay.patch | 34 + ...ternatives-for-glibc-assumptions-hel.patch | 144 +- 3 files changed, 124 insertions(+), 94 deletions(-) create mode 100644 meta/recipes-devtools/elfutils/files/0001-Ensure-that-packed-structs-follow-the-gcc-memory-lay.patch diff --git a/meta/recipes-devtools/elfutils/elfutils_0.170.bb b/meta/recipes-devtools/elfutils/elfutils_0.170.bb index 9242aea26b..54d341d190 100644 --- a/meta/recipes-devtools/elfutils/elfutils_0.170.bb +++ b/meta/recipes-devtools/elfutils/elfutils_0.170.bb @@ -5,35 +5,33 @@ LICENSE = "(GPLv3 & Elfutils-Exception)" LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504" DEPENDS = "libtool bzip2 zlib virtual/libintl" DEPENDS_append_libc-musl = " argp-standalone fts " -SRC_URI = "https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2"; +SRC_URI = "https://sourceware.org/elfutils/ftp/${PV}/${BP}.tar.bz2 \ + file://0001-dso-link-change.patch \ + file://0002-Fix-elf_cvt_gunhash-if-dest-and-src-are-same.patch \ + file://0003-fixheadercheck.patch \ + file://0004-Disable-the-test-to-convert-euc-jp.patch \ + file://0005-fix-a-stack-usage-warning.patch \ + file://0006-Fix-build-on-aarch64-musl.patch \ + file://0007-Fix-control-path-where-we-have-str-as-uninitialized-.patch \ + file://0001-libasm-may-link-with-libbz2-if-found.patch \ + file://debian/hppa_backend.diff \ + file://debian/arm_backend.diff \ + file://debian/mips_backend.patch \ + file://debian/mips_readelf_w.patch \ + file://debian/0001-Ignore-differences-between-mips-machine-identifiers.patch \ + file://debian/0002-Add-support-for-mips64-abis-in-mips_retval.c.patch \ + file://debian/0003-Add-mips-n64-relocation-format-hack.patch \ + file://0001-Use-fallthrough-attribute.patch \ + file://0001-Ensure-that-packed-structs-follow-the-gcc-memory-lay.patch \ + " SRC_URI[md5sum] = "03599aee98c9b726c7a732a2dd0245d5" SRC_URI[sha256sum] = "1f844775576b79bdc9f9c717a50058d08620323c1e935458223a12f249c9e066" -SRC_URI += "\ -file://0001-dso-link-change.patch \ -file://0002-Fix-elf_cvt_gunhash-if-dest-and-src-are-same.patch \ -file://0003-fixheadercheck.patch \ -file://0004-Disable-the-test-to-convert-euc-jp.patch \ -file://0005-fix-a-stack-usage-warning.patch \ -file://0006-Fix-build-on-aarch64-musl.patch \ -file://0007-Fix-control-path-where-we-have-str-as-uninitialized-.patch \ -file://0001-libasm-may-link-with-libbz2-if-found.patch \ -" SRC_URI_append_libc-musl = " file://0008-build-Provide-alternatives-for-glibc-assumptions-hel.patch" # Pick patches from debian # http://ftp.de.debian.org/debian/pool/main/e/elfutils/elfutils_0.168-0.2.debian.tar.xz -SRC_URI += "\ -file://debian/hppa_backend.diff \ -file://debian/arm_backend.diff \ -file://debian/mips_backend.patch \ -file://debian/mips_readelf_w.patch \ - file://debian/0001-Ignore-differences-between-mips-machine-identifiers.patch \ -file://debian/0002-Add-support-for-mips64-abis-in-mips_retval.c.patch \ -file://debian/0003-Add-mips-n64-relocation-format-hack.patch \ -" # Fix fallthrough with GCC7 -SRC_URI += "file://0001-Use-fallthrough-attribute.patch" # The buildsystem wants to generate 2 .h files from source using a binary it just built, # which can not pass the cross compiling, so let's work around it by adding 2 .h files diff --git a/meta/recipes-devtools/elfutils/files/0001-Ensure-that-packed-structs-follow-the-gcc-memory-lay.patch b/meta/recipes-devtools/elfutils/files/0001-Ensure-that-packed-structs-follow-the-gcc-memory-lay.patch new file mode 100644 index 00..cf4d1dfe75 --- /dev/null +++ b/meta/recipes-devtools/elfutils/files/0001-Ensure-that-packed-structs-follow-the-gcc-memory-lay.patch @@ -0,0 +1,34 @@ +From 732913a8c35c7b25c0cbf6903cab1ad6b602b525 Mon Sep 17 00:00:00 2001 +From: Khem Raj +Date: Thu, 22 Mar 2018 22:44:03 -0700 +Subject: [PATCH] Ensure that packed structs follow the gcc memory layout + +Partial backport of +https://sourceware.org/git/?p=elfutils.git;a=commit;h=17d7194d291bf91d130b78e06cbe27b290e0376d + +Helps fixing alignment errors e.g. +linux-core-note.c:116:1: error: alignment 2 of 'struct m68k_prstatus +' is less than 4 [-Werror=packed-not-aligned] + ; + ^ + +Upstream-Status: Backport [https://sourceware.org/git/?p=elfutils.git;a=commit;h=17d7194d291bf91d130b78e06cbe27b290e0376d] + +Signed-off-by: Khem Raj +--- + backends/linux-core-note.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +d
[OE-core] [rocko][PATCH 2/3] elfutils: refresh patches
From: Ross Burton The patch tool will apply patches by default with "fuzz", which is where if the hunk context isn't present but what is there is close enough, it will force the patch in. Whilst this is useful when there's just whitespace changes, when applied to source it is possible for a patch applied with fuzz to produce broken code which still compiles (see #10450). This is obviously bad. We'd like to eventually have do_patch() rejecting any fuzz on these grounds. For that to be realistic the existing patches with fuzz need to be rebased and reviewed. (From OE-Core rev: 2526fcfac8e360d5d27f5ebe26608df470b3b84b) Signed-off-by: Ross Burton --- .../elfutils/files/debian/hppa_backend.diff | 46 +-- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/meta/recipes-devtools/elfutils/files/debian/hppa_backend.diff b/meta/recipes-devtools/elfutils/files/debian/hppa_backend.diff index 44fda7f305..56f852a414 100644 --- a/meta/recipes-devtools/elfutils/files/debian/hppa_backend.diff +++ b/meta/recipes-devtools/elfutils/files/debian/hppa_backend.diff @@ -1,10 +1,10 @@ Upstream-Status: Backport [from debian] Signed-off-by: Hongxu Jia -Index: b/backends/parisc_init.c +Index: elfutils-0.170/backends/parisc_init.c === --- /dev/null -+++ b/backends/parisc_init.c elfutils-0.170/backends/parisc_init.c @@ -0,0 +1,73 @@ +/* Initialization of PA-RISC specific backend library. + Copyright (C) 2002, 2005, 2006 Red Hat, Inc. @@ -79,10 +79,10 @@ Index: b/backends/parisc_init.c + + return MODVERSION; +} -Index: b/backends/parisc_regs.c +Index: elfutils-0.170/backends/parisc_regs.c === --- /dev/null -+++ b/backends/parisc_regs.c elfutils-0.170/backends/parisc_regs.c @@ -0,0 +1,159 @@ +/* Register names and numbers for PA-RISC DWARF. + Copyright (C) 2005, 2006 Red Hat, Inc. @@ -243,10 +243,10 @@ Index: b/backends/parisc_regs.c + name[namelen++] = '\0'; + return namelen; +} -Index: b/backends/parisc_reloc.def +Index: elfutils-0.170/backends/parisc_reloc.def === --- /dev/null -+++ b/backends/parisc_reloc.def elfutils-0.170/backends/parisc_reloc.def @@ -0,0 +1,128 @@ +/* List the relocation types for PA-RISC. -*- C -*- + Copyright (C) 2005 Red Hat, Inc. @@ -376,10 +376,10 @@ Index: b/backends/parisc_reloc.def +RELOC_TYPE (TLS_DTPMOD64, DYN) + +#define NO_RELATIVE_RELOC 1 -Index: b/backends/parisc_retval.c +Index: elfutils-0.170/backends/parisc_retval.c === --- /dev/null -+++ b/backends/parisc_retval.c elfutils-0.170/backends/parisc_retval.c @@ -0,0 +1,213 @@ +/* Function return value location for Linux/PA-RISC ABI. + Copyright (C) 2005 Red Hat, Inc. @@ -594,10 +594,10 @@ Index: b/backends/parisc_retval.c + return parisc_return_value_location_ (functypedie, locp, 1); +} + -Index: b/backends/parisc_symbol.c +Index: elfutils-0.170/backends/parisc_symbol.c === --- /dev/null -+++ b/backends/parisc_symbol.c elfutils-0.170/backends/parisc_symbol.c @@ -0,0 +1,112 @@ +/* PA-RISC specific symbolic name handling. + Copyright (C) 2002, 2005 Red Hat, Inc. @@ -711,10 +711,10 @@ Index: b/backends/parisc_symbol.c + return ELF_T_NUM; +} +} -Index: b/backends/libebl_parisc.h +Index: elfutils-0.170/backends/libebl_parisc.h === --- /dev/null -+++ b/backends/libebl_parisc.h elfutils-0.170/backends/libebl_parisc.h @@ -0,0 +1,9 @@ +#ifndef _LIBEBL_HPPA_H +#define _LIBEBL_HPPA_H 1 @@ -725,10 +725,10 @@ Index: b/backends/libebl_parisc.h +extern int parisc_return_value_location_64(Dwarf_Die *, const Dwarf_Op **locp); + +#endif -Index: b/backends/Makefile.am +Index: elfutils-0.170/backends/Makefile.am === a/backends/Makefile.am -+++ b/backends/Makefile.am +--- elfutils-0.170.orig/backends/Makefile.am elfutils-0.170/backends/Makefile.am @@ -33,12 +33,12 @@ AM_CPPFLAGS += -I$(top_srcdir)/libebl -I @@ -744,7 +744,7 @@ Index: b/backends/Makefile.am noinst_LIBRARIES = $(libebl_pic) noinst_DATA = $(libebl_pic:_pic.a=.so) -@@ -128,6 +128,9 @@ endif +@@ -124,6 +124,9 @@ cpu_bpf = ../libcpu/libcpu_bpf.a libebl_bpf_pic_a_SOURCES = $(bpf_SRCS) am_libebl_bpf_pic_a_OBJECTS = $(bpf_SRCS:.c=.os) @@ -752,13 +752,13 @@ Index: b/backends/Makefile.am +libebl_parisc_pic_a_SOURCES = $(parisc_SRCS) +am_libebl_parisc_pic_a_OBJECTS = $(parisc_SRCS:.c=.os) - libebl_%.so libebl_%.map: libebl_%_pic.a $(libelf) $(libdw) + libebl_%.so libebl_%.map: libebl_%_pic.a $(libelf) $(libdw) $(libeu) @rm -f $(@:.so=.map) -Index: b/libelf/elf.h +Index: elfutils
[OE-core] [rocko][PATCH 1/3] elfutils: Use fallthrough attribute
From: Joshua Watt Patches elfutils to use the fallthrough attribute instead of comments to satisfy the -Wimplicit-fallthrough warning. Using comments is insufficient when compiling remotely with Icecream because the file gets pre-processed locally, removing the comments (From OE-Core rev: cd44cee91b5b17ddf617950d84513d481ab34f58) Signed-off-by: Joshua Watt Signed-off-by: Richard Purdie --- .../elfutils/elfutils_0.170.bb| 4 +- .../0001-Use-fallthrough-attribute.patch | 791 ++ .../elfutils/files/debian/fallthrough.patch | 36 - 3 files changed, 793 insertions(+), 38 deletions(-) create mode 100644 meta/recipes-devtools/elfutils/files/0001-Use-fallthrough-attribute.patch delete mode 100644 meta/recipes-devtools/elfutils/files/debian/fallthrough.patch diff --git a/meta/recipes-devtools/elfutils/elfutils_0.170.bb b/meta/recipes-devtools/elfutils/elfutils_0.170.bb index 3b81e287b0..9242aea26b 100644 --- a/meta/recipes-devtools/elfutils/elfutils_0.170.bb +++ b/meta/recipes-devtools/elfutils/elfutils_0.170.bb @@ -32,8 +32,8 @@ SRC_URI += "\ file://debian/0002-Add-support-for-mips64-abis-in-mips_retval.c.patch \ file://debian/0003-Add-mips-n64-relocation-format-hack.patch \ " -# Fix the patches from Debian with GCC7 -SRC_URI += "file://debian/fallthrough.patch" +# Fix fallthrough with GCC7 +SRC_URI += "file://0001-Use-fallthrough-attribute.patch" # The buildsystem wants to generate 2 .h files from source using a binary it just built, # which can not pass the cross compiling, so let's work around it by adding 2 .h files diff --git a/meta/recipes-devtools/elfutils/files/0001-Use-fallthrough-attribute.patch b/meta/recipes-devtools/elfutils/files/0001-Use-fallthrough-attribute.patch new file mode 100644 index 00..5e2155b3c6 --- /dev/null +++ b/meta/recipes-devtools/elfutils/files/0001-Use-fallthrough-attribute.patch @@ -0,0 +1,791 @@ +From 5cb883f67d00a63531ef195c242763d36b1905ca Mon Sep 17 00:00:00 2001 +From: Joshua Watt +Date: Fri, 9 Feb 2018 12:46:38 -0600 +Subject: [PATCH] Use fallthrough attribute + +Use __attribute__ ((fallthrough)) to indicate switch case fall through +instead of a comment. This ensure that the fallthrough warning is not +triggered even if the file is pre-processed (hence stripping the +comments) before it is compiled. + +The actual fallback implementation is hidden behind a FALLBACK macro in +case the compiler doesn't support it. + +Finally, the -Wimplict-fallthrough warning was upgraded to only allow +the attribute to satisfy it; a comment alone is no longer sufficient. + +Signed-off-by: Joshua Watt + +Upstream-Status: Submitted [1] + +[1] https://sourceware.org/ml/elfutils-devel/2018-q1/msg00027.html +--- + backends/aarch64_retval.c| 2 +- + backends/alpha_retval.c | 4 ++-- + backends/arm_regs.c | 2 +- + backends/arm_retval.c| 2 +- + backends/i386_regs.c | 2 +- + backends/i386_retval.c | 4 ++-- + backends/ia64_retval.c | 2 +- + backends/linux-core-note.c | 2 +- + backends/m68k_retval.c | 4 ++-- + backends/mips_retval.c | 4 ++-- + backends/parisc_retval.c | 5 +++-- + backends/ppc64_retval.c | 6 +++--- + backends/ppc_regs.c | 2 +- + backends/ppc_retval.c| 4 ++-- + backends/s390_retval.c | 4 ++-- + backends/sh_retval.c | 2 +- + backends/sparc_retval.c | 2 +- + backends/tilegx_retval.c | 4 ++-- + backends/x86_64_regs.c | 2 +- + backends/x86_64_retval.c | 2 +- + config/eu.am | 4 +++- + configure.ac | 6 ++ + lib/eu-config.h | 7 +++ + libcpu/i386_disasm.c | 2 +- + libcpu/i386_parse.c | 4 ++-- + libdw/cfi.c | 4 ++-- + libdw/dwarf_frame_register.c | 2 +- + libdwfl/dwfl_report_elf.c| 2 +- + libdwfl/frame_unwind.c | 2 +- + libebl/eblobjnote.c | 2 +- + libelf/elf32_updatenull.c| 2 +- + libelf/elf_begin.c | 4 ++-- + libelf/elf_cntl.c| 2 +- + src/addr2line.c | 2 +- + src/elfcompress.c| 2 +- + src/elflint.c| 8 + src/objdump.c| 2 +- + src/readelf.c| 8 + src/strings.c| 2 +- + tests/backtrace.c| 2 +- + tests/elfstrmerge.c | 3 ++- + 41 files changed, 75 insertions(+), 58 deletions(-) + +diff --git a/backends/aarch64_retval.c b/backends/aarch64_retval.c +index 68de307..1308340 100644 +--- a/backends/aarch64_retval.c b/backends/aarch64_retval.c +@@ -292,7 +292,7 @@ aarch64_return_value_location (Dwarf_Die *functypedie, const Dwarf_Op **locp) + assert (count > 0); + if (count <= 4) + return pass_hfa (locp, base_size, count); +-/* Fall through. */ ++FALLTHROUGH; + + case 1: + /* Not a HFA. */ +diff --git a/backends/alpha_retval.c b/backends/alpha_retval.c +index 53dbfa4..d9bae3b 100644 +
[OE-core] [rocko][PATCH 0/3] elfutils: Backport gcc alignment error fix
This became a patch series because there are two intermediate commits that were not part of rocko but are needed for a painless cherry-pick of the commit I'm really interested in. Without these commits, I am unable to build elfutils-native on Fedora 29. -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH] xcb-proto: use python3native to have reproducible pyc files
Signed-off-by: Ross Burton --- meta/recipes-graphics/xorg-proto/xcb-proto_1.13.bb | 26 ++ 1 file changed, 2 insertions(+), 24 deletions(-) diff --git a/meta/recipes-graphics/xorg-proto/xcb-proto_1.13.bb b/meta/recipes-graphics/xorg-proto/xcb-proto_1.13.bb index d5037868d54..74670909201 100644 --- a/meta/recipes-graphics/xorg-proto/xcb-proto_1.13.bb +++ b/meta/recipes-graphics/xorg-proto/xcb-proto_1.13.bb @@ -12,40 +12,18 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=d763b081cb10c223435b01e00dc0aba7 \ file://src/dri2.xml;beginline=2;endline=28;md5=f8763b13ff432e8597e0d610cf598e65" SRC_URI = "http://xcb.freedesktop.org/dist/${BP}.tar.bz2"; - SRC_URI[md5sum] = "abe9aa4886138150bbc04ae4f29b90e3" SRC_URI[sha256sum] = "7b98721e669be80284e9bbfeab02d2d0d54cd11172b72271e47a2fe875e2bde1" -inherit autotools pkgconfig - -# Force the use of Python 3 and a specific library path so we don't need to -# depend on python3-native -CACHED_CONFIGUREVARS += "PYTHON=python3 am_cv_python_pythondir=${libdir}/xcb-proto" +inherit autotools pkgconfig python3native PACKAGES += "python-xcbgen" FILES_${PN} = "" FILES_${PN}-dev += "${datadir}/xcb/*.xml ${datadir}/xcb/*.xsd" -FILES_python-xcbgen = "${libdir}/xcb-proto" +FILES_python-xcbgen = "${PYTHON_SITEPACKAGES_DIR}" RDEPENDS_${PN}-dev = "" RRECOMMENDS_${PN}-dbg = "${PN}-dev (= ${EXTENDPKGV})" BBCLASSEXTEND = "native nativesdk" - -# Need to do this dance because we're forcing the use of host Python above and -# if xcb-proto is built with Py3.5 and then re-used from sstate on a host with -# Py3.6 the second build will write new cache files into the sysroot which won't -# be listed in the manifest so won't be deleted, resulting in an error on -# rebuilds. Solve this by deleting the entire cache directory when this package -# is removed from the sysroot. -SSTATEPOSTINSTFUNCS += "xcb_sstate_postinst" -xcb_sstate_postinst() { - if [ "${BB_CURRENTTASK}" = "populate_sysroot" -o "${BB_CURRENTTASK}" = "populate_sysroot_setscene" ] - then - cat <${SSTATE_INST_POSTRM} -#!/bin/sh -rm -rf ${libdir}/xcb-proto/xcbgen/__pycache__ -EOF - fi -} -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH] json-glib: add ptest
Signed-off-by: Ross Burton --- meta/recipes-gnome/json-glib/json-glib/run-ptest | 3 +++ meta/recipes-gnome/json-glib/json-glib_1.4.2.bb | 12 +++- 2 files changed, 10 insertions(+), 5 deletions(-) create mode 100644 meta/recipes-gnome/json-glib/json-glib/run-ptest diff --git a/meta/recipes-gnome/json-glib/json-glib/run-ptest b/meta/recipes-gnome/json-glib/json-glib/run-ptest new file mode 100644 index 000..0c6398a177d --- /dev/null +++ b/meta/recipes-gnome/json-glib/json-glib/run-ptest @@ -0,0 +1,3 @@ +#! /bin/sh + +gnome-desktop-testing-runner json-glib diff --git a/meta/recipes-gnome/json-glib/json-glib_1.4.2.bb b/meta/recipes-gnome/json-glib/json-glib_1.4.2.bb index 19fe3cc4427..b4a6a6f2b63 100644 --- a/meta/recipes-gnome/json-glib/json-glib_1.4.2.bb +++ b/meta/recipes-gnome/json-glib/json-glib_1.4.2.bb @@ -10,12 +10,13 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=7fbc338309ac38fefcd64b04bb903e34" DEPENDS = "glib-2.0" +GNOMEBASEBUILDCLASS = "meson" +inherit gnomebase lib_package gobject-introspection gtk-doc gettext ptest-gnome + +SRC_URI += "file://run-ptest" SRC_URI[archive.md5sum] = "35107e23a7bbbc70f31c34f7b9adf1c3" SRC_URI[archive.sha256sum] = "2d7709a44749c7318599a6829322e081915bdc73f5be5045882ed120bb686dc8" -GNOMEBASEBUILDCLASS = "meson" -inherit gnomebase lib_package gobject-introspection gtk-doc gettext - # This builds both API docs (via gtk-doc) and manpages GTKDOC_ENABLE_FLAG = "-Ddocs=true" GTKDOC_DISABLE_FLAG = "-Ddocs=false" @@ -29,8 +30,9 @@ EXTRA_OEMESON_append_class-target = " ${@bb.utils.contains('GI_DATA_ENABLED', 'T '${GI_DISABLE_FLAG}', d)} " do_install_append() { -# FIXME: these need to be provided via ptest -rm -rf ${D}${datadir}/installed-tests ${D}${libexecdir} + if ! ${@bb.utils.contains('PTEST_ENABLED', '1', 'true', 'false', d)}; then + rm -rf ${D}${datadir}/installed-tests ${D}${libexecdir} + fi } BBCLASSEXTEND = "native nativesdk" -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [rocko][PATCH] bison: Fix build break with glibc 2.28
From: Khem Raj (From OE-Core rev: 741415c58b356576428c26efd081212eb989) Signed-off-by: Khem Raj Signed-off-by: Richard Purdie --- .../recipes-devtools/bison/bison/gnulib.patch | 21 +++ meta/recipes-devtools/bison/bison_3.0.4.bb| 1 + 2 files changed, 22 insertions(+) create mode 100644 meta/recipes-devtools/bison/bison/gnulib.patch diff --git a/meta/recipes-devtools/bison/bison/gnulib.patch b/meta/recipes-devtools/bison/bison/gnulib.patch new file mode 100644 index 00..7eaf0ce0fa --- /dev/null +++ b/meta/recipes-devtools/bison/bison/gnulib.patch @@ -0,0 +1,21 @@ +Fix gnulib issues found with glibc 2.28 libio.h removal + +see +https://lists.gnu.org/r/bug-gnulib/2018-03/msg0.html + +Upstream-Status: Pending +Signed-off-by: Khem Raj + +Index: bison-3.0.4/lib/fseterr.c +=== +--- bison-3.0.4.orig/lib/fseterr.c bison-3.0.4/lib/fseterr.c +@@ -29,7 +29,7 @@ fseterr (FILE *fp) + /* Most systems provide FILE as a struct and the necessary bitmask in + , because they need it for implementing getc() and putc() as + fast macros. */ +-#if defined _IO_ftrylockfile || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, Linux libc5 */ ++#if defined _IO_EOF_SEEN || __GNU_LIBRARY__ == 1 /* GNU libc, BeOS, Haiku, Linux libc5 */ + fp->_flags |= _IO_ERR_SEEN; + #elif defined __sferror || defined __DragonFly__ /* FreeBSD, NetBSD, OpenBSD, DragonFly, Mac OS X, Cygwin */ + fp_->_flags |= __SERR; diff --git a/meta/recipes-devtools/bison/bison_3.0.4.bb b/meta/recipes-devtools/bison/bison_3.0.4.bb index 7d066be0ad..a86549c086 100644 --- a/meta/recipes-devtools/bison/bison_3.0.4.bb +++ b/meta/recipes-devtools/bison/bison_3.0.4.bb @@ -15,6 +15,7 @@ SRC_URI = "${GNU_MIRROR}/bison/bison-${PV}.tar.xz \ file://dont-depend-on-help2man.patch.patch \ file://0001-src-local.mk-fix-parallel-issue.patch \ file://add-with-bisonlocaledir.patch \ + file://gnulib.patch \ " # No point in hardcoding path to m4, just use PATH -- 2.19.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool
> -Original Message- > From: openembedded-core-boun...@lists.openembedded.org core-boun...@lists.openembedded.org> On Behalf Of Otavio Salvador > Sent: den 19 november 2018 20:22 > To: Tom Rini > Cc: Marek Vasut ; linux-snps-...@lists.infradead.org; > Alexey Brodkin ; Otavio Salvador > ; Patches and discussions about the oe-core > layer > Subject: Re: [OE-core] [PATCH v3] u-boot: Add mkenvimage tool > > On Mon, Nov 19, 2018 at 5:08 PM Tom Rini wrote: > > > > On Mon, Nov 19, 2018 at 01:51:41PM +0100, Marek Vasut wrote: > > > On 11/19/2018 11:04 AM, Alexey Brodkin wrote: > > > > This utility is used for creation of images containing > > > > usable in run-time U-Boot environment. > > > > > > > > As of today this utility is added per-board like here [1] > > > > for Intel Edison board. > > > > > > > > [1] http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel- > edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot- > tools_2014.04.bb > > > > > > > > Given there're quite some U-Boot tools that we may want to add > later > > > > this recipe name switch from "u-boot-mkimage" to generic "u-boot- > tools" > > > > still for compatibility we provide "u-boot-mkimage" with help > > > > of PROVIDES as well as proposed "u-boot-mkenvimage". > > > > > > > > Signed-off-by: Alexey Brodkin > > > > Cc: Richard Purdie > > > > Cc: Otavio Salvador > > > > Cc: Martin Jansa > > > > Cc: Ross Burton > > > > Cc: Marek Vasut > > > > > > Do we want to generate one package per tool or not ? > > > > I think so, yes. Not all places will want all tools. > > And a bonus to add a u-boot-tools-meta which rdepends on all. You can > copy code to do that from gstreamer-plugins packages. Actually, why mess with an unnecessary meta package. Just use the empty ${PN} package for that. That works for the GStreamer packages as well these days (since commit 8e1009a3), even though they do it by depending on the meta packages for backwards compatibility. //Peter > -- > Otavio Salvador O.S. Systems > http://www.ossystems.com.brhttp://code.ossystems.com.br > Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] musl: Update to latest trunk
On Tue, Nov 20, 2018 at 6:08 AM akuster808 wrote: > > > > On 11/19/18 9:27 AM, Khem Raj wrote: > > Complete changelogs are here > > https://git.musl-libc.org/cgit/musl/log/?qt=range&q=c50985d5c8e316c5c464f352e79eeebfed1121a9..79f653c6bc2881dd6855299c908a442f56cb7c2b > > > > Signed-off-by: Khem Raj > > --- > > meta/recipes-core/musl/musl_git.bb | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/meta/recipes-core/musl/musl_git.bb > > b/meta/recipes-core/musl/musl_git.bb > > index 0d8f8eb2a4..f2b6361446 100644 > > --- a/meta/recipes-core/musl/musl_git.bb > > +++ b/meta/recipes-core/musl/musl_git.bb > > @@ -4,7 +4,7 @@ > > require musl.inc > > inherit linuxloader > > > > -SRCREV = "c50985d5c8e316c5c464f352e79eeebfed1121a9" > > +SRCREV = "79f653c6bc2881dd6855299c908a442f56cb7c2b" > > > > PV = "1.1.20+git${SRCPV}" > still 1.1.20 ? yes > > > > > -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] apt: update SRC_URI
On Mon, 2018-11-19 at 21:54 +1100, Jonathan Liu wrote: > -SRC_URI = " > > http://archive.ubuntu.com/ubuntu/pool/main/a/${BPN}/${BPN}_${PV}.tar.xz > > \ > > +SRC_URI = " > > https://launchpad.net/ubuntu/+archive/primary/+sourcefiles/apt/1.2.24/${BPN}_${PV}.tar.xz > > \ > > Why don't you replace apt with ${BPN} and 1.2.24 with ${PV} in > SRC_URI? I tweaked this since it was easier than trying to remember to track this :) Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] musl: Update to latest trunk
On 11/19/18 9:27 AM, Khem Raj wrote: > Complete changelogs are here > https://git.musl-libc.org/cgit/musl/log/?qt=range&q=c50985d5c8e316c5c464f352e79eeebfed1121a9..79f653c6bc2881dd6855299c908a442f56cb7c2b > > Signed-off-by: Khem Raj > --- > meta/recipes-core/musl/musl_git.bb | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meta/recipes-core/musl/musl_git.bb > b/meta/recipes-core/musl/musl_git.bb > index 0d8f8eb2a4..f2b6361446 100644 > --- a/meta/recipes-core/musl/musl_git.bb > +++ b/meta/recipes-core/musl/musl_git.bb > @@ -4,7 +4,7 @@ > require musl.inc > inherit linuxloader > > -SRCREV = "c50985d5c8e316c5c464f352e79eeebfed1121a9" > +SRCREV = "79f653c6bc2881dd6855299c908a442f56cb7c2b" > > PV = "1.1.20+git${SRCPV}" still 1.1.20 ? > -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [rocko][sumo][PATCH 1/1] mobile-broadband-provider: fix SRC_URI
Ping. Any comments on this backport request? Max Am Mi., 7. Nov. 2018 um 14:25 Uhr schrieb Max Krummenacher < max.oss...@gmail.com>: > From: Anuj Mittal > > Fixes build for older versions of git which don't follow redirects > properly if the .git suffix is missing. > > Signed-off-by: Anuj Mittal > Signed-off-by: Ross Burton > (cherry picked from commit 77c353cc9bdfbf1b6453b2579b72726db05eb69c) > [Fixup for rocko/sumo context] > Signed-off-by: Max Krummenacher > --- > .../mobile-broadband-provider-info_git.bb | > 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > The inital patch 8382cdc088 changing the gnome git repo URL was > backparted to rocko and sumo, however the follow up patch was not. > > This provides the backport of 77c353cc9b. > > Max > > diff --git a/meta/recipes-connectivity/mobile-broadband-provider-info/ > mobile-broadband-provider-info_git.bb > b/meta/recipes-connectivity/mobile-broadband-provider-info/ > mobile-broadband-provider-info_git.bb > index 57f521a6c4..b595d5700c 100644 > --- a/meta/recipes-connectivity/mobile-broadband-provider-info/ > mobile-broadband-provider-info_git.bb > +++ b/meta/recipes-connectivity/mobile-broadband-provider-info/ > mobile-broadband-provider-info_git.bb > @@ -7,8 +7,7 @@ SRCREV = "befcbbc9867e742ac16415660b0b7521218a530c" > PV = "20170310" > PE = "1" > > -SRC_URI = "git:// > gitlab.gnome.org/GNOME/mobile-broadband-provider-info;protocol=https" > - > +SRC_URI = "git:// > gitlab.gnome.org/GNOME/mobile-broadband-provider-info.git;protocol=https" > S = "${WORKDIR}/git" > > inherit autotools > -- > 2.13.6 > > -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [OE-Core][thud][PATCH] systemd: backport fix to stop enabling ECN
>From upstream: Turning on ECN still causes slow or broken network on linux. Our tcp is not yet ready for wide spread use of ECN. https://github.com/systemd/systemd/issues/9748 Signed-off-by: Alex Kiernan Signed-off-by: Richard Purdie --- ...tl.d-request-ECN-on-both-in-and-outgoing-.patch | 36 ++ meta/recipes-core/systemd/systemd_239.bb | 1 + 2 files changed, 37 insertions(+) create mode 100644 meta/recipes-core/systemd/systemd/0001-Revert-sysctl.d-request-ECN-on-both-in-and-outgoing-.patch diff --git a/meta/recipes-core/systemd/systemd/0001-Revert-sysctl.d-request-ECN-on-both-in-and-outgoing-.patch b/meta/recipes-core/systemd/systemd/0001-Revert-sysctl.d-request-ECN-on-both-in-and-outgoing-.patch new file mode 100644 index ..7fd6d0188e90 --- /dev/null +++ b/meta/recipes-core/systemd/systemd/0001-Revert-sysctl.d-request-ECN-on-both-in-and-outgoing-.patch @@ -0,0 +1,36 @@ +From 1e190dfd5bb95036f937ef1dc46f43eb0a146612 Mon Sep 17 00:00:00 2001 +From: Thomas Hindoe Paaboel Andersen +Date: Fri, 17 Aug 2018 21:31:05 +0200 +Subject: [PATCH] Revert "sysctl.d: request ECN on both in and outgoing + connections" + +Turning on ECN still causes slow or broken network on linux. Our tcp +is not yet ready for wide spread use of ECN. + +This reverts commit 919472741dba6ad0a3f6c2b76d390a02d0e2fdc3. + +https://github.com/systemd/systemd/issues/9748 + +Upstream-Status: Backport +Signed-off-by: Alex Kiernan +--- + sysctl.d/50-default.conf | 3 --- + 1 file changed, 3 deletions(-) + +diff --git a/sysctl.d/50-default.conf b/sysctl.d/50-default.conf +index b67ae87ca6b7..e263cf06284a 100644 +--- a/sysctl.d/50-default.conf b/sysctl.d/50-default.conf +@@ -33,9 +33,6 @@ net.ipv4.conf.all.promote_secondaries = 1 + # Fair Queue CoDel packet scheduler to fight bufferbloat + net.core.default_qdisc = fq_codel + +-# Request Explicit Congestion Notification (ECN) on both in and outgoing connections +-net.ipv4.tcp_ecn = 1 +- + # Enable hard and soft link protection + fs.protected_hardlinks = 1 + fs.protected_symlinks = 1 +-- +2.7.4 + diff --git a/meta/recipes-core/systemd/systemd_239.bb b/meta/recipes-core/systemd/systemd_239.bb index 3dbeaac3427a..e19a8babced6 100644 --- a/meta/recipes-core/systemd/systemd_239.bb +++ b/meta/recipes-core/systemd/systemd_239.bb @@ -33,6 +33,7 @@ SRC_URI += "file://touchscreen.rules \ file://0001-core-when-deserializing-state-always-use-read_line-L.patch \ file://0001-chown-recursive-let-s-rework-the-recursive-logic-to-.patch \ file://0001-dhcp6-make-sure-we-have-enough-space-for-the-DHCP6-o.patch \ + file://0001-Revert-sysctl.d-request-ECN-on-both-in-and-outgoing-.patch \ " # patches made for musl are only applied on TCLIBC is musl -- 2.7.4 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/1] nativesdk/sdk: Fix sdk dummy providers for opkg
On Tue, 2018-11-20 at 12:17 +0100, Andrej Valek wrote: > > ERROR: OE-core's config sanity checker detected a potential > > misconfiguration. > > Either fix the cause of this error or at your own risk disable > > the checker (see sanity.conf). > > Following is the list of potential problems / advisories: > > > > Required perl module(s) not found: Text::ParseWords > > Thread::Queue > > Data::Dumper > > It means that perl wasn't excluded from buildtools-tarball. > > Signed-off-by: Andrej Valek > --- > meta/recipes-core/meta/dummy-sdk-package.inc | 8 ++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/meta/recipes-core/meta/dummy-sdk-package.inc > b/meta/recipes-core/meta/dummy-sdk-package.inc > index eafcb823ab..44fb1cb714 100644 > --- a/meta/recipes-core/meta/dummy-sdk-package.inc > +++ b/meta/recipes-core/meta/dummy-sdk-package.inc > @@ -20,7 +20,11 @@ PR[vardeps] += "DUMMYPROVIDES" > python populate_packages_prepend() { > p = d.getVar("PN") > d.appendVar("RPROVIDES_%s" % p, "${DUMMYPROVIDES}") > -#d.appendVar("RCONFLICTS_%s" % p, "${DUMMYPROVIDES}") > -#d.appendVar("RREPLACES_%s" % p, "${DUMMYPROVIDES}") > + > +# opkg needs some additional package dependencies handling > +pkg_type = d.getVar("IMAGE_PKGTYPE") > +if pkg_type == "ipk": > +d.appendVar("RCONFLICTS_%s" % p, "${DUMMYPROVIDES}") > +d.appendVar("RREPLACES_%s" % p, "${DUMMYPROVIDES}") > } Sadly we can't do this, both do_package_write_ipk and do_package_write_rpm depend on do_package and this would cause do_package to change depending on the package backend selected which would mean inconsistently generated packages. We're going to have to find a better solution (and maybe add do_populate_sdk testing to the nightly-packagemanagers test to ensure that doesn't regress). Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 0/1] nativesdk/sdk: Fix sdk dummy providers for opkg
I can't verify functionality of another package types like deb. But condtional could be changed to "not rpm", which covers the rest. Andrej Valek (1): nativesdk/sdk: Fix sdk dummy providers for opkg meta/recipes-core/meta/dummy-sdk-package.inc | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 1/1] nativesdk/sdk: Fix sdk dummy providers for opkg
| ERROR: OE-core's config sanity checker detected a potential | misconfiguration. | Either fix the cause of this error or at your own risk disable | the checker (see sanity.conf). | Following is the list of potential problems / advisories: | | Required perl module(s) not found: Text::ParseWords Thread::Queue | Data::Dumper It means that perl wasn't excluded from buildtools-tarball. Signed-off-by: Andrej Valek --- meta/recipes-core/meta/dummy-sdk-package.inc | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/meta/recipes-core/meta/dummy-sdk-package.inc b/meta/recipes-core/meta/dummy-sdk-package.inc index eafcb823ab..44fb1cb714 100644 --- a/meta/recipes-core/meta/dummy-sdk-package.inc +++ b/meta/recipes-core/meta/dummy-sdk-package.inc @@ -20,7 +20,11 @@ PR[vardeps] += "DUMMYPROVIDES" python populate_packages_prepend() { p = d.getVar("PN") d.appendVar("RPROVIDES_%s" % p, "${DUMMYPROVIDES}") -#d.appendVar("RCONFLICTS_%s" % p, "${DUMMYPROVIDES}") -#d.appendVar("RREPLACES_%s" % p, "${DUMMYPROVIDES}") + +# opkg needs some additional package dependencies handling +pkg_type = d.getVar("IMAGE_PKGTYPE") +if pkg_type == "ipk": +d.appendVar("RCONFLICTS_%s" % p, "${DUMMYPROVIDES}") +d.appendVar("RREPLACES_%s" % p, "${DUMMYPROVIDES}") } -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 1/2] harfbuzz: upgrade 1.9.0 -> 2.1.3
Signed-off-by: Ross Burton --- .../harfbuzz/{harfbuzz_1.9.0.bb => harfbuzz_2.1.3.bb} | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) rename meta/recipes-graphics/harfbuzz/{harfbuzz_1.9.0.bb => harfbuzz_2.1.3.bb} (90%) diff --git a/meta/recipes-graphics/harfbuzz/harfbuzz_1.9.0.bb b/meta/recipes-graphics/harfbuzz/harfbuzz_2.1.3.bb similarity index 90% rename from meta/recipes-graphics/harfbuzz/harfbuzz_1.9.0.bb rename to meta/recipes-graphics/harfbuzz/harfbuzz_2.1.3.bb index 0b276348305..235f8d4a3b7 100644 --- a/meta/recipes-graphics/harfbuzz/harfbuzz_1.9.0.bb +++ b/meta/recipes-graphics/harfbuzz/harfbuzz_2.1.3.bb @@ -11,8 +11,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=e021dd6dda6ff1e6b1044002fc662b9b \ DEPENDS = "glib-2.0 cairo fontconfig freetype" SRC_URI = "http://www.freedesktop.org/software/harfbuzz/release/${BP}.tar.bz2"; -SRC_URI[md5sum] = "263a3f1cb014e3ad2e4123c238a9bfc5" -SRC_URI[sha256sum] = "11eca62bf0ac549b8d6be55f4e130946399939cdfe7a562fdaee711190248b00" +SRC_URI[md5sum] = "38b0a17fca5ae38ff54fc22939ded371" +SRC_URI[sha256sum] = "613264460bb6814c3894e3953225c5357402915853a652d40b4230ce5faf0bee" inherit autotools pkgconfig lib_package gtk-doc -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH 2/2] libinput: 1.12.1 -> 1.12.3
Signed-off-by: Ross Burton --- .../wayland/{libinput_1.12.1.bb => libinput_1.12.3.bb}| 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) rename meta/recipes-graphics/wayland/{libinput_1.12.1.bb => libinput_1.12.3.bb} (86%) diff --git a/meta/recipes-graphics/wayland/libinput_1.12.1.bb b/meta/recipes-graphics/wayland/libinput_1.12.3.bb similarity index 86% rename from meta/recipes-graphics/wayland/libinput_1.12.1.bb rename to meta/recipes-graphics/wayland/libinput_1.12.3.bb index 2cba80f211f..3d1ab59309e 100644 --- a/meta/recipes-graphics/wayland/libinput_1.12.1.bb +++ b/meta/recipes-graphics/wayland/libinput_1.12.3.bb @@ -9,8 +9,8 @@ DEPENDS = "libevdev udev mtdev" SRC_URI = "http://www.freedesktop.org/software/${BPN}/${BP}.tar.xz \ " -SRC_URI[md5sum] = "0b6fff34e30cf7aafc33c8f884806960" -SRC_URI[sha256sum] = "570c48cc7b744b9b4da711ed0c5e8c3de132484684b0c4f0309a9a83df5e8692" +SRC_URI[md5sum] = "083d6bf311d5e1e3a62440dc634bb8f0" +SRC_URI[sha256sum] = "f266ff861a2e7a97e1402c4345e3547b900efde68adbd40e56403e2617fee255" UPSTREAM_CHECK_REGEX = "libinput-(?P\d+\.\d+\.(?!9\d+)\d+)" inherit meson pkgconfig lib_package -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 4/6] librepo: update to 1.9.2
That's true: https://github.com/rpm-software-management/librepo/commit/acbbfe2e76ca1c74c2a1acb25813077d47481f14 You can amend the patch, or I can resend. Alex On Mon, 19 Nov 2018 at 22:01, Burton, Ross wrote: > > On Mon, 19 Nov 2018 at 15:09, Alexander Kanavin > wrote: > > > -DEPENDS = "curl expat glib-2.0 openssl attr gpgme" > > +DEPENDS = "curl expat glib-2.0 openssl attr gpgme libxml2" > > packages/corei7-64-poky-linux/librepo/librepo: RDEPENDS: removed > "expat (['>= 2.2.6'])", added "libxml2 (['>= 2.9.8'])" > > So presumably expat isn't needed as it didn't make it to RDEPENDS? > > Ross -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 5/8] busybox: use example.org instead of google.com in wget test case
Ignore me, you did :) On Tue, 20 Nov 2018 at 11:10, Burton, Ross wrote: > > My point is, if there's a better host then lets send this upstream. > > Ross > On Tue, 20 Nov 2018 at 01:45, ChenQi wrote: > > > > On 11/16/2018 01:43 AM, Burton, Ross wrote: > > > On Thu, 15 Nov 2018 at 01:57, Chen Qi wrote: > > >> Use example.org to ensure it's always reachible. > > > Is this because of China's firewall? You'd think that google.com > > > would have excellent uptime. > > > > > > Ross > > > > > > > I guess so. > > I agree that google.com has excellent uptime. However, I think > > example.org might be better for ptest. > > > > Best Regards, > > Chen Qi -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 5/8] busybox: use example.org instead of google.com in wget test case
My point is, if there's a better host then lets send this upstream. Ross On Tue, 20 Nov 2018 at 01:45, ChenQi wrote: > > On 11/16/2018 01:43 AM, Burton, Ross wrote: > > On Thu, 15 Nov 2018 at 01:57, Chen Qi wrote: > >> Use example.org to ensure it's always reachible. > > Is this because of China's firewall? You'd think that google.com > > would have excellent uptime. > > > > Ross > > > > I guess so. > I agree that google.com has excellent uptime. However, I think > example.org might be better for ptest. > > Best Regards, > Chen Qi -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH] apt: Improve SRC_URI
Use PV and BPN in SRC_URI as a minor improvement. Signed-off-by: Richard Purdie --- meta/recipes-devtools/apt/apt.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-devtools/apt/apt.inc b/meta/recipes-devtools/apt/apt.inc index 50f22c89923..bff59e521ad 100644 --- a/meta/recipes-devtools/apt/apt.inc +++ b/meta/recipes-devtools/apt/apt.inc @@ -2,7 +2,7 @@ SUMMARY = "Advanced front-end for dpkg" LICENSE = "GPLv2.0+" SECTION = "base" -SRC_URI = "https://launchpad.net/ubuntu/+archive/primary/+sourcefiles/apt/1.2.24/${BPN}_${PV}.tar.xz \ +SRC_URI = "https://launchpad.net/ubuntu/+archive/primary/+sourcefiles/${BPN}/${PV}/${BPN}_${PV}.tar.xz \ file://use-host.patch \ file://makerace.patch \ file://no-nls-dpkg.patch \ -- 2.17.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [V2 PATCH 1/1] ltp: Use a legal bad address for mips in setrlimit05.c
This testcase fails on mips32. The process is killed by SIGBUS which is not as expect. This is because: ((void *)-1) is not a legal bad address which causes the process killed by SIGBUG on mips. 'tst_get_bad_addr()' returns an address that should works on mips and other arches. Signed-off-by: Hongzhi.Song --- ...t05-Use-another-method-to-get-bad-address.patch | 61 ++ meta/recipes-extended/ltp/ltp_20180926.bb | 1 + 2 files changed, 62 insertions(+) create mode 100644 meta/recipes-extended/ltp/ltp/0001-setrlimit05-Use-another-method-to-get-bad-address.patch diff --git a/meta/recipes-extended/ltp/ltp/0001-setrlimit05-Use-another-method-to-get-bad-address.patch b/meta/recipes-extended/ltp/ltp/0001-setrlimit05-Use-another-method-to-get-bad-address.patch new file mode 100644 index 00..4bfe2767d8 --- /dev/null +++ b/meta/recipes-extended/ltp/ltp/0001-setrlimit05-Use-another-method-to-get-bad-address.patch @@ -0,0 +1,61 @@ +From c4d02b23d12b7136b5739b3075e817fc037e41d4 Mon Sep 17 00:00:00 2001 +From: "Hongzhi.Song" +Date: Mon, 19 Nov 2018 23:07:01 -0800 +Subject: [PATCH] setrlimit05: Use another method to get bad address + +This testcase fails on mips32. The process is killed by SIGBUS which +is not as expect. + +This is because: +((void *)-1) is not a legal bad address which causes the process +killed by SIGBUG on mips. + +'tst_get_bad_addr()' returns an address that should works on mips +and other arches. + +Upstream-Status: Accepted [80bed467bc6ab48a6cd88a8ab74ca15d08830cb0] + +Signed-off-by: Hongzhi.Song +--- + testcases/kernel/syscalls/setrlimit/setrlimit05.c | 10 +- + 1 file changed, 9 insertions(+), 1 deletion(-) + +diff --git a/testcases/kernel/syscalls/setrlimit/setrlimit05.c b/testcases/kernel/syscalls/setrlimit/setrlimit05.c +index e7167229b4..004a149a65 100644 +--- a/testcases/kernel/syscalls/setrlimit/setrlimit05.c b/testcases/kernel/syscalls/setrlimit/setrlimit05.c +@@ -26,6 +26,8 @@ + + #include "tst_test.h" + ++static void *bad_addr; ++ + static void verify_setrlimit(void) + { + int status; +@@ -33,7 +35,7 @@ static void verify_setrlimit(void) + + pid = SAFE_FORK(); + if (!pid) { +- TEST(setrlimit(RLIMIT_NOFILE, (void *) -1)); ++ TEST(setrlimit(RLIMIT_NOFILE, bad_addr)); + if (TST_RET != -1) { + tst_res(TFAIL, "setrlimit() succeeded unexpectedly"); + exit(0); +@@ -67,7 +69,13 @@ static void verify_setrlimit(void) + tst_res(TBROK, "child %s", tst_strstatus(status)); + } + ++static void setup(void) ++{ ++ bad_addr = tst_get_bad_addr(NULL); ++} ++ + static struct tst_test test = { + .test_all = verify_setrlimit, + .forks_child = 1, ++ .setup = setup, + }; +-- +2.11.0 + diff --git a/meta/recipes-extended/ltp/ltp_20180926.bb b/meta/recipes-extended/ltp/ltp_20180926.bb index 622c1f42f7..00e2c05cb6 100644 --- a/meta/recipes-extended/ltp/ltp_20180926.bb +++ b/meta/recipes-extended/ltp/ltp_20180926.bb @@ -52,6 +52,7 @@ SRC_URI = "git://github.com/linux-test-project/ltp.git \ file://0001-netns_helper.sh-use-ping-6-when-ping6-is-not-avaliab.patch \ file://0001-open_posix_testsuite-mmap24-2-Relax-condition-a-bit.patch \ file://0001-statx-fix-compile-errors.patch \ + file://0001-setrlimit05-Use-another-method-to-get-bad-address.patch \ " S = "${WORKDIR}/git" -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [V2 PATCH 0/1] ltp-setrlimit05.c: use a legal bad address for mips
V2: Remake the patch for ltp-20180926 which was ltp-20180505 at the time of the last commit. Hongzhi.Song (1): ltp: Use a legal bad address for mips in setrlimit05.c ...t05-Use-another-method-to-get-bad-address.patch | 61 ++ meta/recipes-extended/ltp/ltp_20180926.bb | 1 + 2 files changed, 62 insertions(+) create mode 100644 meta/recipes-extended/ltp/ltp/0001-setrlimit05-Use-another-method-to-get-bad-address.patch -- 2.11.0 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core