Hi Martin,
Thank you for making us aware of the problem when compiling the ptest. We were
able to reproduce the problem when compiling for qemuarm. We were not able to
spot the problem on the autobuilder. How did you trigger the problem?
We looked into it and have to admit that we cannot explain why g++ 14.1.0 trips
over that line.
- It is perfectly legal C++ 17.
- It compiles without that warning error for other machines, i.e. qemux86,
qemux86-64, qemuarm64.
- It does not act up with g++ 11.4 (Kirkstone) with qemuarm, our own NXP i.MX6
machine, or any other machine we tried.
The problem can be mitigated by replacing the offending line
magic_enum::containers::set color_set_not_const {Color::RED, Color::GREEN,
Color::BLUE};
with
magic_enum::containers::set color_set_not_const = {Color::RED, Color::GREEN,
Color::BLUE};
We are working on sending a patch for the recipe containing a patch for
magic_enum to the OE mailing list (this one).
We already got in contact with the upstream magic_enum project.
https://github.com/Neargye/magic_enum/issues/362
Cheers,
Mark
Building Technologies, Panel Software Fire (BT-FIR/ENG1-Grb)
Bosch Sicherheitssysteme GmbH | Postfach 11 11 | 85626 Grasbrunn | GERMANY |
www.boschsecurity.com
Sitz: Stuttgart, Registergericht: Amtsgericht Stuttgart HRB 23118
Aufsichtsratsvorsitzender: Frank Meyer; Geschäftsführung: Thomas Quante, Peter
Löffler, Henrik Siegle, Peter von Wartenberg
> -----Ursprüngliche Nachricht-----
> Von: Martin Jansa <[email protected]>
> Gesendet: Mittwoch, 26. Juni 2024 17:53
> An: Jonas Mark (BT-FS/ENG1-GRB) <[email protected]>
> Cc: [email protected]; [email protected];
> Simoes Ricardo (BT-FS/ENG1.1-Ovr) <[email protected]>
> Betreff: Re: [oe] [meta-oe][PATCH] magic-enum: add recipe
>
> ptest seems to fail building
>
> lib32-magic-enum/0.9.5/git/test/test_containers.cpp:290:89: error:
> value computed is not used [-Werror=unused-value]
> 290 | magic_enum::containers::set color_set_not_const {Color::RED,
> Color::GREEN, Color::BLUE};
> |
> ^
>
> On Wed, Jun 19, 2024 at 11:47 AM Mark Jonas via lists.openembedded.org
> <[email protected]> wrote:
> >
> > From: Ricardo Simoes <[email protected]>
> >
> > magic-enum is a header-only C++17 library provides static reflection
> > for enums, works with any enum type without any macro or boilerplate
> > code.
> >
> > Signed-off-by: Ricardo Simoes <[email protected]>
> > Signed-off-by: Mark Jonas <[email protected]>
> > ---
> > .../magic-enum/magic-enum/run-ptest | 27 ++++++++++++
> > .../magic-enum/magic-enum_0.9.5.bb | 44
> +++++++++++++++++++
> > 2 files changed, 71 insertions(+)
> > create mode 100644
> > meta-oe/recipes-extended/magic-enum/magic-enum/run-ptest
> > create mode 100644
> > meta-oe/recipes-extended/magic-enum/magic-enum_0.9.5.bb
> >
> > diff --git a/meta-oe/recipes-extended/magic-enum/magic-enum/run-
> ptest
> > b/meta-oe/recipes-extended/magic-enum/magic-enum/run-ptest
> > new file mode 100644
> > index 000000000..865da9f69
> > --- /dev/null
> > +++ b/meta-oe/recipes-extended/magic-enum/magic-enum/run-ptest
> > @@ -0,0 +1,27 @@
> > +#!/bin/sh
> > +
> > +# SPDX-FileCopyrightText: 2024 Bosch Sicherheitssysteme GmbH # #
> > +SPDX-License-Identifier: MIT
> > +
> > +fail_count=0
> > +all_count=0
> > +
> > +for test_suite in tests/test_*
> > +do
> > + if "./$test_suite"
> > + then
> > + echo "PASS: $test_suite"
> > + else
> > + echo "FAIL: $test_suite"
> > + fail_count=$((fail_count + 1))
> > + fi
> > + all_count=$((all_count + 1))
> > +done
> > +
> > +if [ $fail_count -eq 0 ]
> > +then
> > + echo "PASS: All $all_count tests passed"
> > +else
> > + echo "FAIL: $fail_count of $all_count tests failed"
> > +fi
> > diff --git a/meta-oe/recipes-extended/magic-enum/magic-enum_0.9.5.bb
> > b/meta-oe/recipes-extended/magic-enum/magic-enum_0.9.5.bb
> > new file mode 100644
> > index 000000000..096f869c3
> > --- /dev/null
> > +++ b/meta-oe/recipes-extended/magic-enum/magic-enum_0.9.5.bb
> > @@ -0,0 +1,44 @@
> > +# SPDX-FileCopyrightText: 2024 Bosch Sicherheitssysteme GmbH # #
> > +SPDX-License-Identifier: MIT
> > +
> > +SUMMARY = "Static reflection for enums"
> > +DESCRIPTION = "Header-only C++17 library provides static reflection
> > +for enums, works \ with any enum type without any macro or
> boilerplate code."
> > +BUGTRACKER =
> "https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit
> hub.com%2FNeargye%2Fmagic_enum%2Fissues&data=05%7C02%7Cmark.jonas%40de
> .bosch.com%7C30c7277c5aa54b0fd67c08dc95f80c79%7C0ae51e1907c84e4bbb6d64
> 8ee58410f4%7C0%7C0%7C638550139794967735%7CUnknown%7CTWFpbGZsb3d8eyJWIj
> oiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C
> %7C&sdata=dTRcL1OePxuq8rbGeyR4SgdAjkLBeKX79lwaRns9pmQ%3D&reserved=0"
> > +HOMEPAGE =
> "https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit
> hub.com%2FNeargye%2Fmagic_enum&data=05%7C02%7Cmark.jonas%40de.bosch.co
> m%7C30c7277c5aa54b0fd67c08dc95f80c79%7C0ae51e1907c84e4bbb6d648ee58410f
> 4%7C0%7C0%7C638550139794976366%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA
> wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata
> =6GyAhu2fPHE0jZyaohpwL6%2FkP70fly7o4Bfxs9oPdDQ%3D&reserved=0"
> > +
> > +LICENSE = "MIT"
> > +LIC_FILES_CHKSUM =
> "file://LICENSE;md5=b15f48588464ec8ef87d2b560aad2caa"
> > +
> > +SRC_URI = " \
> > +
> git://github.com/Neargye/magic_enum.git;protocol=https;branch=master \
> > + file://run-ptest \
> > +"
> > +
> > +SRCREV = "e55b9b54d5cf61f8e117cafb17846d7d742dd3b4"
> > +S = "${WORKDIR}/git"
> > +
> > +inherit cmake ptest
> > +
> > +EXTRA_OECMAKE = "\
> > + -DMAGIC_ENUM_OPT_BUILD_EXAMPLES=OFF \ "
> > +
> > +do_install_ptest () {
> > + install -d ${D}${PTEST_PATH}/tests
> > + install -m 0755 ${B}/test/test_* ${D}${PTEST_PATH}/tests }
> > +
> > +# Add catkin and colcon (ROS build system) support FILES:${PN}-dev
> +=
> > +"\
> > + ${datadir}/magic_enum/package.xml \ "
> > +
> > +# Header-only library
> > +# ${PN} is empty so we need to tweak -dev and -dbg package
> > +dependencies RDEPENDS:${PN}-dev = ""
> > +RDEPENDS:${PN}-ptest = ""
> > +RRECOMMENDS:${PN}-dbg = "${PN}-dev (= ${EXTENDPKGV})"
> > +
> > +BBCLASSEXTEND = "native nativesdk"
> > --
> > 2.34.1
> >
> >
> >
> >
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#111117):
https://lists.openembedded.org/g/openembedded-devel/message/111117
Mute This Topic: https://lists.openembedded.org/mt/106757432/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
