[Buildroot] [PATCH] bluez-utils: Add GATT support

[Buildroot] [PATCH] bluez-utils: Add GATT support

[Alexandre Belloni]

This allows to compile gatttool which can help use Bluetooth Low Energy
devices.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 package/bluez_utils/Config.in      | 4 ++++
 package/bluez_utils/bluez_utils.mk | 7 +++++++
 2 files changed, 11 insertions(+)

diff --git a/package/bluez_utils/Config.in b/package/bluez_utils/Config.in
index e08e9be2db95..ebcd526522ba 100644
--- a/package/bluez_utils/Config.in
+++ b/package/bluez_utils/Config.in
@@ -36,6 +36,10 @@ config BR2_PACKAGE_BLUEZ_UTILS_USB
 	help
 	  USB support
 
+config BR2_PACKAGE_BLUEZ_UTILS_GATT
+	bool "GATT support"
+	help
+	  GATT support
 endif
 
 comment "bluez-utils needs a toolchain w/ wchar, threads"
diff --git a/package/bluez_utils/bluez_utils.mk b/package/bluez_utils/bluez_utils.mk
index 7038638a0748..f6484ae723af 100644
--- a/package/bluez_utils/bluez_utils.mk
+++ b/package/bluez_utils/bluez_utils.mk
@@ -37,6 +37,13 @@ BLUEZ_UTILS_CONF_OPT +=	\
 	--disable-audio
 endif
 
+ifeq ($(BR2_PACKAGE_BLUEZ_UTILS_GATT),y)
+BLUEZ_UTILS_DEPENDENCIES +=	readline
+BLUEZ_UTILS_CONF_OPT +=	--enable-gatt
+else
+BLUEZ_UTILS_CONF_OPT +=	--disable-gatt
+endif
+
 # USB support
 ifeq ($(BR2_PACKAGE_BLUEZ_UTILS_USB),y)
 BLUEZ_UTILS_DEPENDENCIES += libusb
-- 
1.8.3.2

[Buildroot] [PATCH] bluez-utils: Add GATT support

[Thomas Petazzoni]

Dear Alexandre Belloni,

On Thu, 19 Dec 2013 20:28:26 +0100, Alexandre Belloni wrote:

> +config BR2_PACKAGE_BLUEZ_UTILS_GATT
> +	bool "GATT support"

You need to select BR2_PACKAGE_READLINE here.

> +	help
> +	  GATT support

Maybe it would be nice to say a few words about what GATT is here,
since it's not immediately clear what it is.

>  endif
>  
>  comment "bluez-utils needs a toolchain w/ wchar, threads"
> diff --git a/package/bluez_utils/bluez_utils.mk b/package/bluez_utils/bluez_utils.mk
> index 7038638a0748..f6484ae723af 100644
> --- a/package/bluez_utils/bluez_utils.mk
> +++ b/package/bluez_utils/bluez_utils.mk
> @@ -37,6 +37,13 @@ BLUEZ_UTILS_CONF_OPT +=	\
>  	--disable-audio
>  endif
>  
> +ifeq ($(BR2_PACKAGE_BLUEZ_UTILS_GATT),y)
> +BLUEZ_UTILS_DEPENDENCIES +=	readline
> +BLUEZ_UTILS_CONF_OPT +=	--enable-gatt
> +else
> +BLUEZ_UTILS_CONF_OPT +=	--disable-gatt
> +endif

Please use spaces after the += operator, and not tabs.

feq ($(BR2_PACKAGE_BLUEZ_UTILS_GATT),y)
BLUEZ_UTILS_DEPENDENCIES += readline
BLUEZ_UTILS_CONF_OPT += --enable-gatt
else
BLUEZ_UTILS_CONF_OPT += --disable-gatt
endif

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com