* [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package
@ 2015-02-20 12:28 Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 1/7] libsigrok: remove --disable-udev from CONF_OPTS Bartosz Golaszewski
` (7 more replies)
0 siblings, 8 replies; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:28 UTC (permalink / raw)
To: buildroot
This series fixes a couple of bugs which have been noticed on the sigrok
mailing list, adds some needed infrastructure in order to be able to build
pulseview (the graphical front-end for sigrok) and the pulseview package
itself.
The first patch removes an unused flag from libsigrok.mk. The second
modifies the help text for libserialport to indicate to users that it's
a reusable library not tied to the sigrok project in any way. The third
patch makes it possible to compile libsigrok without libserialport support.
The fourth adds autoconf-archive host package needed to build libsigrok C++
bindings. The fifth updates the commit tag to checkout as several things
have been merged since the previous version - BayLibre ACME driver has
been merged and several bugs preventing buildroot from building pulseview
have been fixed. The sixth patch adds the option of building libsigrokcxx.
Finally the seventh patch adds the pulseview package.
Please note, that pulseview can be built using Qt4 as well - this option
should be added in the future.
Bartosz Golaszewski (7):
libsigrok: remove --disable-udev from CONF_OPTS
libserialport: drop untrue statement from Config.in help
libsigrok: don't select libserialport in Config.in
autoconf-archive: new package
libsigrok: bump git commit tag
libsigrok: enable building C++ bindings
pulseview: new package
package/Config.in | 1 +
package/autoconf-archive/autoconf-archive.mk | 13 ++++++++++++
package/libserialport/Config.in | 7 ++++---
package/libsigrok/Config.in | 16 +++++++++++++-
package/libsigrok/libsigrok.mk | 18 ++++++++++++++--
package/pulseview/Config.in | 31 ++++++++++++++++++++++++++++
package/pulseview/pulseview.mk | 24 +++++++++++++++++++++
package/sigrok-cli/Config.in | 1 -
8 files changed, 104 insertions(+), 7 deletions(-)
create mode 100644 package/autoconf-archive/autoconf-archive.mk
create mode 100644 package/pulseview/Config.in
create mode 100644 package/pulseview/pulseview.mk
--
2.1.4
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/7] libsigrok: remove --disable-udev from CONF_OPTS
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
@ 2015-02-20 12:28 ` Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 2/7] libserialport: drop untrue statement from Config.in help Bartosz Golaszewski
` (6 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:28 UTC (permalink / raw)
To: buildroot
This option is invalid and thus ignored by libsigrok configure.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
package/libsigrok/libsigrok.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/package/libsigrok/libsigrok.mk b/package/libsigrok/libsigrok.mk
index f91b8cb..109d4d3 100644
--- a/package/libsigrok/libsigrok.mk
+++ b/package/libsigrok/libsigrok.mk
@@ -13,7 +13,7 @@ LIBSIGROK_LICENSE_FILES = COPYING
LIBSIGROK_AUTORECONF = YES
LIBSIGROK_INSTALL_STAGING = YES
LIBSIGROK_DEPENDENCIES = libglib2 libzip host-pkgconf
-LIBSIGROK_CONF_OPTS = --disable-libudev --disable-bindings --disable-glibtest
+LIBSIGROK_CONF_OPTS = --disable-bindings --disable-glibtest
define LIBSIGROK_ADD_MISSING
mkdir -p $(@D)/autostuff
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 2/7] libserialport: drop untrue statement from Config.in help
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 1/7] libsigrok: remove --disable-udev from CONF_OPTS Bartosz Golaszewski
@ 2015-02-20 12:28 ` Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 3/7] libsigrok: don't select libserialport in Config.in Bartosz Golaszewski
` (5 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:28 UTC (permalink / raw)
To: buildroot
According to http://sigrok.org/wiki/Libserialport:
Note: While libserialport is hosted on sigrok.org (and sigrok
uses libserialport), this is a completely independent library
that can be used by other projects as well. The libserialport
library does not depend on any sigrok related libraries or
projects.
Drop the fragment about being a part of the sigrok suite and
extend the help text.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
package/libserialport/Config.in | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/package/libserialport/Config.in b/package/libserialport/Config.in
index 1e2935f..d821830 100644
--- a/package/libserialport/Config.in
+++ b/package/libserialport/Config.in
@@ -1,8 +1,9 @@
config BR2_PACKAGE_LIBSERIALPORT
bool "libserialport"
help
- Serial port access library.
-
- This library is a part of the sigrok software suite.
+ 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.
http://sigrok.org/wiki/Libserialport
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 3/7] libsigrok: don't select libserialport in Config.in
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 1/7] libsigrok: remove --disable-udev from CONF_OPTS Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 2/7] libserialport: drop untrue statement from Config.in help Bartosz Golaszewski
@ 2015-02-20 12:28 ` Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 4/7] autoconf-archive: new package Bartosz Golaszewski
` (4 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:28 UTC (permalink / raw)
To: buildroot
Libsigrok can be built without libserialport. Don't select it
in Config.in from both libsigrok and sigrok-cli and add a check
to libsigrok.mk to determine whether libserialport should be
enabled.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
package/libsigrok/Config.in | 1 -
package/libsigrok/libsigrok.mk | 7 +++++++
package/sigrok-cli/Config.in | 1 -
3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/package/libsigrok/Config.in b/package/libsigrok/Config.in
index 09077fb..47f812c 100644
--- a/package/libsigrok/Config.in
+++ b/package/libsigrok/Config.in
@@ -1,6 +1,5 @@
config BR2_PACKAGE_LIBSIGROK
bool "libsigrok"
- select BR2_PACKAGE_LIBSERIALPORT
select BR2_PACKAGE_LIBZIP
select BR2_PACKAGE_LIBGLIB2
# libglib2
diff --git a/package/libsigrok/libsigrok.mk b/package/libsigrok/libsigrok.mk
index 109d4d3..d19c3c8 100644
--- a/package/libsigrok/libsigrok.mk
+++ b/package/libsigrok/libsigrok.mk
@@ -21,6 +21,13 @@ endef
LIBSIGROK_PRE_CONFIGURE_HOOKS += LIBSIGROK_ADD_MISSING
+ifeq ($(BR2_PACKAGE_LIBSERIALPORT),y)
+LIBSIGROK_CONF_OPTS += --enable-libserialport
+LIBSIGROK_DEPENDENCIES += libserialport
+else
+LIBSIGROK_CONF_OPTS += --disable-libserialport
+endif
+
ifeq ($(BR2_PACKAGE_LIBFTDI),y)
LIBSIGROK_CONF_OPTS += --enable-libftdi
LIBSIGROK_DEPENDENCIES += libftdi
diff --git a/package/sigrok-cli/Config.in b/package/sigrok-cli/Config.in
index e3e5420..4e7f137 100644
--- a/package/sigrok-cli/Config.in
+++ b/package/sigrok-cli/Config.in
@@ -1,6 +1,5 @@
config BR2_PACKAGE_SIGROK_CLI
bool "sigrok-cli"
- select BR2_PACKAGE_LIBSERIALPORT
select BR2_PACKAGE_LIBSIGROK
# libsigrok->libglib2:
depends on BR2_USE_WCHAR
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 4/7] autoconf-archive: new package
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
` (2 preceding siblings ...)
2015-02-20 12:28 ` [Buildroot] [PATCH 3/7] libsigrok: don't select libserialport in Config.in Bartosz Golaszewski
@ 2015-02-20 12:28 ` Bartosz Golaszewski
2015-02-20 18:28 ` Thomas Petazzoni
2015-02-20 12:29 ` [Buildroot] [PATCH 5/7] libsigrok: bump git commit tag Bartosz Golaszewski
` (3 subsequent siblings)
7 siblings, 1 reply; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:28 UTC (permalink / raw)
To: buildroot
This package contains a collection of freely re-usable autoconf macros.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
package/autoconf-archive/autoconf-archive.mk | 13 +++++++++++++
1 file changed, 13 insertions(+)
create mode 100644 package/autoconf-archive/autoconf-archive.mk
diff --git a/package/autoconf-archive/autoconf-archive.mk b/package/autoconf-archive/autoconf-archive.mk
new file mode 100644
index 0000000..e11df28
--- /dev/null
+++ b/package/autoconf-archive/autoconf-archive.mk
@@ -0,0 +1,13 @@
+################################################################################
+#
+# autoconf-archive
+#
+################################################################################
+
+AUTOCONF_ARCHIVE_VERSION = 2015.02.04
+AUTOCONF_ARCHIVE_SOURCE = autoconf-archive-$(AUTOCONF_ARCHIVE_VERSION).tar.xz
+AUTOCONF_ARCHIVE_SITE = http://gnu.cardse.net/autoconf-archive/
+AUTOCONF_ARCHIVE_LICENSE = GPLv3+
+AUTOCONF_ARCHIVE_LICENSE_FILES = COPYING
+
+$(eval $(host-autotools-package))
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 5/7] libsigrok: bump git commit tag
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
` (3 preceding siblings ...)
2015-02-20 12:28 ` [Buildroot] [PATCH 4/7] autoconf-archive: new package Bartosz Golaszewski
@ 2015-02-20 12:29 ` Bartosz Golaszewski
2015-02-20 12:29 ` [Buildroot] [PATCH 6/7] libsigrok: enable building C++ bindings Bartosz Golaszewski
` (2 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:29 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
package/libsigrok/libsigrok.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/package/libsigrok/libsigrok.mk b/package/libsigrok/libsigrok.mk
index d19c3c8..6bc210e 100644
--- a/package/libsigrok/libsigrok.mk
+++ b/package/libsigrok/libsigrok.mk
@@ -4,7 +4,7 @@
#
################################################################################
-LIBSIGROK_VERSION = 8656a71790133d4de42252a1e75b4209c03b4983
+LIBSIGROK_VERSION = 380ee96fdfe0895ca0aa0b158d5c332ef08f8b3c
# No https access on upstream git
LIBSIGROK_SITE = git://sigrok.org/libsigrok
LIBSIGROK_LICENSE = GPLv3+
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 6/7] libsigrok: enable building C++ bindings
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
` (4 preceding siblings ...)
2015-02-20 12:29 ` [Buildroot] [PATCH 5/7] libsigrok: bump git commit tag Bartosz Golaszewski
@ 2015-02-20 12:29 ` Bartosz Golaszewski
2015-02-20 18:30 ` Thomas Petazzoni
2015-02-20 12:29 ` [Buildroot] [PATCH 7/7] pulseview: new package Bartosz Golaszewski
2015-02-20 18:15 ` [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Thomas Petazzoni
7 siblings, 1 reply; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:29 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
package/libsigrok/Config.in | 15 +++++++++++++++
package/libsigrok/libsigrok.mk | 9 ++++++++-
2 files changed, 23 insertions(+), 1 deletion(-)
diff --git a/package/libsigrok/Config.in b/package/libsigrok/Config.in
index 47f812c..8f7e36f 100644
--- a/package/libsigrok/Config.in
+++ b/package/libsigrok/Config.in
@@ -16,6 +16,21 @@ config BR2_PACKAGE_LIBSIGROK
http://sigrok.org/wiki/Libsigrok
+config BR2_PACKAGE_LIBSIGROKCXX
+ bool "build C++ bindings"
+ select BR2_PACKAGE_GLIBMM
+ depends on BR2_PACKAGE_LIBSIGROK
+ depends on BR2_TOOLCHAIN_BUILDROOT_CXX
+ # C++11 support
+ depends on BR2_TOOLCHAIN_BUILDROOT_GLIBC || BR2_TOOLCHAIN_BUILDROOT_EGLIBC
+ help
+ Build libsigrok C++ bindings as well.
+
comment "libsigrok needs a toolchain w/ wchar, threads"
depends on BR2_USE_MMU
depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
+
+comment "libsigrokcxx needs a toolchain w/ C++, (e)glibc"
+ depends on BR2_PACKAGE_LIBSIGROK
+ depends on !BR2_TOOLCHAIN_BUILDROOT_CXX
+ depends on !BR2_TOOLCHAIN_BUILDROOT_GLIBC && !BR2_TOOLCHAIN_BUILDROOT_EGLIBC
diff --git a/package/libsigrok/libsigrok.mk b/package/libsigrok/libsigrok.mk
index 6bc210e..1e6080d 100644
--- a/package/libsigrok/libsigrok.mk
+++ b/package/libsigrok/libsigrok.mk
@@ -13,7 +13,7 @@ LIBSIGROK_LICENSE_FILES = COPYING
LIBSIGROK_AUTORECONF = YES
LIBSIGROK_INSTALL_STAGING = YES
LIBSIGROK_DEPENDENCIES = libglib2 libzip host-pkgconf
-LIBSIGROK_CONF_OPTS = --disable-bindings --disable-glibtest
+LIBSIGROK_CONF_OPTS = --disable-glibtest --disable-java --disable-python
define LIBSIGROK_ADD_MISSING
mkdir -p $(@D)/autostuff
@@ -46,4 +46,11 @@ ifeq ($(BR2_PACKAGE_GLIBMM),y)
LIBSIGROK_DEPENDENCIES += glibmm
endif
+ifeq ($(BR2_PACKAGE_LIBSIGROKCXX),y)
+LIBSIGROK_CONF_OPTS += --enable-cxx
+LIBSIGROK_DEPENDENCIES += host-autoconf-archive glibmm
+else
+LIBSIGROK_CONF_OPTS += --disable-cxx
+endif
+
$(eval $(autotools-package))
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 7/7] pulseview: new package
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
` (5 preceding siblings ...)
2015-02-20 12:29 ` [Buildroot] [PATCH 6/7] libsigrok: enable building C++ bindings Bartosz Golaszewski
@ 2015-02-20 12:29 ` Bartosz Golaszewski
2015-02-20 18:15 ` [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Thomas Petazzoni
7 siblings, 0 replies; 12+ messages in thread
From: Bartosz Golaszewski @ 2015-02-20 12:29 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
package/Config.in | 1 +
package/pulseview/Config.in | 31 +++++++++++++++++++++++++++++++
package/pulseview/pulseview.mk | 24 ++++++++++++++++++++++++
3 files changed, 56 insertions(+)
create mode 100644 package/pulseview/Config.in
create mode 100644 package/pulseview/pulseview.mk
diff --git a/package/Config.in b/package/Config.in
index fe3d3d0..17b4e76 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -360,6 +360,7 @@ endif
source "package/picocom/Config.in"
source "package/pifmrds/Config.in"
source "package/pps-tools/Config.in"
+ source "package/pulseview/Config.in"
source "package/read-edid/Config.in"
source "package/rng-tools/Config.in"
source "package/rpi-userland/Config.in"
diff --git a/package/pulseview/Config.in b/package/pulseview/Config.in
new file mode 100644
index 0000000..ccf3fd7
--- /dev/null
+++ b/package/pulseview/Config.in
@@ -0,0 +1,31 @@
+config BR2_PACKAGE_PULSEVIEW
+ bool "pulseview"
+ select BR2_PACKAGE_LIBSERIALPORT
+ select BR2_PACKAGE_LIBSIGROK
+ select BR2_PACKAGE_LIBSIGROKCXX
+ depends on BR2_PACKAGE_QT5
+ select BR2_PACKAGE_QT5BASE_WIDGETS
+ select BR2_PACKAGE_QT5SVG
+ select BR2_PACKAGE_BOOST
+ select BR2_PACKAGE_BOOST_FILESYSTEM
+ select BR2_PACKAGE_BOOST_SYSTEM
+ select BR2_PACKAGE_BOOST_THREAD
+ # libsigrok->libglib2:
+ depends on BR2_USE_WCHAR
+ depends on BR2_TOOLCHAIN_HAS_THREADS
+ depends on BR2_USE_MMU
+ depends on BR2_TOOLCHAIN_BUILDROOT_CXX
+ # libsigrokcxx
+ depends on BR2_TOOLCHAIN_BUILDROOT_GLIBC || BR2_TOOLCHAIN_BUILDROOT_EGLIBC
+ help
+ PulseView is a Qt based logic analyzer, oscilloscope
+ and MSO GUI for sigrok.
+
+ http://sigrok.org/wiki/PulseView
+
+comment "pulseview needs a toolchain w/ wchar, threads, C++, (e)glibc"
+ depends on BR2_USE_MMU
+ depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
+ depends on !BR2_PACKAGE_QT5
+ depends on !BR2_TOOLCHAIN_BUILDROOT_CXX
+ depends on !BR2_TOOLCHAIN_BUILDROOT_GLIBC && !BR2_TOOLCHAIN_BUILDROOT_EGLIBC
diff --git a/package/pulseview/pulseview.mk b/package/pulseview/pulseview.mk
new file mode 100644
index 0000000..923459d
--- /dev/null
+++ b/package/pulseview/pulseview.mk
@@ -0,0 +1,24 @@
+################################################################################
+#
+# pulseview
+#
+################################################################################
+
+# TODO Pulseview can be built and linked against Qt4 as well.
+
+# No https access on upstream git
+PULSEVIEW_SITE = git://sigrok.org/pulseview
+PULSEVIEW_VERSION = 19be0af16af83ca10f7ce69cb64f0b0c6f6a0d81
+PULSEVIEW_LICENSE = GPLv3+
+PULSEVIEW_LICENSE_FILES = COPYING
+PULSEVIEW_DEPENDENCIES = libsigrok qt5base qt5svg boost
+PULSEVIEW_CONF_OPTS = -DDISABLE_WERROR=y
+
+ifeq ($(BR2_PACKAGE_LIBSIGROKDECODE),y)
+PULSEVIEW_CONF_OPTS += -DENABLE_DECODE=TRUE
+PULSEVIEW_DEPENDENCIES += libsigrokdecode
+else
+PULSEVIEW_CONF_OPTS += -DENABLE_DECODE=FALSE
+endif
+
+$(eval $(cmake-package))
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
` (6 preceding siblings ...)
2015-02-20 12:29 ` [Buildroot] [PATCH 7/7] pulseview: new package Bartosz Golaszewski
@ 2015-02-20 18:15 ` Thomas Petazzoni
7 siblings, 0 replies; 12+ messages in thread
From: Thomas Petazzoni @ 2015-02-20 18:15 UTC (permalink / raw)
To: buildroot
Dear Bartosz Golaszewski,
On Fri, 20 Feb 2015 13:28:55 +0100, Bartosz Golaszewski wrote:
> Bartosz Golaszewski (7):
> libsigrok: remove --disable-udev from CONF_OPTS
> libserialport: drop untrue statement from Config.in help
> libsigrok: don't select libserialport in Config.in
> autoconf-archive: new package
> libsigrok: bump git commit tag
All these patches applied to next. On the autoconf-archive, I did a few
changes, I'll indicate which ones as a comment to the patch itself.
> libsigrok: enable building C++ bindings
> pulseview: new package
For those ones, I have some issues/concerns.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 4/7] autoconf-archive: new package
2015-02-20 12:28 ` [Buildroot] [PATCH 4/7] autoconf-archive: new package Bartosz Golaszewski
@ 2015-02-20 18:28 ` Thomas Petazzoni
0 siblings, 0 replies; 12+ messages in thread
From: Thomas Petazzoni @ 2015-02-20 18:28 UTC (permalink / raw)
To: buildroot
Dear Bartosz Golaszewski,
On Fri, 20 Feb 2015 13:28:59 +0100, Bartosz Golaszewski wrote:
> +AUTOCONF_ARCHIVE_VERSION = 2015.02.04
> +AUTOCONF_ARCHIVE_SOURCE = autoconf-archive-$(AUTOCONF_ARCHIVE_VERSION).tar.xz
> +AUTOCONF_ARCHIVE_SITE = http://gnu.cardse.net/autoconf-archive/
Since this package is a GNU package, using $(BR2_GNU_MIRROR) was a
better idea here. I've changed it.
> +AUTOCONF_ARCHIVE_LICENSE = GPLv3+
> +AUTOCONF_ARCHIVE_LICENSE_FILES = COPYING
The license is actually GPLv3+ with exceptions, and I've added
COPYING.EXCEPTION to the LICENSE_FILES variable.
Also, could you send a follow-up patch adding a .hash file for this
package?
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 6/7] libsigrok: enable building C++ bindings
2015-02-20 12:29 ` [Buildroot] [PATCH 6/7] libsigrok: enable building C++ bindings Bartosz Golaszewski
@ 2015-02-20 18:30 ` Thomas Petazzoni
2015-02-23 11:23 ` Nicolas Cavallari
0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2015-02-20 18:30 UTC (permalink / raw)
To: buildroot
Dear Bartosz Golaszewski,
On Fri, 20 Feb 2015 13:29:01 +0100, Bartosz Golaszewski wrote:
> +config BR2_PACKAGE_LIBSIGROKCXX
> + bool "build C++ bindings"
> + select BR2_PACKAGE_GLIBMM
> + depends on BR2_PACKAGE_LIBSIGROK
> + depends on BR2_TOOLCHAIN_BUILDROOT_CXX
> + # C++11 support
> + depends on BR2_TOOLCHAIN_BUILDROOT_GLIBC || BR2_TOOLCHAIN_BUILDROOT_EGLIBC
I am not really happy with this, because there are potentially plenty
of external toolchains that support C++11. So I believe we really need
to go ahead and add a:
BR2_TOOLCHAIN_HAS_CXX11
And make sure it gets properly selected depending on whether the
toolchain has C++11 support or not. You can look at how
BR2_TOOLCHAIN_HAS_SSP or BR2_TOOLCHAIN_HAS_THREADS are implemented for
example.
Moreover, you make this apparently depend on the selected C library.
But isn't C++11 support purely a gcc problem ?
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 6/7] libsigrok: enable building C++ bindings
2015-02-20 18:30 ` Thomas Petazzoni
@ 2015-02-23 11:23 ` Nicolas Cavallari
0 siblings, 0 replies; 12+ messages in thread
From: Nicolas Cavallari @ 2015-02-23 11:23 UTC (permalink / raw)
To: buildroot
On 20/02/2015 19:30, Thomas Petazzoni wrote:
> Moreover, you make this apparently depend on the selected C library.
> But isn't C++11 support purely a gcc problem ?
This software uses std::stod, which suffers from the same problem as
std::to_string with uclibc : GCC currently requires a lot more than
necessary to enable it.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58393
So it's just a matter of updating buildroot's
850-libstdcxx-uclibc-c99.patch. Again.
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2015-02-23 11:23 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-20 12:28 [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 1/7] libsigrok: remove --disable-udev from CONF_OPTS Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 2/7] libserialport: drop untrue statement from Config.in help Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 3/7] libsigrok: don't select libserialport in Config.in Bartosz Golaszewski
2015-02-20 12:28 ` [Buildroot] [PATCH 4/7] autoconf-archive: new package Bartosz Golaszewski
2015-02-20 18:28 ` Thomas Petazzoni
2015-02-20 12:29 ` [Buildroot] [PATCH 5/7] libsigrok: bump git commit tag Bartosz Golaszewski
2015-02-20 12:29 ` [Buildroot] [PATCH 6/7] libsigrok: enable building C++ bindings Bartosz Golaszewski
2015-02-20 18:30 ` Thomas Petazzoni
2015-02-23 11:23 ` Nicolas Cavallari
2015-02-20 12:29 ` [Buildroot] [PATCH 7/7] pulseview: new package Bartosz Golaszewski
2015-02-20 18:15 ` [Buildroot] [PATCH 0/7] sigrok fixes and pulseview package Thomas Petazzoni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.