[meta-oe][PATCH 0/6] sigrok: set of new packages

[meta-oe][PATCH 0/6] sigrok: set of new packages

[Bartosz Golaszewski]

This series adds recipes for libsigrok and accompanying libraries and
tools. The eventual goal is to add pulseview - the graphical interface
for sigrok - to meta-qt5-extra.

First two patches add external packages used by libsigrok. The third
patch adds a recipe for native doxygen, which is needed to generate
libsigrok C++ bindings which will in turn be necessary to build
pulseview.

Last three patches add recipes for the sigrok suite including the
command-line utility - sigrok-cli.

Bartosz Golaszewski (6):
  libzip: new package
  libserialport: new package
  doxygen: new native package
  libsigrok: new package
  libsigrokdecode: new package
  sigrok-cli: new package

 .../doxygen/doxygen-native_1.8.14.bb          | 15 ++++++++
 .../0001-build-don-t-look-for-Iconv.patch     | 36 +++++++++++++++++++
 .../libserialport/libserialport_0.1.1.bb      | 12 +++++++
 .../recipes-extended/libzip/libzip_1.5.1.bb   | 18 ++++++++++
 .../sigrok/libsigrok_0.5.1.bb                 | 23 ++++++++++++
 .../sigrok/libsigrokdecode_0.5.2.bb           | 14 ++++++++
 .../sigrok/sigrok-cli_0.7.0.bb                | 18 ++++++++++
 7 files changed, 136 insertions(+)
 create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
 create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
 create mode 100644 meta-oe/recipes-extended/libserialport/libserialport_0.1.1.bb
 create mode 100644 meta-oe/recipes-extended/libzip/libzip_1.5.1.bb
 create mode 100644 meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
 create mode 100644 meta-oe/recipes-extended/sigrok/libsigrokdecode_0.5.2.bb
 create mode 100644 meta-oe/recipes-extended/sigrok/sigrok-cli_0.7.0.bb

-- 
2.19.1

[meta-oe][PATCH 1/6] libzip: new package

[Bartosz Golaszewski]

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
---
 .../recipes-extended/libzip/libzip_1.5.1.bb    | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 meta-oe/recipes-extended/libzip/libzip_1.5.1.bb

diff --git a/meta-oe/recipes-extended/libzip/libzip_1.5.1.bb b/meta-oe/recipes-extended/libzip/libzip_1.5.1.bb
new file mode 100644
index 000000000..ce73700d7
--- /dev/null
+++ b/meta-oe/recipes-extended/libzip/libzip_1.5.1.bb
@@ -0,0 +1,18 @@
+DESCRIPTION = "libzip is a C library for reading, creating, and modifying zip archives."
+HOMEPAGE = "https://libzip.org/"
+
+LICENSE = "BSD-3-Clause"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=01f8b1b8da6403739094396e15b1e722"
+
+DEPENDS = "zlib bzip2"
+
+PACKAGECONFIG[ssl] = "-DENABLE_OPENSSL=ON,-DENABLE_OPENSSL=OFF,openssl"
+
+PACKAGECONFIG ?= "ssl"
+
+inherit cmake
+
+SRC_URI = "https://libzip.org/download/libzip-${PV}.tar.xz"
+
+SRC_URI[md5sum] = "6fe665aa6d6bf3a99eb6fa9c553283fd"
+SRC_URI[sha256sum] = "04ea35b6956c7b3453f1ed3f3fe40e3ddae1f43931089124579e8384e79ed372"
-- 
2.19.1

[meta-oe][PATCH 2/6] libserialport: new package

[Bartosz Golaszewski]

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
---
 .../libserialport/libserialport_0.1.1.bb             | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 meta-oe/recipes-extended/libserialport/libserialport_0.1.1.bb

diff --git a/meta-oe/recipes-extended/libserialport/libserialport_0.1.1.bb b/meta-oe/recipes-extended/libserialport/libserialport_0.1.1.bb
new file mode 100644
index 000000000..192d4bce6
--- /dev/null
+++ b/meta-oe/recipes-extended/libserialport/libserialport_0.1.1.bb
@@ -0,0 +1,12 @@
+DESCRIPTION = "libserialport is a minimal, cross-platform shared library written in C that is intended to take care of the OS-specific details when writing software that uses serial ports."
+HOMEPAGE = "https://sigrok.org/wiki/Libserialport"
+
+LICENSE = "LGPL-3.0+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=e6a600fd5e1d9cbde2d983680233ad02"
+
+inherit autotools
+
+SRC_URI = "http://sigrok.org/download/source/libserialport/libserialport-${PV}.tar.gz"
+
+SRC_URI[md5sum] = "b93f0325a6157198152b5bd7e8182b51"
+SRC_URI[sha256sum] = "4a2af9d9c3ff488e92fb75b4ba38b35bcf9b8a66df04773eba2a7bbf1fa7529d"
-- 
2.19.1

[meta-oe][PATCH 3/6] doxygen: new native package

[Bartosz Golaszewski]

Doxygen binary is necessary to generate C++ bindings for libsigrok.

We're adding this as a native-only package since it's very unlikely
it would be needed on any target.

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
---
 .../doxygen/doxygen-native_1.8.14.bb          | 15 ++++++++
 .../0001-build-don-t-look-for-Iconv.patch     | 36 +++++++++++++++++++
 2 files changed, 51 insertions(+)
 create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
 create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch

diff --git a/meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb b/meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
new file mode 100644
index 000000000..38157085e
--- /dev/null
+++ b/meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
@@ -0,0 +1,15 @@
+DESCRIPTION = "Doxygen is the de facto standard tool for generating documentation from annotated C++ sources."
+HOMEPAGE = "http://www.doxygen.org/"
+
+LICENSE = "GPL-2.0"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=b234ee4d69f5fce4486a80fdaf4a4263"
+
+inherit cmake python3native native
+
+DEPENDS = "flex-native bison-native"
+
+SRC_URI = "ftp://ftp.stack.nl/pub/users/dimitri/doxygen-${PV}.src.tar.gz \
+           file://0001-build-don-t-look-for-Iconv.patch"
+
+SRC_URI[md5sum] = "41d8821133e8d8104280030553e2b42b"
+SRC_URI[sha256sum] = "d1757e02755ef6f56fd45f1f4398598b920381948d6fcfa58f5ca6aa56f59d4d"
diff --git a/meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch b/meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
new file mode 100644
index 000000000..0eefe0729
--- /dev/null
+++ b/meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
@@ -0,0 +1,36 @@
+From f3a6c05e3a8afdac547c6c8176fa6b57636db65c Mon Sep 17 00:00:00 2001
+From: Bartosz Golaszewski <brgl@bgdev.pl>
+Date: Fri, 23 Nov 2018 11:44:56 +0100
+Subject: [PATCH] build: don't look for Iconv
+
+Drop the find_package() for Iconv. CMake is unable to find iconv.h in
+native build but all modern systems supply it as part of the standard
+C library. We don't need this check in meta-openembedded.
+
+Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
+
+Upstream-status: Inappropriate
+- upstream doxygen must build on many architectures, this change is
+  too intrusive for upstream
+---
+ CMakeLists.txt | 4 ----
+ 1 file changed, 4 deletions(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index ba105b89..16252247 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -89,10 +89,6 @@ if (sqlite3)
+     find_package(SQLite3 REQUIRED)
+ endif()
+ 
+-find_package(Iconv REQUIRED)
+-include_directories(${ICONV_INCLUDE_DIR})
+-
+-
+ #set(DOXYDOCS ${CMAKE_SOURCE_DIR}/doc CACHE INTERNAL "Path to doxygen docs")
+ set(DOXYDOCS ${PROJECT_BINARY_DIR}/doc)
+ set(ENV{DOXYGEN_DOCDIR} ${DOXYDOCS})
+-- 
+2.19.1
+
-- 
2.19.1

[meta-oe][PATCH 4/6] libsigrok: new package

[Bartosz Golaszewski]

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
---
 .../sigrok/libsigrok_0.5.1.bb                 | 23 +++++++++++++++++++
 1 file changed, 23 insertions(+)
 create mode 100644 meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb

diff --git a/meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb b/meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
new file mode 100644
index 000000000..b60c64c3b
--- /dev/null
+++ b/meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
@@ -0,0 +1,23 @@
+DESCRIPTION = "libsigrok is a shared library written in C, which provides the basic hardware access drivers for logic analyzers and other supported devices, as well as input/output file format support."
+HOMEPAGE = "http://sigrok.org/wiki/Main_Page"
+
+LICENSE = "GPLv3"
+LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
+
+DEPENDS = "glib-2.0 libzip"
+
+PACKAGECONFIG[serialport] = "--with-libserialport,--without-libserialport,libserialport"
+PACKAGECONFIG[ftdi] = "--with-libftdi,--without-libftdi,libftdi"
+PACKAGECONFIG[usb] = "--with-libusb,--without-libusb,libusb"
+PACKAGECONFIG[cxx] = "--enable-cxx,--disable-cxx,glibmm doxygen-native"
+
+PACKAGECONFIG ??= "serialport ftdi usb"
+
+inherit autotools pkgconfig
+
+SRC_URI = "http://sigrok.org/download/source/libsigrok/libsigrok-${PV}.tar.gz"
+
+SRC_URI[md5sum] = "a3de9e52a660e51d27a6aca025d204a7"
+SRC_URI[sha256sum] = "e40fde7af98d29e922e9d3cbe0a6c0569889153fc31e47b8b1afe4d846292b9c"
+
+FILES_${PN} += "/usr/share/*"
-- 
2.19.1

[meta-oe][PATCH 5/6] libsigrokdecode: new package

[Bartosz Golaszewski]

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
---
 .../sigrok/libsigrokdecode_0.5.2.bb                | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
 create mode 100644 meta-oe/recipes-extended/sigrok/libsigrokdecode_0.5.2.bb

diff --git a/meta-oe/recipes-extended/sigrok/libsigrokdecode_0.5.2.bb b/meta-oe/recipes-extended/sigrok/libsigrokdecode_0.5.2.bb
new file mode 100644
index 000000000..b8e1e4705
--- /dev/null
+++ b/meta-oe/recipes-extended/sigrok/libsigrokdecode_0.5.2.bb
@@ -0,0 +1,14 @@
+DESCRIPTION = "libsigrokdecode is a shared library written in C, which provides (streaming) protocol decoding functionality."
+HOMEPAGE = "http://sigrok.org/wiki/Main_Page"
+
+LICENSE = "GPLv3"
+LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
+
+DEPENDS = "glib-2.0 python3"
+
+inherit autotools pkgconfig
+
+SRC_URI = "http://sigrok.org/download/source/libsigrokdecode/libsigrokdecode-${PV}.tar.gz"
+
+SRC_URI[md5sum] = "b9033bc7e68bc17fffffd4fdd793f5a1"
+SRC_URI[sha256sum] = "e08d9e797c54eccf3144da631b6e5f1498ac531e51520428df537a1da82583f0"
-- 
2.19.1

[meta-oe][PATCH 6/6] sigrok-cli: new package

[Bartosz Golaszewski]

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
---
 .../sigrok/sigrok-cli_0.7.0.bb                 | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 meta-oe/recipes-extended/sigrok/sigrok-cli_0.7.0.bb

diff --git a/meta-oe/recipes-extended/sigrok/sigrok-cli_0.7.0.bb b/meta-oe/recipes-extended/sigrok/sigrok-cli_0.7.0.bb
new file mode 100644
index 000000000..d31bcd282
--- /dev/null
+++ b/meta-oe/recipes-extended/sigrok/sigrok-cli_0.7.0.bb
@@ -0,0 +1,18 @@
+DESCRIPTION = "sigrok-cli is a command-line frontend for sigrok."
+HOMEPAGE = "http://sigrok.org/wiki/Main_Page"
+
+LICENSE = "GPLv3"
+LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
+
+DEPENDS = "libsigrok"
+
+PACKAGECONFIG[decode] = "--with-libsigrokdecode,--without-libsigrokdecode,libsigrokdecode"
+
+PACKAGECONFIG ??= "decode"
+
+inherit autotools pkgconfig
+
+SRC_URI = "http://sigrok.org/download/source/sigrok-cli/sigrok-cli-${PV}.tar.gz"
+
+SRC_URI[md5sum] = "77cb745e2fa239c7bd1ea81e2d67ede9"
+SRC_URI[sha256sum] = "5669d968c2de3dfc6adfda76e83789b6ba76368407c832438cef5e7099a65e1c"
-- 
2.19.1

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Andreas Müller]

On Fri, Nov 23, 2018 at 4:39 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> This series adds recipes for libsigrok and accompanying libraries and
> tools. The eventual goal is to add pulseview - the graphical interface
> for sigrok - to meta-qt5-extra.
I had started this at meta-qt5-extra but... Did you prepare already
something for sigrok? If yes: Would be happy to receive your patches -
if no I can try the recipe already started together with the recipes
you sent here.

Andreas
>
> First two patches add external packages used by libsigrok. The third
> patch adds a recipe for native doxygen, which is needed to generate
> libsigrok C++ bindings which will in turn be necessary to build
> pulseview.
>
> Last three patches add recipes for the sigrok suite including the
> command-line utility - sigrok-cli.
>
> Bartosz Golaszewski (6):
>   libzip: new package
>   libserialport: new package
>   doxygen: new native package
>   libsigrok: new package
>   libsigrokdecode: new package
>   sigrok-cli: new package
>
>  .../doxygen/doxygen-native_1.8.14.bb          | 15 ++++++++
>  .../0001-build-don-t-look-for-Iconv.patch     | 36 +++++++++++++++++++
>  .../libserialport/libserialport_0.1.1.bb      | 12 +++++++
>  .../recipes-extended/libzip/libzip_1.5.1.bb   | 18 ++++++++++
>  .../sigrok/libsigrok_0.5.1.bb                 | 23 ++++++++++++
>  .../sigrok/libsigrokdecode_0.5.2.bb           | 14 ++++++++
>  .../sigrok/sigrok-cli_0.7.0.bb                | 18 ++++++++++
>  7 files changed, 136 insertions(+)
>  create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
>  create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
>  create mode 100644 meta-oe/recipes-extended/libserialport/libserialport_0.1.1.bb
>  create mode 100644 meta-oe/recipes-extended/libzip/libzip_1.5.1.bb
>  create mode 100644 meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
>  create mode 100644 meta-oe/recipes-extended/sigrok/libsigrokdecode_0.5.2.bb
>  create mode 100644 meta-oe/recipes-extended/sigrok/sigrok-cli_0.7.0.bb
>
> --
> 2.19.1
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [meta-oe][PATCH 3/6] doxygen: new native package

[Khem Raj]

I think its better to have doxygen for target as well. So probably use
bbclassextend for native see

http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/tree/recipes-devtools/doxygen/doxygen_1.8.9.1.bb?h=master

Once you merge the changes from this into your patch. Also send a
patch to meta-ti to remove it,

On Fri, Nov 23, 2018 at 7:39 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> Doxygen binary is necessary to generate C++ bindings for libsigrok.
>
> We're adding this as a native-only package since it's very unlikely
> it would be needed on any target.
>
> Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
> ---
>  .../doxygen/doxygen-native_1.8.14.bb          | 15 ++++++++
>  .../0001-build-don-t-look-for-Iconv.patch     | 36 +++++++++++++++++++
>  2 files changed, 51 insertions(+)
>  create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
>  create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
>
> diff --git a/meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb b/meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
> new file mode 100644
> index 000000000..38157085e
> --- /dev/null
> +++ b/meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
> @@ -0,0 +1,15 @@
> +DESCRIPTION = "Doxygen is the de facto standard tool for generating documentation from annotated C++ sources."
> +HOMEPAGE = "http://www.doxygen.org/"
> +
> +LICENSE = "GPL-2.0"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> +
> +inherit cmake python3native native
> +
> +DEPENDS = "flex-native bison-native"
> +
> +SRC_URI = "ftp://ftp.stack.nl/pub/users/dimitri/doxygen-${PV}.src.tar.gz \
> +           file://0001-build-don-t-look-for-Iconv.patch"
> +
> +SRC_URI[md5sum] = "41d8821133e8d8104280030553e2b42b"
> +SRC_URI[sha256sum] = "d1757e02755ef6f56fd45f1f4398598b920381948d6fcfa58f5ca6aa56f59d4d"
> diff --git a/meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch b/meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
> new file mode 100644
> index 000000000..0eefe0729
> --- /dev/null
> +++ b/meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
> @@ -0,0 +1,36 @@
> +From f3a6c05e3a8afdac547c6c8176fa6b57636db65c Mon Sep 17 00:00:00 2001
> +From: Bartosz Golaszewski <brgl@bgdev.pl>
> +Date: Fri, 23 Nov 2018 11:44:56 +0100
> +Subject: [PATCH] build: don't look for Iconv
> +
> +Drop the find_package() for Iconv. CMake is unable to find iconv.h in
> +native build but all modern systems supply it as part of the standard
> +C library. We don't need this check in meta-openembedded.
> +
> +Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
> +
> +Upstream-status: Inappropriate
> +- upstream doxygen must build on many architectures, this change is
> +  too intrusive for upstream
> +---
> + CMakeLists.txt | 4 ----
> + 1 file changed, 4 deletions(-)
> +
> +diff --git a/CMakeLists.txt b/CMakeLists.txt
> +index ba105b89..16252247 100644
> +--- a/CMakeLists.txt
> ++++ b/CMakeLists.txt
> +@@ -89,10 +89,6 @@ if (sqlite3)
> +     find_package(SQLite3 REQUIRED)
> + endif()
> +
> +-find_package(Iconv REQUIRED)
> +-include_directories(${ICONV_INCLUDE_DIR})
> +-
> +-
> + #set(DOXYDOCS ${CMAKE_SOURCE_DIR}/doc CACHE INTERNAL "Path to doxygen docs")
> + set(DOXYDOCS ${PROJECT_BINARY_DIR}/doc)
> + set(ENV{DOXYGEN_DOCDIR} ${DOXYDOCS})
> +--
> +2.19.1
> +
> --
> 2.19.1
>

Re: [meta-oe][PATCH 4/6] libsigrok: new package

[Khem Raj]

On Fri, Nov 23, 2018 at 7:39 AM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
> ---
>  .../sigrok/libsigrok_0.5.1.bb                 | 23 +++++++++++++++++++
>  1 file changed, 23 insertions(+)
>  create mode 100644 meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
>
> diff --git a/meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb b/meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
> new file mode 100644
> index 000000000..b60c64c3b
> --- /dev/null
> +++ b/meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
> @@ -0,0 +1,23 @@
> +DESCRIPTION = "libsigrok is a shared library written in C, which provides the basic hardware access drivers for logic analyzers and other supported devices, as well as input/output file format support."
> +HOMEPAGE = "http://sigrok.org/wiki/Main_Page"
> +
> +LICENSE = "GPLv3"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
> +
> +DEPENDS = "glib-2.0 libzip"
> +
> +PACKAGECONFIG[serialport] = "--with-libserialport,--without-libserialport,libserialport"
> +PACKAGECONFIG[ftdi] = "--with-libftdi,--without-libftdi,libftdi"
> +PACKAGECONFIG[usb] = "--with-libusb,--without-libusb,libusb"
> +PACKAGECONFIG[cxx] = "--enable-cxx,--disable-cxx,glibmm doxygen-native"
> +
> +PACKAGECONFIG ??= "serialport ftdi usb"
> +
> +inherit autotools pkgconfig
> +
> +SRC_URI = "http://sigrok.org/download/source/libsigrok/libsigrok-${PV}.tar.gz"
> +
> +SRC_URI[md5sum] = "a3de9e52a660e51d27a6aca025d204a7"
> +SRC_URI[sha256sum] = "e40fde7af98d29e922e9d3cbe0a6c0569889153fc31e47b8b1afe4d846292b9c"
> +
> +FILES_${PN} += "/usr/share/*"

may be use ${datadir} here

> --
> 2.19.1
>

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Bartosz Golaszewski]

pt., 23 lis 2018 o 22:00 Andreas Müller <schnitzeltony@gmail.com> napisał(a):
>
> On Fri, Nov 23, 2018 at 4:39 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> >
> > This series adds recipes for libsigrok and accompanying libraries and
> > tools. The eventual goal is to add pulseview - the graphical interface
> > for sigrok - to meta-qt5-extra.
> I had started this at meta-qt5-extra but... Did you prepare already
> something for sigrok? If yes: Would be happy to receive your patches -
> if no I can try the recipe already started together with the recipes
> you sent here.
>
> Andreas

I'm not sure I understood you - do you mean if I prepared something
for pulseview for meta-qt5-extra or if I want to put libsigrok
packages in meta-qt5-extra? For the latter: I think that the right
place for those packages in meta-openembedded and only pulseview
should go into meta-qt5-extra. For the former: I'll be creating the
recipe for pulseview on Monday probably.

Bart

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Andreas Müller]

On Sat, Nov 24, 2018 at 9:11 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> pt., 23 lis 2018 o 22:00 Andreas Müller <schnitzeltony@gmail.com> napisał(a):
> >
> > On Fri, Nov 23, 2018 at 4:39 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > >
> > > This series adds recipes for libsigrok and accompanying libraries and
> > > tools. The eventual goal is to add pulseview - the graphical interface
> > > for sigrok - to meta-qt5-extra.
> > I had started this at meta-qt5-extra but... Did you prepare already
> > something for sigrok? If yes: Would be happy to receive your patches -
> > if no I can try the recipe already started together with the recipes
> > you sent here.
> >
> > Andreas
>
> I'm not sure I understood you - do you mean if I prepared something
> for pulseview for meta-qt5-extra or if I want to put libsigrok
> packages in meta-qt5-extra? For the latter: I think that the right
> place for those packages in meta-openembedded and only pulseview
> should go into meta-qt5-extra. For the former: I'll be creating the
> recipe for pulseview on Monday probably.
>
> Bart
I meant first option. Like your suggestion you prepare and I wait - no
need to get my unfinished work polished.

Andreas

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Nathan Rossi]

On Sat, 24 Nov 2018 at 01:39, Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> This series adds recipes for libsigrok and accompanying libraries and
> tools. The eventual goal is to add pulseview - the graphical interface
> for sigrok - to meta-qt5-extra.
>
> First two patches add external packages used by libsigrok. The third
> patch adds a recipe for native doxygen, which is needed to generate
> libsigrok C++ bindings which will in turn be necessary to build
> pulseview.
>
> Last three patches add recipes for the sigrok suite including the
> command-line utility - sigrok-cli.
>
> Bartosz Golaszewski (6):
>   libzip: new package
>   libserialport: new package
>   doxygen: new native package
>   libsigrok: new package
>   libsigrokdecode: new package
>   sigrok-cli: new package

Anything against marking all the recipes (excluding doxygen) with
BBCLASSEXTEND = "native nativesdk"?

Also any plans to include recipes for any of the firmware data
packages that sigrok provides? (e.g. sigrok-firmware-fx2lafw)

Thanks,
Nathan

>
>  .../doxygen/doxygen-native_1.8.14.bb          | 15 ++++++++
>  .../0001-build-don-t-look-for-Iconv.patch     | 36 +++++++++++++++++++
>  .../libserialport/libserialport_0.1.1.bb      | 12 +++++++
>  .../recipes-extended/libzip/libzip_1.5.1.bb   | 18 ++++++++++
>  .../sigrok/libsigrok_0.5.1.bb                 | 23 ++++++++++++
>  .../sigrok/libsigrokdecode_0.5.2.bb           | 14 ++++++++
>  .../sigrok/sigrok-cli_0.7.0.bb                | 18 ++++++++++
>  7 files changed, 136 insertions(+)
>  create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen-native_1.8.14.bb
>  create mode 100644 meta-oe/recipes-devtools/doxygen/doxygen/0001-build-don-t-look-for-Iconv.patch
>  create mode 100644 meta-oe/recipes-extended/libserialport/libserialport_0.1.1.bb
>  create mode 100644 meta-oe/recipes-extended/libzip/libzip_1.5.1.bb
>  create mode 100644 meta-oe/recipes-extended/sigrok/libsigrok_0.5.1.bb
>  create mode 100644 meta-oe/recipes-extended/sigrok/libsigrokdecode_0.5.2.bb
>  create mode 100644 meta-oe/recipes-extended/sigrok/sigrok-cli_0.7.0.bb
>
> --
> 2.19.1
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Bartosz Golaszewski]

niedz., 25 lis 2018 o 07:04 Nathan Rossi <nathan@nathanrossi.com> napisał(a):
>
> On Sat, 24 Nov 2018 at 01:39, Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> >
> > This series adds recipes for libsigrok and accompanying libraries and
> > tools. The eventual goal is to add pulseview - the graphical interface
> > for sigrok - to meta-qt5-extra.
> >
> > First two patches add external packages used by libsigrok. The third
> > patch adds a recipe for native doxygen, which is needed to generate
> > libsigrok C++ bindings which will in turn be necessary to build
> > pulseview.
> >
> > Last three patches add recipes for the sigrok suite including the
> > command-line utility - sigrok-cli.
> >
> > Bartosz Golaszewski (6):
> >   libzip: new package
> >   libserialport: new package
> >   doxygen: new native package
> >   libsigrok: new package
> >   libsigrokdecode: new package
> >   sigrok-cli: new package
>
> Anything against marking all the recipes (excluding doxygen) with
> BBCLASSEXTEND = "native nativesdk"?
>

What would be the reason for that? Also: you'd need to do that for ALL
non-direct sigrok dependencies and I think there are quite a few.

> Also any plans to include recipes for any of the firmware data
> packages that sigrok provides? (e.g. sigrok-firmware-fx2lafw)
>

I don't personally need it and it seems like it requires integrating
the sdcc compiler into yocto. Probably more work than I can spend on
that.

Best regards,
Bart

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Nathan Rossi]

On Mon, 26 Nov 2018 at 01:12, Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> niedz., 25 lis 2018 o 07:04 Nathan Rossi <nathan@nathanrossi.com> napisał(a):
> >
> > On Sat, 24 Nov 2018 at 01:39, Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > >
> > > This series adds recipes for libsigrok and accompanying libraries and
> > > tools. The eventual goal is to add pulseview - the graphical interface
> > > for sigrok - to meta-qt5-extra.
> > >
> > > First two patches add external packages used by libsigrok. The third
> > > patch adds a recipe for native doxygen, which is needed to generate
> > > libsigrok C++ bindings which will in turn be necessary to build
> > > pulseview.
> > >
> > > Last three patches add recipes for the sigrok suite including the
> > > command-line utility - sigrok-cli.
> > >
> > > Bartosz Golaszewski (6):
> > >   libzip: new package
> > >   libserialport: new package
> > >   doxygen: new native package
> > >   libsigrok: new package
> > >   libsigrokdecode: new package
> > >   sigrok-cli: new package
> >
> > Anything against marking all the recipes (excluding doxygen) with
> > BBCLASSEXTEND = "native nativesdk"?
> >
>
> What would be the reason for that? Also: you'd need to do that for ALL
> non-direct sigrok dependencies and I think there are quite a few.

It makes it really easy to ship with a nativesdk toolset or use on
your build host. I have personally been using it this way and made
some recipes for it a few months ago
(https://github.com/nathanrossi/meta-random/tree/master/recipes-toolbox/sigrok).

With the recipes in this series the only dependency that is not also
enabled for native/nativesdk is glibmm. However that is not enabled by
default (since its a dependency of 'cxx' config).

>
> > Also any plans to include recipes for any of the firmware data
> > packages that sigrok provides? (e.g. sigrok-firmware-fx2lafw)
> >
>
> I don't personally need it and it seems like it requires integrating
> the sdcc compiler into yocto. Probably more work than I can spend on
> that.

I was only referring to the pre-built firmware binaries that are
released by the sigrok project.
https://sigrok.org/download/binary/sigrok-firmware-fx2lafw/.

Regards,
Nathan

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Bartosz Golaszewski]

niedz., 25 lis 2018 o 16:40 Nathan Rossi <nathan@nathanrossi.com> napisał(a):
>
> On Mon, 26 Nov 2018 at 01:12, Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> >
> > niedz., 25 lis 2018 o 07:04 Nathan Rossi <nathan@nathanrossi.com> napisał(a):
> > >
> > > On Sat, 24 Nov 2018 at 01:39, Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > > >
> > > > This series adds recipes for libsigrok and accompanying libraries and
> > > > tools. The eventual goal is to add pulseview - the graphical interface
> > > > for sigrok - to meta-qt5-extra.
> > > >
> > > > First two patches add external packages used by libsigrok. The third
> > > > patch adds a recipe for native doxygen, which is needed to generate
> > > > libsigrok C++ bindings which will in turn be necessary to build
> > > > pulseview.
> > > >
> > > > Last three patches add recipes for the sigrok suite including the
> > > > command-line utility - sigrok-cli.
> > > >
> > > > Bartosz Golaszewski (6):
> > > >   libzip: new package
> > > >   libserialport: new package
> > > >   doxygen: new native package
> > > >   libsigrok: new package
> > > >   libsigrokdecode: new package
> > > >   sigrok-cli: new package
> > >
> > > Anything against marking all the recipes (excluding doxygen) with
> > > BBCLASSEXTEND = "native nativesdk"?
> > >
> >
> > What would be the reason for that? Also: you'd need to do that for ALL
> > non-direct sigrok dependencies and I think there are quite a few.
>
> It makes it really easy to ship with a nativesdk toolset or use on
> your build host. I have personally been using it this way and made
> some recipes for it a few months ago
> (https://github.com/nathanrossi/meta-random/tree/master/recipes-toolbox/sigrok).
>
> With the recipes in this series the only dependency that is not also
> enabled for native/nativesdk is glibmm. However that is not enabled by
> default (since its a dependency of 'cxx' config).
>

I guess it can always be added later on. I'll stick to target packages for now.

> >
> > > Also any plans to include recipes for any of the firmware data
> > > packages that sigrok provides? (e.g. sigrok-firmware-fx2lafw)
> > >
> >
> > I don't personally need it and it seems like it requires integrating
> > the sdcc compiler into yocto. Probably more work than I can spend on
> > that.
>
> I was only referring to the pre-built firmware binaries that are
> released by the sigrok project.
> https://sigrok.org/download/binary/sigrok-firmware-fx2lafw/.
>

Same as above, no problem adding it later.

Bart

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Bartosz Golaszewski]

sob., 24 lis 2018 o 22:37 Andreas Müller <schnitzeltony@gmail.com> napisał(a):
>
> On Sat, Nov 24, 2018 at 9:11 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> >
> > pt., 23 lis 2018 o 22:00 Andreas Müller <schnitzeltony@gmail.com> napisał(a):
> > >
> > > On Fri, Nov 23, 2018 at 4:39 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > > >
> > > > This series adds recipes for libsigrok and accompanying libraries and
> > > > tools. The eventual goal is to add pulseview - the graphical interface
> > > > for sigrok - to meta-qt5-extra.
> > > I had started this at meta-qt5-extra but... Did you prepare already
> > > something for sigrok? If yes: Would be happy to receive your patches -
> > > if no I can try the recipe already started together with the recipes
> > > you sent here.
> > >
> > > Andreas
> >
> > I'm not sure I understood you - do you mean if I prepared something
> > for pulseview for meta-qt5-extra or if I want to put libsigrok
> > packages in meta-qt5-extra? For the latter: I think that the right
> > place for those packages in meta-openembedded and only pulseview
> > should go into meta-qt5-extra. For the former: I'll be creating the
> > recipe for pulseview on Monday probably.
> >
> > Bart
> I meant first option. Like your suggestion you prepare and I wait - no
> need to get my unfinished work polished.
>
> Andreas

This[1] recipe is enough to build pulseview, but I'm not sure how to
go about requiring libsigrok to have cxx set in PACKAGECONFIG. Should
I add a bbappend for libsigrok that would do PACKAGECONFIG_append = "
cxx"?

Bart

[1] https://github.com/brgl/meta-qt5-extra/commit/06852587c29de7039098f9dfce52b6b4ba13b766

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Andreas Müller]

On Mon, Nov 26, 2018 at 5:26 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> This[1] recipe is enough to build pulseview, but I'm not sure how to
> go about requiring libsigrok to have cxx set in PACKAGECONFIG. Should
> I add a bbappend for libsigrok that would do PACKAGECONFIG_append = "
> cxx"?
Have no background but why not enable it by default? Both dependencies
are in meta-oe so layer trouble is not to be expected.

Since your patch is already in master-next: I am fine with an append
in meta-qt5-extra.

I suggest once this series is in master please send me a pull request
- will test and create thud for meta-qt5-extra (don't expect this
series to land in meta-oe thud..)

Andreas
>
> Bart
>
> [1] https://github.com/brgl/meta-qt5-extra/commit/06852587c29de7039098f9dfce52b6b4ba13b766

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Bartosz Golaszewski]

pon., 26 lis 2018 o 23:49 Andreas Müller <schnitzeltony@gmail.com> napisał(a):
>
> On Mon, Nov 26, 2018 at 5:26 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > This[1] recipe is enough to build pulseview, but I'm not sure how to
> > go about requiring libsigrok to have cxx set in PACKAGECONFIG. Should
> > I add a bbappend for libsigrok that would do PACKAGECONFIG_append = "
> > cxx"?
> Have no background but why not enable it by default? Both dependencies
> are in meta-oe so layer trouble is not to be expected.
>
> Since your patch is already in master-next: I am fine with an append
> in meta-qt5-extra.
>
> I suggest once this series is in master please send me a pull request
> - will test and create thud for meta-qt5-extra (don't expect this
> series to land in meta-oe thud..)
>

PR sent.

Bart

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Andreas Müller]

On Fri, Dec 7, 2018 at 11:00 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> pon., 26 lis 2018 o 23:49 Andreas Müller <schnitzeltony@gmail.com> napisał(a):
> >
> > On Mon, Nov 26, 2018 at 5:26 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > > This[1] recipe is enough to build pulseview, but I'm not sure how to
> > > go about requiring libsigrok to have cxx set in PACKAGECONFIG. Should
> > > I add a bbappend for libsigrok that would do PACKAGECONFIG_append = "
> > > cxx"?
> > Have no background but why not enable it by default? Both dependencies
> > are in meta-oe so layer trouble is not to be expected.
> >
> > Since your patch is already in master-next: I am fine with an append
> > in meta-qt5-extra.
> >
> > I suggest once this series is in master please send me a pull request
> > - will test and create thud for meta-qt5-extra (don't expect this
> > series to land in meta-oe thud..)
> >
>
> PR sent.
>
> Bart
LOL - am testing build currently. Off topic: doxgen ftp download fails
- try http...

Andreas

Re: [meta-oe][PATCH 0/6] sigrok: set of new packages

[Khem Raj]

On Fri, Dec 7, 2018 at 2:30 PM Andreas Müller <schnitzeltony@gmail.com> wrote:
>
> On Fri, Dec 7, 2018 at 11:00 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> >
> > pon., 26 lis 2018 o 23:49 Andreas Müller <schnitzeltony@gmail.com> napisał(a):
> > >
> > > On Mon, Nov 26, 2018 at 5:26 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > > > This[1] recipe is enough to build pulseview, but I'm not sure how to
> > > > go about requiring libsigrok to have cxx set in PACKAGECONFIG. Should
> > > > I add a bbappend for libsigrok that would do PACKAGECONFIG_append = "
> > > > cxx"?
> > > Have no background but why not enable it by default? Both dependencies
> > > are in meta-oe so layer trouble is not to be expected.
> > >
> > > Since your patch is already in master-next: I am fine with an append
> > > in meta-qt5-extra.
> > >
> > > I suggest once this series is in master please send me a pull request
> > > - will test and create thud for meta-qt5-extra (don't expect this
> > > series to land in meta-oe thud..)
> > >
> >
> > PR sent.
> >
> > Bart
> LOL - am testing build currently. Off topic: doxgen ftp download fails
> - try http...

yes I have fixed it locally see
https://github.com/YoeDistro/meta-openembedded/commit/ff6bead1624a1e261408516b3d064a04aab5f592

>
> Andreas