Skip to contentSkip to conversation

libwebsockets: add LWS_ prefix to LWS_WITHOUT_TESTAPPS=ON cmake option

PATCH New Baptiste Moraine 3 participants 3 messages Apr 29, 2015

lore
Share & tools
lore

Baptiste Moraine

author
#0 lore raw

PATCH 1/1 libwebsockets: add LWS_ prefix to LWS_WITHOUT_TESTAPPS=ON cmake option

Currently, libwebsockets.mk is passing -DWITHOUT_TESTAPPS=ON to disable the building of libwebsockets test applications, but this is doing nothing because the option is really named LWS_WITHOUT_TESTAPPS.
This commit fixes that by using the correct option name.

Signed-off-by Baptiste Moraine <[email protected]>
---
 package/libwebsockets/libwebsockets.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
package/libwebsockets/libwebsockets.mk +1 -1
index 9e62f1c..2f83748 100644
@@ -11,7 +11,7 @@ LIBWEBSOCKETS_LICENSE = LGPLv2.1 with exceptions
 LIBWEBSOCKETS_LICENSE_FILES = LICENSE
 LIBWEBSOCKETS_DEPENDENCIES = zlib
 LIBWEBSOCKETS_INSTALL_STAGING = YES
-LIBWEBSOCKETS_CONF_OPTS = -DWITHOUT_TESTAPPS=ON -DLWS_IPV6=ON
+LIBWEBSOCKETS_CONF_OPTS = -DLWS_WITHOUT_TESTAPPS=ON -DLWS_IPV6=ON
 
 ifeq ($(BR2_PACKAGE_OPENSSL),y)
 LIBWEBSOCKETS_DEPENDENCIES += openssl host-openssl
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 libwebsockets: add LWS_ prefix to LWS_WITHOUT_TESTAPPS=ON cmake option

Quoted 6 lines · Baptiste Moraine

Currently, libwebsockets.mk is passing -DWITHOUT_TESTAPPS=ON to disable the building of libwebsockets test applications, but this is doing nothing because the option is really named LWS_WITHOUT_TESTAPPS.
This commit fixes that by using the correct option name.

Signed-off-by Baptiste Moraine <[email protected]>
Reviewed-by Samuel Martin <[email protected]>

Regards,

Signature · 1 line
Samuel
Applied
#2 lore raw

PATCH 1/1 libwebsockets: add LWS_ prefix to LWS_WITHOUT_TESTAPPS=ON cmake option

In reply to Baptiste Moraine

Dear Baptiste Moraine,

Quoted 9 lines · Baptiste Moraine

Currently, libwebsockets.mk is passing -DWITHOUT_TESTAPPS=ON to disable the building of libwebsockets test applications, but this is doing nothing because the option is really named LWS_WITHOUT_TESTAPPS.
This commit fixes that by using the correct option name.

Signed-off-by Baptiste Moraine <[email protected]>
---
 package/libwebsockets/libwebsockets.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

Thomas

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