Date   

Re: [meta-cgl][PATCH V2] ucarp: fix build error with gcc-10

Chen Qi
 

On 09/24/2020 01:43 PM, Khem Raj wrote:
On Wed, Sep 23, 2020 at 8:38 PM Chen Qi <Qi.Chen@...> wrote:
Add a patch to remove the useless '__packed' variable to avoid
multiple definition error with gcc10.
I wonder if the intention is to mark this structure as packed struct
then it should be replaced with __attribute__((packed))
It's possible, but I didn't look at the logic and I don't want to. Anyway, the variable never had any effect. So I figure it's safe to remove it, and this will not change any behavior.

Unfortunately, the codes are really old, and I cannot find a place to send the patch, not to mention to confirm this with its author :(

Regards,
Chen Qi

Signed-off-by: Chen Qi <Qi.Chen@...>
---
...001-ip_carp.h-remove-unused-__packed.patch | 31 +++++++++++++++++++
.../recipes-cgl/ucarp/ucarp_1.5.2.bb | 1 +
2 files changed, 32 insertions(+)
create mode 100644 meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch

diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch b/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch
new file mode 100644
index 0000000..496c20d
--- /dev/null
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch
@@ -0,0 +1,31 @@
+From ad33498ac86fe2df0a100d17aa54ca8c30920d8b Mon Sep 17 00:00:00 2001
+From: Chen Qi <Qi.Chen@...>
+Date: Thu, 24 Sep 2020 11:30:08 +0800
+Subject: [PATCH] ip_carp.h: remove unused __packed
+
+This variable is useless and it's causing multiple definition
+error with gcc10, thus removing it.
+
+Upstream-Status: Pending
+
+Signed-off-by: Chen Qi <Qi.Chen@...>
+---
+ src/ip_carp.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/ip_carp.h b/src/ip_carp.h
+index 9504a8e..9dcf8cc 100644
+--- a/src/ip_carp.h
++++ b/src/ip_carp.h
+@@ -70,7 +70,7 @@ struct carp_header {
+ u_int16_t carp_cksum;
+ u_int32_t carp_counter[2];
+ unsigned char carp_md[20]; /* SHA1 HMAC */
+-} __packed;
++};
+
+ #define CARP_DFLTTL 255
+
+--
+2.17.1
+
diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
index d17baa0..c6fa901 100644
--- a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
@@ -17,6 +17,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=278a886e91f2f6c983ffdf040130cdc6"
SRC_URI = "http://download.pureftpd.org/pub/ucarp/${BPN}-${PV}.tar.gz \
file://ucarp-configure-sha1.patch \
file://ucarp-configure-snprintf.patch \
+ file://0001-ip_carp.h-remove-unused-__packed.patch \
file://ucarp.init \
file://vip-001.conf.example \
file://vip-common.conf \
--
2.17.1



Re: [meta-cgl][PATCH V2] ucarp: fix build error with gcc-10

Khem Raj
 

On Wed, Sep 23, 2020 at 8:38 PM Chen Qi <Qi.Chen@...> wrote:

Add a patch to remove the useless '__packed' variable to avoid
multiple definition error with gcc10.
I wonder if the intention is to mark this structure as packed struct
then it should be replaced with __attribute__((packed))

Signed-off-by: Chen Qi <Qi.Chen@...>
---
...001-ip_carp.h-remove-unused-__packed.patch | 31 +++++++++++++++++++
.../recipes-cgl/ucarp/ucarp_1.5.2.bb | 1 +
2 files changed, 32 insertions(+)
create mode 100644 meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch

diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch b/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch
new file mode 100644
index 0000000..496c20d
--- /dev/null
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch
@@ -0,0 +1,31 @@
+From ad33498ac86fe2df0a100d17aa54ca8c30920d8b Mon Sep 17 00:00:00 2001
+From: Chen Qi <Qi.Chen@...>
+Date: Thu, 24 Sep 2020 11:30:08 +0800
+Subject: [PATCH] ip_carp.h: remove unused __packed
+
+This variable is useless and it's causing multiple definition
+error with gcc10, thus removing it.
+
+Upstream-Status: Pending
+
+Signed-off-by: Chen Qi <Qi.Chen@...>
+---
+ src/ip_carp.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/ip_carp.h b/src/ip_carp.h
+index 9504a8e..9dcf8cc 100644
+--- a/src/ip_carp.h
++++ b/src/ip_carp.h
+@@ -70,7 +70,7 @@ struct carp_header {
+ u_int16_t carp_cksum;
+ u_int32_t carp_counter[2];
+ unsigned char carp_md[20]; /* SHA1 HMAC */
+-} __packed;
++};
+
+ #define CARP_DFLTTL 255
+
+--
+2.17.1
+
diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
index d17baa0..c6fa901 100644
--- a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
@@ -17,6 +17,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=278a886e91f2f6c983ffdf040130cdc6"
SRC_URI = "http://download.pureftpd.org/pub/ucarp/${BPN}-${PV}.tar.gz \
file://ucarp-configure-sha1.patch \
file://ucarp-configure-snprintf.patch \
+ file://0001-ip_carp.h-remove-unused-__packed.patch \
file://ucarp.init \
file://vip-001.conf.example \
file://vip-common.conf \
--
2.17.1




[meta-cgl][PATCH V2] ucarp: fix build error with gcc-10

Chen Qi
 

Add a patch to remove the useless '__packed' variable to avoid
multiple definition error with gcc10.

Signed-off-by: Chen Qi <Qi.Chen@...>
---
...001-ip_carp.h-remove-unused-__packed.patch | 31 +++++++++++++++++++
.../recipes-cgl/ucarp/ucarp_1.5.2.bb | 1 +
2 files changed, 32 insertions(+)
create mode 100644 meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch

diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch b/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch
new file mode 100644
index 0000000..496c20d
--- /dev/null
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp/0001-ip_carp.h-remove-unused-__packed.patch
@@ -0,0 +1,31 @@
+From ad33498ac86fe2df0a100d17aa54ca8c30920d8b Mon Sep 17 00:00:00 2001
+From: Chen Qi <Qi.Chen@...>
+Date: Thu, 24 Sep 2020 11:30:08 +0800
+Subject: [PATCH] ip_carp.h: remove unused __packed
+
+This variable is useless and it's causing multiple definition
+error with gcc10, thus removing it.
+
+Upstream-Status: Pending
+
+Signed-off-by: Chen Qi <Qi.Chen@...>
+---
+ src/ip_carp.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/ip_carp.h b/src/ip_carp.h
+index 9504a8e..9dcf8cc 100644
+--- a/src/ip_carp.h
++++ b/src/ip_carp.h
+@@ -70,7 +70,7 @@ struct carp_header {
+ u_int16_t carp_cksum;
+ u_int32_t carp_counter[2];
+ unsigned char carp_md[20]; /* SHA1 HMAC */
+-} __packed;
++};
+
+ #define CARP_DFLTTL 255
+
+--
+2.17.1
+
diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
index d17baa0..c6fa901 100644
--- a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
@@ -17,6 +17,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=278a886e91f2f6c983ffdf040130cdc6"
SRC_URI = "http://download.pureftpd.org/pub/ucarp/${BPN}-${PV}.tar.gz \
file://ucarp-configure-sha1.patch \
file://ucarp-configure-snprintf.patch \
+ file://0001-ip_carp.h-remove-unused-__packed.patch \
file://ucarp.init \
file://vip-001.conf.example \
file://vip-common.conf \
--
2.17.1


QA notification for completed autobuilder build (yocto-3.2_M3.rc2)

Pokybuild User
 

A build flagged for QA (yocto-3.2_M3.rc2) was completed on the autobuilder and is available at:


https://autobuilder.yocto.io/pub/releases/yocto-3.2_M3.rc2


Build hash information:

bitbake: 27b53186fa67d281d29b2f8e15bcff8dc2557b8a
meta-arm: c7a1a5f9fd415e3ae1078c2a1d6af9c25e9e6498
meta-gplv2: a8da8eb127a56561bf633ab53bec57fb5dbba537
meta-intel: 254e3a79b3a37045f829848e0a5e04801bec1439
meta-kernel: cb7f0dc5bb1ea0998c8d4fcb486148d4cab575f4
meta-mingw: f1e1d5a803a48ed70f2b0eaf0ef15ade70479fce
oecore: e68df923e1bf9e82662ee2619ac9e2f239e8f7be
poky: 0a243cd21263f498b6eca7c0b168e655dd823e22



This is an automated message from the Yocto Project Autobuilder
Git: git://git.yoctoproject.org/yocto-autobuilder2
Email: richard.purdie@...


Re: [meta-cgl][PATCH] meta-cgl-common: depend on meta-python2 layer

Chen Qi
 

Just like Randy points out, please ignore this patch.
I've sent out a new patch to remove 'python-setuptools-native' from crmsh's DEPENDS.

On 09/23/2020 04:01 PM, Chen Qi wrote:
It requires meta-python2 to be there at the moment.
More specifically, the crmsh recipe needs python-setuptools-native.

Signed-off-by: Chen Qi <Qi.Chen@...>
---
 meta-cgl-common/conf/layer.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-cgl-common/conf/layer.conf b/meta-cgl-common/conf/layer.conf
index 6035b4b..6b689ea 100644
--- a/meta-cgl-common/conf/layer.conf
+++ b/meta-cgl-common/conf/layer.conf
@@ -9,7 +9,7 @@ BBFILE_COLLECTIONS += "cgl-common"
 BBFILE_PATTERN_cgl-common = "^${LAYERDIR}/"
 BBFILE_PRIORITY_cgl-common = "7"
 
-LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux"
+LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux meta-python2"
 
 LAYERSERIES_COMPAT_cgl-common = "warrior zeus dunfell"
 






Re: [meta-cgl][PATCH] meta-cgl-common: depend on meta-python2 layer

Chen Qi
 

THANKS. I've sent out a new patch.

On 09/23/2020 11:58 PM, Randy MacLeod wrote:
On 2020-09-23 4:01 a.m., Chen Qi wrote:
It requires meta-python2 to be there at the moment.
More specifically, the crmsh recipe needs python-setuptools-native.

Hi Qi,

Are you sure?
It looks like crmsh moved to python3 a while ago as
indicated below.

If there's still a dependency, please open an issue:

   https://github.com/ClusterLabs/crmsh/issues?q=is%3Aissue+is%3Aopen+setuptools

Thanks,

../Randy


$ cd .../crmsh.git

$ git status
HEAD detached at 4.2.0
nothing to commit, working tree clean

$ grep -r setuptools *

contrib/setup.py:from setuptools import setup
crmsh.spec.in:BuildRequires:  python3-setuptools
doc/website-v1/development.adoc:* `setuptools`
Dockerfile:RUN zypper -n --gpg-auto-import-keys ref && zypper -n --gpg-auto-import-keys in pacemaker python3 python3-lxml python3-python-dateutil python3-parallax libglue-devel python3-setuptools python3-tox asciidoc autoconf automake make pkgconfig which libxslt-tools mailx procps python3-nose python3-PyYAML python3-curses tar
README.md:It then calls the python setuptools setup.py to actually process the
setup.py:from setuptools import setup


git log -1 --stat 845af7b9fb7a4c212d2e67b8a44cf655e702edaa
commit 845af7b9fb7a4c212d2e67b8a44cf655e702edaa
Author: Kristoffer Grönlund <krig@...>
Date:   Fri Oct 27 05:47:02 2017

    dev: Updated spec file

 crmsh.spec | 106 +++++++++++++++++...+++++++++-----------------------------------------
 1 file changed, 65 insertions(+), 41 deletions(-)

$ grep setuptools crmsh.spec.in
BuildRequires:  python3-setuptools


Signed-off-by: Chen Qi <Qi.Chen@...>
---
 meta-cgl-common/conf/layer.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-cgl-common/conf/layer.conf b/meta-cgl-common/conf/layer.conf
index 6035b4b..6b689ea 100644
--- a/meta-cgl-common/conf/layer.conf
+++ b/meta-cgl-common/conf/layer.conf
@@ -9,7 +9,7 @@ BBFILE_COLLECTIONS += "cgl-common"
 BBFILE_PATTERN_cgl-common = "^${LAYERDIR}/"
 BBFILE_PRIORITY_cgl-common = "7"
 
-LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux"
+LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux meta-python2"
 
 LAYERSERIES_COMPAT_cgl-common = "warrior zeus dunfell"
 



-- 
# Randy MacLeod
# Wind River Linux



[meta-cgl][PATCH] crmsh: remove python-setuptools-native from DEPENDS

Chen Qi
 

Current version of crmsh uses python3-setuptools, which is from
setuptools3.bbclass. So we can safely remove this python2 dependency.

Signed-off-by: Chen Qi <Qi.Chen@...>
---
meta-cgl-common/recipes-cgl/crmsh/crmsh_4.20.bb | 1 -
1 file changed, 1 deletion(-)

diff --git a/meta-cgl-common/recipes-cgl/crmsh/crmsh_4.20.bb b/meta-cgl-common/recipes-cgl/crmsh/crmsh_4.20.bb
index add7dba..830802d 100644
--- a/meta-cgl-common/recipes-cgl/crmsh/crmsh_4.20.bb
+++ b/meta-cgl-common/recipes-cgl/crmsh/crmsh_4.20.bb
@@ -9,7 +9,6 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
DEPENDS = "asciidoc-native \
docbook-xsl-stylesheets-native \
libxslt-native \
- python-setuptools-native \
"
RDEPENDS_${PN} = "pacemaker python3-lxml python3-parallax gawk bash"

--
2.17.1


[meta-zephyr][PATCH] acrn.conf: set tune corei7-64

Naveen Saini
 

Added CONFIG_X86_64 since ACRN hypervisor supports x86_64 capable
platforms only.

Ref:
https://github.com/zephyrproject-rtos/zephyr/commit/d0126a037d23484feebba00d2c0eac27e6393fef

Signed-off-by: Naveen Saini <naveen.kumar.saini@...>
---
conf/machine/acrn.conf | 3 +--
conf/machine/include/tune-corei7-common.inc | 3 +++
2 files changed, 4 insertions(+), 2 deletions(-)
create mode 100644 conf/machine/include/tune-corei7-common.inc

diff --git a/conf/machine/acrn.conf b/conf/machine/acrn.conf
index 96a7d66..c044933 100644
--- a/conf/machine/acrn.conf
+++ b/conf/machine/acrn.conf
@@ -3,8 +3,7 @@
#@DESCRIPTION: Machine for Zephyr BOARD acrn

require conf/machine/include/qemu.inc
-require conf/machine/include/tune-i586.inc
-
+require conf/machine/include/tune-corei7-common.inc
ZEPHYR_INHERIT_CLASSES += "zephyr-qemuboot"

ARCH_acrn = "x86"
diff --git a/conf/machine/include/tune-corei7-common.inc b/conf/machine/include/tune-corei7-common.inc
new file mode 100644
index 0000000..7ad9516
--- /dev/null
+++ b/conf/machine/include/tune-corei7-common.inc
@@ -0,0 +1,3 @@
+DEFAULTTUNE ?= "corei7-64"
+require conf/machine/include/tune-corei7.inc
+require conf/machine/include/x86-base.inc
--
2.7.4


Re: [meta-mingw][PATCH] Override SDK_VENDOR

kergoth@...
 

Assuming AB success, is this also suitable for stable branches like dunfell?


On Mon, Sep 21, 2020 at 6:21 AM Joshua Watt <JPEWhacker@...> wrote:


On Mon, Sep 21, 2020, 8:12 AM Samuli Piippo <samuli.piippo@...> wrote:

On Mon, 21 Sep 2020 at 15:53, Joshua Watt <JPEWhacker@...> wrote:
On Fri, Sep 18, 2020 at 7:30 AM Samuli Piippo <samuli.piippo@...> wrote:
>
> Set SDK_VENDOR to '-w64', which makes the host triplet match what GCC
> expect to find when using mingw32-w64. This enables features that are
> not functional in the classic mingw32, but have been implemented in the
> mingw32-w64.

Does this enable it for the i686 toolchain also? Does that make sense?

This enables it for both x86_64-mingw32 and i686-mingw32 targets and it makes sense
since it's not about the target bitness but the mingw implementation. w64 has support
for both targets and provides improved support over the original mingw32.

Thanks. I figured that was the case. This is testing on the AB.

 
>
> Disable 32bit libs from the runtime component when compiling for 64bit,
> which were enabled as a side effect of the GCC config change.
>
> Signed-off-by: Samuli Piippo <samuli.piippo@...>
> ---
>  conf/machine-sdk/include/mingw32-common.inc                    | 3 +++
>  .../mingw-w64/nativesdk-mingw-w64-runtime_7.0.0.bb             | 2 ++
>  2 files changed, 5 insertions(+)
>
> diff --git a/conf/machine-sdk/include/mingw32-common.inc b/conf/machine-sdk/include/mingw32-common.inc
> index 9011ded..bc6c91e 100644
> --- a/conf/machine-sdk/include/mingw32-common.inc
> +++ b/conf/machine-sdk/include/mingw32-common.inc
> @@ -1,4 +1,7 @@
>  SDK_OS = "mingw32"
> +SDK_VENDOR_mingw32 = "-w64"
> +SDK_VENDOR_sdkmingw32 = "-w64"
> +
>  NATIVESDKLIBC = "libc-mingw"
>
>  PREFERRED_PROVIDER_virtual/nativesdk-${SDK_PREFIX}libc-for-gcc_mingw32 = "nativesdk-mingw-w64-runtime"
> diff --git a/recipes-devtools/mingw-w64/nativesdk-mingw-w64-runtime_7.0.0.bb b/recipes-devtools/mingw-w64/nativesdk-mingw-w64-runtime_7.0.0.bb
> index cf39c6a..9f79ffe 100644
> --- a/recipes-devtools/mingw-w64/nativesdk-mingw-w64-runtime_7.0.0.bb
> +++ b/recipes-devtools/mingw-w64/nativesdk-mingw-w64-runtime_7.0.0.bb
> @@ -19,6 +19,8 @@ PROVIDES += "virtual/nativesdk-libintl"
>
>  TOOLCHAIN_OPTIONS = " --sysroot=${STAGING_DIR_TARGET}"
>
> +EXTRA_OECONF_x86-64 = "--disable-lib32"
> +
>  do_configure() {
>      oe_runconf
>  }
> --
> 2.17.1
>








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


Re: [meta-cgl][PATCH] meta-cgl-common: depend on meta-python2 layer

Randy MacLeod
 

On 2020-09-23 4:01 a.m., Chen Qi wrote:
It requires meta-python2 to be there at the moment.
More specifically, the crmsh recipe needs python-setuptools-native.

Hi Qi,

Are you sure?
It looks like crmsh moved to python3 a while ago as
indicated below.

If there's still a dependency, please open an issue:

   https://github.com/ClusterLabs/crmsh/issues?q=is%3Aissue+is%3Aopen+setuptools

Thanks,

../Randy


$ cd .../crmsh.git

$ git status
HEAD detached at 4.2.0
nothing to commit, working tree clean

$ grep -r setuptools *

contrib/setup.py:from setuptools import setup
crmsh.spec.in:BuildRequires:  python3-setuptools
doc/website-v1/development.adoc:* `setuptools`
Dockerfile:RUN zypper -n --gpg-auto-import-keys ref && zypper -n --gpg-auto-import-keys in pacemaker python3 python3-lxml python3-python-dateutil python3-parallax libglue-devel python3-setuptools python3-tox asciidoc autoconf automake make pkgconfig which libxslt-tools mailx procps python3-nose python3-PyYAML python3-curses tar
README.md:It then calls the python setuptools setup.py to actually process the
setup.py:from setuptools import setup


git log -1 --stat 845af7b9fb7a4c212d2e67b8a44cf655e702edaa
commit 845af7b9fb7a4c212d2e67b8a44cf655e702edaa
Author: Kristoffer Grönlund <krig@...>
Date:   Fri Oct 27 05:47:02 2017

    dev: Updated spec file

 crmsh.spec | 106 +++++++++++++++++...+++++++++-----------------------------------------
 1 file changed, 65 insertions(+), 41 deletions(-)

$ grep setuptools crmsh.spec.in
BuildRequires:  python3-setuptools



Signed-off-by: Chen Qi <Qi.Chen@...>
---
 meta-cgl-common/conf/layer.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-cgl-common/conf/layer.conf b/meta-cgl-common/conf/layer.conf
index 6035b4b..6b689ea 100644
--- a/meta-cgl-common/conf/layer.conf
+++ b/meta-cgl-common/conf/layer.conf
@@ -9,7 +9,7 @@ BBFILE_COLLECTIONS += "cgl-common"
 BBFILE_PATTERN_cgl-common = "^${LAYERDIR}/"
 BBFILE_PRIORITY_cgl-common = "7"
 
-LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux"
+LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux meta-python2"
 
 LAYERSERIES_COMPAT_cgl-common = "warrior zeus dunfell"
 




-- 
# Randy MacLeod
# Wind River Linux


useless-rpaths : How to solve it/Know if it should be solved ?

Vitor Crestani Goergen <crestanivitor@...>
 

Hello,

I have been dealing with Yocto for about a month, so I am still a beginner.

I am usign a NXP Sabresd board with an iMX6 processor. I have the recipes from oe, bsp, freescale etc downloaded from the NXP website, although I replaced the kernel with the EVL core project kernel. I did this in a workspace using devtool.

I was able to bitbake and build this new kernel without a problem. After that, I set on to install some libraries, specifically libevl, which is the library from the EVL project. This source code comes with several makefiles. I don't understand most of the codes in these makefiles but I am currently working on that. 

I created a new layer and a recipe for this library. Initially I had an error of "files shipped but not installed" but that was fixed by adding some more information to FILES_${PN}. Now, I am stuck with the following error message when trying to build:

Error: QA issue: libevl: /work/{machine}/libevl/0.1-r0/packages-split/libevl/usr/lib/libevl-0.so.0 contains probably redundant RPATH /usr/lib 

I am aware that a "solution" exists using INSANE_SKIP. I did that and everything worked. However, I would not like to put more things than I should just to take up space on the board.

What I tried so far was to add somefiles to FILES_${PN}-dev, telling Make to ignore RPATH (Using the variable CMAKE_IGNORE_RPATH) but none of these worked.

I've also seen in some discussions that maybe this should probably stay as it is. How can I determine if this is normal behaviour and I should just use INSANE_SKIP?

Thanks in advance,

Vitor


Re: [meta-cgl][PATCH] ucarp: fix build error with gcc-10

Bas Mevissen
 

On 2020-09-23 11:36, ChenQi wrote:
On 09/23/2020 04:55 PM, Bas Mevissen wrote:
On 2020-09-23 09:56, Chen Qi wrote:
gcc-10 uses '-fno-common' by default, causing multiple definition
error. Use '-fcommon' to fix this problem.
Signed-off-by: Chen Qi <Qi.Chen@...>
---
meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb | 3 +++
1 file changed, 3 insertions(+)
diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
index d17baa0..87e5ada 100644
--- a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
@@ -37,6 +37,9 @@ SYSTEMD_AUTO_ENABLE = "disable"
export FETCHCMD_wget = "/usr/bin/env wget --secure-protocol=TLSv1_2
-t 2 -T 30 --passive-ftp --no-check-certificate"
EXTRA_OECONF += "--sysconfdir=${sysconfdir}/${BPN}"
+# Fix build error with gcc-10
+CFLAGS_append = " -fcommon"
+
Why are you not patching the source to use "extern" where it is needed, like in pacemaker? How many errors does ucarp give?
Then it might also a patch that upstream will take, solving the issue forever in a proper way.
Could you please send out one?
I don't have a tree with this layer in front of me. I just reviewed your proposed fix by setting the compiler flag, while you demonstrated in the pacemaker patch that the issue is easily solved by adding some "exern"s where they need to be. So I wondered why you didn't do that for ucarp.

# fix the perms for config.rpath
do_configure_prepend() {
chmod 755 ${S}/config.rpath


Re: [meta-cgl][PATCH] ucarp: fix build error with gcc-10

Chen Qi
 

On 09/23/2020 04:55 PM, Bas Mevissen wrote:
On 2020-09-23 09:56, Chen Qi wrote:
gcc-10 uses '-fno-common' by default, causing multiple definition
error. Use '-fcommon' to fix this problem.

Signed-off-by: Chen Qi <Qi.Chen@...>
---
meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb | 3 +++
1 file changed, 3 insertions(+)

diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
index d17baa0..87e5ada 100644
--- a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
@@ -37,6 +37,9 @@ SYSTEMD_AUTO_ENABLE = "disable"
export FETCHCMD_wget = "/usr/bin/env wget --secure-protocol=TLSv1_2
-t 2 -T 30 --passive-ftp --no-check-certificate"
EXTRA_OECONF += "--sysconfdir=${sysconfdir}/${BPN}"

+# Fix build error with gcc-10
+CFLAGS_append = " -fcommon"
+
Why are you not patching the source to use "extern" where it is needed, like in pacemaker? How many errors does ucarp give?
Then it might also a patch that upstream will take, solving the issue forever in a proper way.
Could you please send out one?

# fix the perms for config.rpath
do_configure_prepend() {
chmod 755 ${S}/config.rpath


Re: [meta-cgl][PATCH] ucarp: fix build error with gcc-10

Bas Mevissen
 

On 2020-09-23 09:56, Chen Qi wrote:
gcc-10 uses '-fno-common' by default, causing multiple definition
error. Use '-fcommon' to fix this problem.
Signed-off-by: Chen Qi <Qi.Chen@...>
---
meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb | 3 +++
1 file changed, 3 insertions(+)
diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
index d17baa0..87e5ada 100644
--- a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
@@ -37,6 +37,9 @@ SYSTEMD_AUTO_ENABLE = "disable"
export FETCHCMD_wget = "/usr/bin/env wget --secure-protocol=TLSv1_2
-t 2 -T 30 --passive-ftp --no-check-certificate"
EXTRA_OECONF += "--sysconfdir=${sysconfdir}/${BPN}"
+# Fix build error with gcc-10
+CFLAGS_append = " -fcommon"
+
Why are you not patching the source to use "extern" where it is needed, like in pacemaker? How many errors does ucarp give?
Then it might also a patch that upstream will take, solving the issue forever in a proper way.

# fix the perms for config.rpath
do_configure_prepend() {
chmod 755 ${S}/config.rpath


[meta-cgl][PATCH] meta-cgl-common: depend on meta-python2 layer

Chen Qi
 

It requires meta-python2 to be there at the moment.
More specifically, the crmsh recipe needs python-setuptools-native.

Signed-off-by: Chen Qi <Qi.Chen@...>
---
meta-cgl-common/conf/layer.conf | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-cgl-common/conf/layer.conf b/meta-cgl-common/conf/layer.conf
index 6035b4b..6b689ea 100644
--- a/meta-cgl-common/conf/layer.conf
+++ b/meta-cgl-common/conf/layer.conf
@@ -9,7 +9,7 @@ BBFILE_COLLECTIONS += "cgl-common"
BBFILE_PATTERN_cgl-common = "^${LAYERDIR}/"
BBFILE_PRIORITY_cgl-common = "7"

-LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux"
+LAYERDEPENDS_cgl-common = "core openembedded-layer networking-layer perl-layer filesystems-layer security selinux meta-python2"

LAYERSERIES_COMPAT_cgl-common = "warrior zeus dunfell"

--
2.17.1


[meta-cgl][PATCH] ucarp: fix build error with gcc-10

Chen Qi
 

gcc-10 uses '-fno-common' by default, causing multiple definition
error. Use '-fcommon' to fix this problem.

Signed-off-by: Chen Qi <Qi.Chen@...>
---
meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb | 3 +++
1 file changed, 3 insertions(+)

diff --git a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
index d17baa0..87e5ada 100644
--- a/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
+++ b/meta-cgl-common/recipes-cgl/ucarp/ucarp_1.5.2.bb
@@ -37,6 +37,9 @@ SYSTEMD_AUTO_ENABLE = "disable"
export FETCHCMD_wget = "/usr/bin/env wget --secure-protocol=TLSv1_2 -t 2 -T 30 --passive-ftp --no-check-certificate"
EXTRA_OECONF += "--sysconfdir=${sysconfdir}/${BPN}"

+# Fix build error with gcc-10
+CFLAGS_append = " -fcommon"
+
# fix the perms for config.rpath
do_configure_prepend() {
chmod 755 ${S}/config.rpath
--
2.17.1


[meta-cgl][PATCH] pacemaker: Fix build with -fno-common

Chen Qi
 

From: Mingli Yu <mingli.yu@...>

Starting with GCC >= 10.x, -fno-common is used as default
instead of -fcommon.

Make the function definiton extern to fix the build failure.

Signed-off-by: Mingli Yu <mingli.yu@...>
---
.../0001-Mark-declaration-with-extern.patch | 46 +++++++++++++++++++
...maker-set-OCF_ROOT_DIR-to-libdir-ocf.patch | 32 +++++++++++++
.../recipes-cgl/pacemaker/pacemaker_2.0.3.bb | 2 +
3 files changed, 80 insertions(+)
create mode 100644 meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-Mark-declaration-with-extern.patch
create mode 100644 meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-pacemaker-set-OCF_ROOT_DIR-to-libdir-ocf.patch

diff --git a/meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-Mark-declaration-with-extern.patch b/meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-Mark-declaration-with-extern.patch
new file mode 100644
index 0000000..5729447
--- /dev/null
+++ b/meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-Mark-declaration-with-extern.patch
@@ -0,0 +1,46 @@
+From e1abd3b7c7a0122813e4d0abdb079df10104882c Mon Sep 17 00:00:00 2001
+From: Mingli Yu <mingli.yu@...>
+Date: Thu, 3 Sep 2020 04:44:09 +0000
+Subject: [PATCH] Mark declaration with extern
+
+Fixes build with gcc 10+
+
+Upstream-Status: Pending
+
+Signed-off-by: Mingli Yu <mingli.yu@...>
+---
+ daemons/attrd/pacemaker-attrd.h | 4 ++--
+ daemons/execd/pacemaker-execd.h | 2 +-
+ 2 files changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/daemons/attrd/pacemaker-attrd.h b/daemons/attrd/pacemaker-attrd.h
+index cc8e29ee1..76778915e 100644
+--- a/daemons/attrd/pacemaker-attrd.h
++++ b/daemons/attrd/pacemaker-attrd.h
+@@ -106,8 +106,8 @@ typedef struct attribute_value_s {
+ gboolean seen;
+ } attribute_value_t;
+
+-crm_cluster_t *attrd_cluster;
+-GHashTable *attributes;
++extern crm_cluster_t *attrd_cluster;
++extern GHashTable *attributes;
+
+ #define attrd_send_ack(client, id, flags) \
+ crm_ipcs_send_ack((client), (id), (flags), "ack", __FUNCTION__, __LINE__)
+diff --git a/daemons/execd/pacemaker-execd.h b/daemons/execd/pacemaker-execd.h
+index 4a52d9183..dab3ccdbe 100644
+--- a/daemons/execd/pacemaker-execd.h
++++ b/daemons/execd/pacemaker-execd.h
+@@ -20,7 +20,7 @@
+ # include <gnutls/gnutls.h>
+ # endif
+
+-GHashTable *rsc_list;
++extern GHashTable *rsc_list;
+
+ typedef struct lrmd_rsc_s {
+ char *rsc_id;
+--
+2.26.2
+
diff --git a/meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-pacemaker-set-OCF_ROOT_DIR-to-libdir-ocf.patch b/meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-pacemaker-set-OCF_ROOT_DIR-to-libdir-ocf.patch
new file mode 100644
index 0000000..1ff9c7d
--- /dev/null
+++ b/meta-cgl-common/recipes-cgl/pacemaker/pacemaker/0001-pacemaker-set-OCF_ROOT_DIR-to-libdir-ocf.patch
@@ -0,0 +1,32 @@
+From 3ca78a6441eefc26f18211375b18205ed6fc28c6 Mon Sep 17 00:00:00 2001
+From: Mingli Yu <mingli.yu@...>
+Date: Thu, 3 Sep 2020 05:26:36 +0000
+Subject: [PATCH] pacemaker: set OCF_ROOT_DIR to $libdir/ocf
+
+* Set the default OCF_ROOT_DIR to $libdir/ocf
+ to make the resource agents components more
+ compatible
+
+Upstream-Status: Pending
+
+Signed-off-by: Mingli Yu <mingli.yu@...>
+---
+ configure.ac | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index 58d39cdc0..eb4275560 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -315,7 +315,7 @@ dnl This defaults to /usr/lib rather than libdir because it's determined by the
+ dnl OCF project and not pacemaker. Even if a user wants to install pacemaker to
+ dnl /usr/local or such, the OCF agents will be expected in their usual
+ dnl location. However, we do give the user the option to override it.
+-OCF_ROOT_DIR="/usr/lib/ocf"
++OCF_ROOT_DIR="$libdir/ocf"
+ AC_ARG_WITH([ocfdir],
+ [AS_HELP_STRING([--with-ocfdir=DIR],
+ [OCF resource agent root directory (advanced option: changing this may break other cluster components unless similarly configured) @<:@/usr/lib/ocf@:>@])],
+--
+2.26.2
+
diff --git a/meta-cgl-common/recipes-cgl/pacemaker/pacemaker_2.0.3.bb b/meta-cgl-common/recipes-cgl/pacemaker/pacemaker_2.0.3.bb
index 9b63acd..56f3bc4 100644
--- a/meta-cgl-common/recipes-cgl/pacemaker/pacemaker_2.0.3.bb
+++ b/meta-cgl-common/recipes-cgl/pacemaker/pacemaker_2.0.3.bb
@@ -16,6 +16,8 @@ DEPENDS = "corosync libxslt libxml2 gnutls resource-agents libqb python3-native"
SRC_URI = "git://github.com/ClusterLabs/${BPN}.git \
file://0006-Fix-tools-Fix-definition-of-curses_indented_printf.patch \
file://0001-Fix-python3-usage.patch \
+ file://0001-Mark-declaration-with-extern.patch \
+ file://0001-pacemaker-set-OCF_ROOT_DIR-to-libdir-ocf.patch \
file://volatiles \
file://tmpfiles \
"
--
2.17.1


Re: bitbake recipe / Network UPS tool

Yocto
 


On 9/23/20 12:48 PM, Maciej Pijanowski wrote:


On 07.09.2020 11:58, Yocto wrote:

On 9/7/20 1:23 PM, Maciej Pijanowski wrote:

Hi,

I happen to have this recipe lying around. I have not upstreamed it for some
reason (probably lack of time and I have not been using this package at the end,
so it is not properly tested out). I gave it a try today and at least it builds.

Please try it out: https://github.com/3mdeb/meta-openembedded/commit/e523d0bb4bddf0ef8521804459b265c14100f83c

If it works for you, please let me know. I would be happy to push the patches upstream.

wow... much appreciated, probably just saved me a few headaches and a ton of time.

Ill get it onboard and configured and let you know how i make out!

I was actually surprised to not find it already in the tree....
Have you got a chance to test something?

its  in the build we are assembling the system the past 2 days actually with the ups / power supply and case for prototype... testing will come in days when i have a functional device built i can flash it and test it all out



Thanks


-- 
Maciej Pijanowski
Embedded Systems Engineer
GPG: 9963C36AAC3B2B46
https://3mdeb.com | @3mdeb_com




[meta-security][master][dunfell][PATCH] gitignore added

Adrian Freihofer
 

After running testimage there are some python left overs at
lib/oeqa/runtime/cases/__pycache__/

Signed-off-by: Adrian Freihofer <adrian.freihofer@...>
---
.gitignore | 7 +++++++
1 file changed, 7 insertions(+)
create mode 100644 .gitignore

diff --git a/.gitignore b/.gitignore
new file mode 100644
index 0000000..c01df45
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,7 @@
+*.pyc
+*.pyo
+/*.patch
+*.swp
+*.orig
+*.rej
+*~
--
2.26.2


Re: bitbake recipe / Network UPS tool

Maciej Pijanowski
 


On 07.09.2020 11:58, Yocto wrote:

On 9/7/20 1:23 PM, Maciej Pijanowski wrote:

Hi,

I happen to have this recipe lying around. I have not upstreamed it for some
reason (probably lack of time and I have not been using this package at the end,
so it is not properly tested out). I gave it a try today and at least it builds.

Please try it out: https://github.com/3mdeb/meta-openembedded/commit/e523d0bb4bddf0ef8521804459b265c14100f83c

If it works for you, please let me know. I would be happy to push the patches upstream.

wow... much appreciated, probably just saved me a few headaches and a ton of time.

Ill get it onboard and configured and let you know how i make out!

I was actually surprised to not find it already in the tree....
Have you got a chance to test something?

Thanks



    
-- 
Maciej Pijanowski
Embedded Systems Engineer
GPG: 9963C36AAC3B2B46
https://3mdeb.com | @3mdeb_com

6621 - 6640 of 57398