[PATCH BlueZ] build: Make bt_shell conditional to readline

[PATCH BlueZ] build: Make bt_shell conditional to readline

[Luiz Augusto von Dentz]

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

bt_shell APIs shall only be build if readline is present on the system
since it currently depend on it.
---
 Makefile.am | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/Makefile.am b/Makefile.am
index daf34b6ca..9c3c17139 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -119,7 +119,11 @@ shared_sources = src/shared/io.h src/shared/timeout.h \
 			src/shared/gatt-server.h src/shared/gatt-server.c \
 			src/shared/gatt-db.h src/shared/gatt-db.c \
 			src/shared/gap.h src/shared/gap.c \
-			src/shared/tty.h src/shared/shell.c src/shared/shell.h
+			src/shared/tty.h
+
+if READLINE
+shared_sources += src/shared/shell.c src/shared/shell.h
+endif
 
 src_libshared_glib_la_SOURCES = $(shared_sources) \
 				src/shared/io-glib.c \
-- 
2.14.3

Re: [PATCH BlueZ] build: Make bt_shell conditional to readline

[Thomas Petazzoni]

Hello,

On Fri, 20 Apr 2018 16:02:54 +0300, Luiz Augusto von Dentz wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
> 
> bt_shell APIs shall only be build if readline is present on the system
> since it currently depend on it.

Tested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

I've tested with a few different BlueZ configurations (not all
combinations though), and all of them built fine. The specific
configuration that was failing to build due to the dependency on
readline no longer fails to build.

Thanks a lot Luiz for having looked so quickly at this bug report!

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

Re: [PATCH BlueZ] build: Make bt_shell conditional to readline

[Luiz Augusto von Dentz]

Hi,

On Fri, Apr 20, 2018 at 5:03 PM, Thomas Petazzoni
<thomas.petazzoni@bootlin.com> wrote:
> Hello,
>
> On Fri, 20 Apr 2018 16:02:54 +0300, Luiz Augusto von Dentz wrote:
>> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>>
>> bt_shell APIs shall only be build if readline is present on the system
>> since it currently depend on it.
>
> Tested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
>
> I've tested with a few different BlueZ configurations (not all
> combinations though), and all of them built fine. The specific
> configuration that was failing to build due to the dependency on
> readline no longer fails to build.
>
> Thanks a lot Luiz for having looked so quickly at this bug report!

Applied.

-- 
Luiz Augusto von Dentz