Skip to contentSkip to conversation

paho-mqtt-c: new package

PATCH New Pranit Tanaji Sirsat 2 participants 4 messages Sep 29, 2016 – Oct 7, 2016

lore
Share & tools
lore

Pranit Tanaji Sirsat

author
#0 lore raw

PATCH v3 1/2 paho-mqtt-c: new package

mqtt client c library

Reviewed-by Abhimanyu Vishwakarma <[email protected]>
Reviewed-by Rahul Bedarkar <[email protected]>
Signed-off-by Pranit Tanaji Sirsat <[email protected]>

---
Chnages v2 -> v3:
- rename the package (suggested by Arnout)
- correct license info (suggested by Arnout)
- add explicit cmake option (suggested by Arnout)

Changes v1 -> v2:
- remove updates to DEVELOPERS file (suggested by Baruch Siach)

---
 package/Config.in                    |  1 +
 package/paho-mqtt-c/Config.in        | 11 +++++++++++
 package/paho-mqtt-c/paho-mqtt-c.hash |  2 ++
 package/paho-mqtt-c/paho-mqtt-c.mk   | 20 ++++++++++++++++++++
 4 files changed, 34 insertions(+)
 create mode 100644 package/paho-mqtt-c/Config.in
 create mode 100644 package/paho-mqtt-c/paho-mqtt-c.hash
 create mode 100644 package/paho-mqtt-c/paho-mqtt-c.mk
package/Config.in +1 -0
index 9ca6c15..a04d6d6 100644
@@ -1239,6 +1239,7 @@ menu "Networking"
 	source "package/openpgm/Config.in"
 	source "package/openzwave/Config.in"
 	source "package/ortp/Config.in"
+	source "package/paho-mqtt-c/Config.in"
 	source "package/qdecoder/Config.in"
 	source "package/qpid-proton/Config.in"
 	source "package/rabbitmq-c/Config.in"
package/paho-mqtt-c/Config.in new file +11 -0
new file mode 100644
index 0000000..3560d8c
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_PAHO_MQTT_C
+	bool "paho-mqtt-c"
+	depends on !BR2_STATIC_LIBS  # dlopen()
+	depends on BR2_TOOLCHAIN_HAS_THREADS
+	help
+	  MQTT client C library
+
+	  https://eclipse.org/paho/clients/c/
+
+comment "paho-mqtt-c needs a toolchain w/ threads and dynamic library support"
+	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS
package/paho-mqtt-c/paho-mqtt-c.hash new file +2 -0
new file mode 100644
index 0000000..8807d78
@@ -0,0 +1,2 @@
+# Locally computed:
+sha256  970b9621efc24251441d386274cd0a90cf6ad52b64031f3c8cb53fdd5526f5f3  paho-mqtt-c-v1.1.0.tar.gz
package/paho-mqtt-c/paho-mqtt-c.mk new file +20 -0
new file mode 100644
index 0000000..9180ead
@@ -0,0 +1,20 @@
+################################################################################
+#
+# paho-mqtt-c
+#
+################################################################################
+
+PAHO_MQTT_C_VERSION = v1.1.0
+PAHO_MQTT_C_SITE = $(call github,eclipse,paho.mqtt.c,$(PAHO_MQTT_C_VERSION))
+PAHO_MQTT_C_LICENSE = EPL-1.0 or BSD-3c
+PAHO_MQTT_C_LICENSE_FILES = epl-v10 edl-v10
+PAHO_MQTT_C_INSTALL_STAGING = YES
+
+ifeq ($(BR2_PACKAGE_OPENSSL),y)
+PAHO_MQTT_C_DEPENDENCIES = openssl
+PAHO_MQTT_C_CONF_OPTS = -DPAHO_WITH_SSL=TRUE
+else
+PAHO_MQTT_C_CONF_OPTS = -DPAHO_WITH_SSL=FALSE
+endif
+
+$(eval $(cmake-package))
Signature · 1 line
2.6.2
author
#1 lore raw

PATCH v3 2/2 DEVELOPERS: list paho-mqtt-c package under my name

Signed-off-by Pranit Tanaji Sirsat <[email protected]>

---
Changes v2 -> v3:
- rename package (suggested by Arnout)
Changes v1 -> v2:
- no change

---
 DEVELOPERS | 3 +++
 1 file changed, 3 insertions(+)
DEVELOPERS +3 -0
index 84e00c8..0989ed4 100644
@@ -1148,6 +1148,9 @@ N:	Pieterjan Camerlynck <[email protected]>
 F:	package/libdvbpsi/
 F:	package/mraa/
 
+N:	Pranit Sirsat <[email protected]>
+F:	package/paho-mqtt-c/
+
 N:	Qais Yousef <[email protected]>
 F:	package/bellagio/
Signature · 1 line
2.6.2
Applied
#2 lore raw

PATCH v3 2/2 DEVELOPERS: list paho-mqtt-c package under my name

Hello,

Quoted 10 lines · Pranit Tanaji Sirsat
Signed-off-by Pranit Tanaji Sirsat <[email protected]>

---
Changes v2 -> v3:
- rename package (suggested by Arnout)
Changes v1 -> v2:
- no change

---
 DEVELOPERS | 3 +++
 1 file changed, 3 insertions(+)

Applied to master, thanks.

Thomas

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

PATCH v3 1/2 paho-mqtt-c: new package

In reply to Pranit Tanaji Sirsat

Hello,

package/paho-mqtt-c/paho-mqtt-c.mk line 18

Quoted 6 lines · Pranit Tanaji Sirsat
package/paho-mqtt-c/paho-mqtt-c.mk new file +6 -0
+ifeq ($(BR2_PACKAGE_OPENSSL),y)
+PAHO_MQTT_C_DEPENDENCIES = openssl
+PAHO_MQTT_C_CONF_OPTS = -DPAHO_WITH_SSL=TRUE
+else
+PAHO_MQTT_C_CONF_OPTS = -DPAHO_WITH_SSL=FALSE
+endif

We normally use += instead of = for assignments inside conditions. So I've changed these to use +=, and applied your patch. Thanks!

Thomas

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