[oe] [meta-oe][PATCH v2] catch2: new package

[Bartosz Golaszewski]

pon., 15 paź 2018 o 20:04 Khem Raj <raj.khem at gmail.com> napisał(a):
>
> On Mon, Oct 15, 2018 at 8:54 AM Bartosz Golaszewski <brgl at bgdev.pl> wrote:
> >
> > From: Bartosz Golaszewski <bgolaszewski at baylibre.com>
> >
> > Add a recipe for the catch2 testing framework. There's a bug upstream
> > which makes it impossible to build with gcc7 so include a patch.
> >
> > Signed-off-by: Bartosz Golaszewski <bgolaszewski at baylibre.com>
> > ---
> > v1 -> v2:
> > - upstream merged a different solution for the build error: backport
> >   the relevant patch, include Upstream-Status tag
> > - use ${datadir} instead of hardcoded paths
> > - depend on python3-native, not python-native
> > - replace ALLOW_EMPTY_${PN} = "1" with RDEPENDS/RRECOMMENDS
> >
> >  ...1-Fix-convert-from-char-on-ARM-build.patch | 46 +++++++++++++++++++
> >  meta-oe/recipes-test/catch2/catch2_2.4.1.bb   | 26 +++++++++++
> >  2 files changed, 72 insertions(+)
> >  create mode 100644 meta-oe/recipes-test/catch2/catch2/0001-Fix-convert-from-char-on-ARM-build.patch
> >  create mode 100644 meta-oe/recipes-test/catch2/catch2_2.4.1.bb
> >
> > diff --git a/meta-oe/recipes-test/catch2/catch2/0001-Fix-convert-from-char-on-ARM-build.patch b/meta-oe/recipes-test/catch2/catch2/0001-Fix-convert-from-char-opython3nativen-ARM-build.patch
> > new file mode 100644
> > index 000000000..307377563
> > --- /dev/null
> > +++ b/meta-oe/recipes-test/catch2/catch2/0001-Fix-convert-from-char-on-ARM-build.patch
> > @@ -0,0 +1,46 @@
> > +From 5729874fa5059c39aa97cfa08fddf107b7a65c9e Mon Sep 17 00:00:00 2001
> > +From: Miguel Gaio <mgaio35 at gmail.com>
> > +Date: Wed, 3 Oct 2018 10:22:16 +0200
> > +Subject: [PATCH] Fix convert from char on ARM build
> > +
> > +Some platforms set the signedness of char to unsigned (eg. ARM).
> > +Convert from char should not assume the signedness of char.
> > +
> > +Fix build issue with -Werror,-Wtautological-unsigned-zero-compare flags.
> > +
> > +Signed-off-by: Miguel Gaio <mgaio35 at gmail.com>
> > +
> > +Upstream-Status: Accepted [Commit f1faaa9c107113692301ad8bb56084460ef1a2ff]
> > +
> > +Signed-off-by: Bartosz Golaszewski <bgolaszewski at baylibre.com>
> > +---
> > + include/internal/catch_tostring.cpp | 6 +++---
> > + 1 file changed, 3 insertions(+), 3 deletions(-)
> > +
> > +diff --git a/include/internal/catch_tostring.cpp b/include/internal/catch_tostring.cpp
> > +index 4e0c027d..8cbabbf2 100644
> > +--- a/include/internal/catch_tostring.cpp
> > ++++ b/include/internal/catch_tostring.cpp
> > +@@ -205,7 +205,7 @@ std::string StringMaker<bool>::convert(bool b) {
> > +     return b ? "true" : "false";
> > + }
> > +
> > +-std::string StringMaker<char>::convert(char value) {
> > ++std::string StringMaker<signed char>::convert(signed char value) {
> > +     if (value == '\r') {
> > +         return "'\\r'";
> > +     } else if (value == '\f') {
> > +@@ -222,8 +222,8 @@ std::string StringMaker<char>::convert(char value) {
> > +         return chstr;
> > +     }
> > + }
> > +-std::string StringMaker<signed char>::convert(signed char c) {
> > +-    return ::Catch::Detail::stringify(static_cast<char>(c));
> > ++std::string StringMaker<char>::convert(char c) {
> > ++    return ::Catch::Detail::stringify(static_cast<signed char>(c));
> > + }
> > + std::string StringMaker<unsigned char>::convert(unsigned char c) {
> > +     return ::Catch::Detail::stringify(static_cast<char>(c));
> > +--
> > +2.19.1
> > +
> > diff --git a/meta-oe/recipes-test/catch2/catch2_2.4.1.bb b/meta-oe/recipes-test/catch2/catch2_2.4.1.bb
> > new file mode 100644
> > index 000000000..370ade150
> > --- /dev/null
> > +++ b/meta-oe/recipes-test/catch2/catch2_2.4.1.bb
> > @@ -0,0 +1,26 @@
> > +DESCRIPTION = "A modern, C++-native, header-only, test framework for unit-tests, \
> > +TDD and BDD - using C++11, C++14, C++17 and later."
> > +AUTHOR = "Phil Nash, Martin Horenovsky and others"
> > +HOMEPAGE = "https://github.com/catchorg/Catch2"
> > +LICENSE = "BSL-1.0"
> > +LIC_FILES_CHKSUM = "file://LICENSE.txt;md5=e4224ccaecb14d942c71d31bef20d78c"
> > +
> > +SRC_URI = "git://github.com/catchorg/Catch2.git \
> > +           file://0001-Fix-convert-from-char-on-ARM-build.patch"
> > +SRCREV = "9e1bdca4667295fcb16265eae00efa8423f07007"
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +DEPENDS = "python3-native"
>
> this is redundant since python3native is being inherited.
>

But if I don't do it, cmake fails:

| -- Detecting CXX compiler ABI info
| -- Detecting CXX compiler ABI info - done
| -- Detecting CXX compile features
| -- Detecting CXX compile features - done
| -- Could NOT find PythonInterp (missing: PYTHON_EXECUTABLE)
| CMake Error at CMakeLists.txt:43 (message):
|   Python not found, but required for tests
|
|
| -- Configuring incomplete, errors occurred!

Specifically this macro: find_package(PythonInterp)

Bart

> > +
> > +inherit cmake python3native
> > +
> > +# Header-only library
> > +RDEPENDS_${PN}-dev = ""
> > +RRECOMMENDS_${PN}-dbg = "${PN}-dev (= ${EXTENDPKGV})"
> > +
> > +do_install_append() {
> > +    rm ${D}/${datadir}/Catch2/lldbinit
> > +    rm ${D}/${datadir}/Catch2/gdbinit
> > +    rmdir ${D}/${datadir}/Catch2/
>
> why not just delete whole dir in one go ?
>
> > +}
> > --
> > 2.19.1
> >