> 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?
I've just seen new build failure in world build and it was indeed with 32bit arm (from multilib builds). Thanks for looking into it quickly. > We looked into it and have to admit that we cannot explain why g++ 14.1.0 > trips over that line. Have you tried to compare preprocessed output from g++ from qemuarm and qemuarm64 to see if there is some strange difference? The code where color_set_not_const is used looks simple enough, but maybe some REQUIRE* macro expansion does more than it looks in https://github.com/search?q=repo%3ANeargye%2Fmagic_enum+color_set_not_const&type=code Regards, On Thu, Jun 27, 2024 at 9:00 AM Jonas Mark (BT-FS/ENG1-GRB) <mark.jo...@de.bosch.com> wrote: > > 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 <martin.ja...@gmail.com> > > Gesendet: Mittwoch, 26. Juni 2024 17:53 > > An: Jonas Mark (BT-FS/ENG1-GRB) <mark.jo...@de.bosch.com> > > Cc: openembedded-devel@lists.openembedded.org; raj.k...@gmail.com; > > Simoes Ricardo (BT-FS/ENG1.1-Ovr) <ricardo.sim...@pt.bosch.com> > > 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 > > <mark.jonas=de.bosch....@lists.openembedded.org> wrote: > > > > > > From: Ricardo Simoes <ricardo.sim...@pt.bosch.com> > > > > > > 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 <ricardo.sim...@pt.bosch.com> > > > Signed-off-by: Mark Jonas <mark.jo...@de.bosch.com> > > > --- > > > .../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 (#111118): https://lists.openembedded.org/g/openembedded-devel/message/111118 Mute This Topic: https://lists.openembedded.org/mt/106757432/21656 Group Owner: openembedded-devel+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
