All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v4 1/3] ustream-ssl: new package
@ 2016-10-17 11:02 Rahul Jain
  2016-10-17 11:02 ` [Buildroot] [PATCH v4 2/3] uhttpd: " Rahul Jain
                   ` (4 more replies)
  0 siblings, 5 replies; 9+ messages in thread
From: Rahul Jain @ 2016-10-17 11:02 UTC (permalink / raw)
  To: buildroot

ustream-ssl has support for polarssl and cyassl
but since polarssl is made obsolete by mbedtls and cyassl
is not present in buildroot, we have not added support for it.

Signed-off-by: Rahul Bedarkar <Rahul.Bedarkar@imgtec.com>
Signed-off-by: Rahul Jain <Rahul.Jain@imgtec.com>
---
 Changes v3 -> v4:
   - add USTREAM_SSL_LICENSE_FILES variable
 Changes v2 -> v3:
   - Remove entry from DEVELOPERS file
 Changes v1 -> v2:
   - No changes

 package/Config.in                  |  1 +
 package/ustream-ssl/Config.in      | 12 ++++++++++++
 package/ustream-ssl/ustream-ssl.mk | 21 +++++++++++++++++++++
 3 files changed, 34 insertions(+)
 create mode 100644 package/ustream-ssl/Config.in
 create mode 100644 package/ustream-ssl/ustream-ssl.mk

diff --git a/package/Config.in b/package/Config.in
index 9399f63..89de7af 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -911,6 +911,7 @@ menu "Crypto"
 	source "package/polarssl/Config.in"
 	source "package/tinydtls/Config.in"
 	source "package/trousers/Config.in"
+	source "package/ustream-ssl/Config.in"
 endmenu
 
 menu "Database"
diff --git a/package/ustream-ssl/Config.in b/package/ustream-ssl/Config.in
new file mode 100644
index 0000000..540e436
--- /dev/null
+++ b/package/ustream-ssl/Config.in
@@ -0,0 +1,12 @@
+config BR2_PACKAGE_USTREAM_SSL
+	bool "ustream-ssl"
+	depends on !BR2_STATIC_LIBS #libubox
+	select BR2_PACKAGE_LIBUBOX
+	select BR2_PACKAGE_OPENSSL if !BR2_PACKAGE_MBEDTLS
+	help
+	  ustream SSL wrapper
+
+	  https://git.openwrt.org/?p=project/ustream-ssl.git;a=summary
+
+comment "ustream-ssl needs a toolchain w/ dynamic library"
+	depends on BR2_STATIC_LIBS
diff --git a/package/ustream-ssl/ustream-ssl.mk b/package/ustream-ssl/ustream-ssl.mk
new file mode 100644
index 0000000..b83d129
--- /dev/null
+++ b/package/ustream-ssl/ustream-ssl.mk
@@ -0,0 +1,21 @@
+################################################################################
+#
+# ustream-ssl
+#
+################################################################################
+
+USTREAM_SSL_VERSION = ec80adaa1b47f28d426fa19c692011ce60b992d6
+USTREAM_SSL_SITE = git://git.openwrt.org/project/ustream-ssl.git
+USTREAM_SSL_LICENSE = ISC
+USTREAM_SSL_LICENSE_FILES = ustream-ssl.h
+USTREAM_SSL_INSTALL_STAGING = YES
+USTREAM_SSL_DEPENDENCIES = libubox
+
+ifeq ($(BR2_PACKAGE_MBEDTLS),y)
+USTREAM_SSL_DEPENDENCIES += mbedtls
+USTREAM_SSL_CONF_OPTS += -DMBEDTLS=ON
+else
+USTREAM_SSL_DEPENDENCIES += openssl
+endif
+
+$(eval $(cmake-package))
-- 
2.6.2

^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 2/3] uhttpd: new package
  2016-10-17 11:02 [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Rahul Jain
@ 2016-10-17 11:02 ` Rahul Jain
  2016-11-05 13:20   ` Thomas Petazzoni
  2016-10-17 11:02 ` [Buildroot] [PATCH v4 3/3] DEVELOPERS: add entry for uhttpd and ustream-ssl Rahul Jain
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 9+ messages in thread
From: Rahul Jain @ 2016-10-17 11:02 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Rahul Bedarkar <Rahul.Bedarkar@imgtec.com>
Signed-off-by: Neha Bairathi <Neha.Bairathi@imgtec.com>
Signed-off-by: Abhishek Singh <Abhishek.Singh@imgtec.com>
Signed-off-by: Rahul Jain <Rahul.Jain@imgtec.com>
---
 Changes v3 -> v4:
   - add UHTTPD_LICENSE_FILES variable
 Changes v2 -> v3:
   - Remove entry from DEVELOPERS file
 Changes v1 -> v2:
   - Updated the url in help (Suggested by Arnout)
   - Use http url (Suggested by Arnout)
   - Add hash file (Suggested by Arnout)
   - Improve handling of optional dependency on lua (Suggested by Arnout)
   - Remove unrequired config options (Suggested by Arnout)
   - Explicitly specify the config option when enabling lua, ssl and ubus support (Suggested by Arnout)

 package/Config.in          |  1 +
 package/uhttpd/Config.in   | 18 ++++++++++++++++++
 package/uhttpd/uhttpd.hash |  2 ++
 package/uhttpd/uhttpd.mk   | 39 +++++++++++++++++++++++++++++++++++++++
 4 files changed, 60 insertions(+)
 create mode 100644 package/uhttpd/Config.in
 create mode 100644 package/uhttpd/uhttpd.hash
 create mode 100644 package/uhttpd/uhttpd.mk

diff --git a/package/Config.in b/package/Config.in
index 89de7af..84a75c2 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1613,6 +1613,7 @@ endif
 	source "package/tunctl/Config.in"
 	source "package/tvheadend/Config.in"
 	source "package/udpcast/Config.in"
+	source "package/uhttpd/Config.in"
 	source "package/ulogd/Config.in"
 	source "package/ushare/Config.in"
 	source "package/ussp-push/Config.in"
diff --git a/package/uhttpd/Config.in b/package/uhttpd/Config.in
new file mode 100644
index 0000000..e463b93
--- /dev/null
+++ b/package/uhttpd/Config.in
@@ -0,0 +1,18 @@
+config BR2_PACKAGE_UHTTPD
+	bool "uhttpd"
+	depends on !BR2_STATIC_LIBS # dlopen()
+	depends on BR2_TOOLCHAIN_HAS_SYNC_4 # json-c
+	depends on BR2_USE_MMU # fork()
+	select BR2_PACKAGE_LIBUBOX
+	select BR2_PACKAGE_JSON_C
+	help
+	  uHTTPd is a tiny single threaded HTTP server with TLS, CGI and Lua
+	  support. It is intended as a drop-in replacement for the Busybox
+	  HTTP daemon.
+
+	  https://wiki.openwrt.org/doc/howto/http.uhttpd
+
+comment "uhttpd needs a toolchain w/ dynamic library"
+	depends on BR2_TOOLCHAIN_HAS_SYNC_4
+	depends on BR2_USE_MMU
+	depends on BR2_STATIC_LIBS
diff --git a/package/uhttpd/uhttpd.hash b/package/uhttpd/uhttpd.hash
new file mode 100644
index 0000000..59e54ad
--- /dev/null
+++ b/package/uhttpd/uhttpd.hash
@@ -0,0 +1,2 @@
+# No hash for this git snapshot
+none	xxx	uhttpd-59e0c739634f46a164d939e54416287b91ff8a9b.tar.gz
diff --git a/package/uhttpd/uhttpd.mk b/package/uhttpd/uhttpd.mk
new file mode 100644
index 0000000..328b621
--- /dev/null
+++ b/package/uhttpd/uhttpd.mk
@@ -0,0 +1,39 @@
+################################################################################
+#
+# uhttpd
+#
+################################################################################
+
+UHTTPD_VERSION = 59e0c739634f46a164d939e54416287b91ff8a9b
+UHTTPD_SITE = http://git.openwrt.org/project/uhttpd.git
+UHTTPD_SITE_METHOD = git
+UHTTPD_LICENSE = ISC
+UHTTPD_LICENSE_FILES = uhttpd.h
+UHTTPD_DEPENDENCIES = libubox json-c
+
+ifeq ($(BR2_PACKAGE_LUAINTERPRETER_ABI_VERSION),"5.1")
+ifeq ($(BR2_PACKAGE_LUA),y)
+UHTTPD_DEPENDENCIES += lua
+else ifeq ($(BR2_PACKAGE_LUAJIT),y)
+UHTTPD_DEPENDENCIES += luajit
+endif
+UHTTPD_CONF_OPTS += -DLUA_SUPPORT=ON
+else
+UHTTPD_CONF_OPTS += -DLUA_SUPPORT=OFF
+endif
+
+ifeq ($(BR2_PACKAGE_USTREAM_SSL),y)
+UHTTPD_DEPENDENCIES += ustream-ssl
+UHTTPD_CONF_OPTS += -DTLS_SUPPORT=ON
+else
+UHTTPD_CONF_OPTS += -DTLS_SUPPORT=OFF
+endif
+
+ifeq ($(BR2_PACKAGE_UBUS),y)
+UHTTPD_DEPENDENCIES += ubus
+UHTTPD_CONF_OPTS += -DUBUS_SUPPORT=ON
+else
+UHTTPD_CONF_OPTS += -DUBUS_SUPPORT=OFF
+endif
+
+$(eval $(cmake-package))
-- 
2.6.2

^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 3/3] DEVELOPERS: add entry for uhttpd and ustream-ssl
  2016-10-17 11:02 [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Rahul Jain
  2016-10-17 11:02 ` [Buildroot] [PATCH v4 2/3] uhttpd: " Rahul Jain
@ 2016-10-17 11:02 ` Rahul Jain
  2016-11-05 13:20   ` Thomas Petazzoni
  2016-10-17 20:03 ` [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Samuel Martin
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 9+ messages in thread
From: Rahul Jain @ 2016-10-17 11:02 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Rahul Jain <Rahul.Jain@imgtec.com>
---
 Changes v1 -> v2:
   - No changes

 DEVELOPERS | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/DEVELOPERS b/DEVELOPERS
index ed89b09..30fe6fd 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -1212,6 +1212,10 @@ F:	package/gssdp/
 F:	package/gupnp/
 F:	package/gupnp-av/
 
+N:	Rahul Jain <rahul.jain@imgtec.com>
+F:	package/uhttpd
+F:	package/ustream-ssl
+
 N:	Renaud Aubin <root@renaud.io>
 F:	package/libhttpparser/
 
-- 
2.6.2

^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 1/3] ustream-ssl: new package
  2016-10-17 11:02 [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Rahul Jain
  2016-10-17 11:02 ` [Buildroot] [PATCH v4 2/3] uhttpd: " Rahul Jain
  2016-10-17 11:02 ` [Buildroot] [PATCH v4 3/3] DEVELOPERS: add entry for uhttpd and ustream-ssl Rahul Jain
@ 2016-10-17 20:03 ` Samuel Martin
  2016-10-18  7:41   ` Rahul Jain
  2016-11-05 13:19 ` Thomas Petazzoni
  2016-12-04 14:07 ` Thomas Petazzoni
  4 siblings, 1 reply; 9+ messages in thread
From: Samuel Martin @ 2016-10-17 20:03 UTC (permalink / raw)
  To: buildroot

Hi,

On Mon, Oct 17, 2016 at 1:02 PM, Rahul Jain <Rahul.Jain@imgtec.com> wrote:
> ustream-ssl has support for polarssl and cyassl
> but since polarssl is made obsolete by mbedtls and cyassl
> is not present in buildroot, we have not added support for it.
>
> Signed-off-by: Rahul Bedarkar <Rahul.Bedarkar@imgtec.com>
> Signed-off-by: Rahul Jain <Rahul.Jain@imgtec.com>
> ---
>  Changes v3 -> v4:
>    - add USTREAM_SSL_LICENSE_FILES variable
>  Changes v2 -> v3:
>    - Remove entry from DEVELOPERS file
>  Changes v1 -> v2:
>    - No changes
>
>  package/Config.in                  |  1 +
>  package/ustream-ssl/Config.in      | 12 ++++++++++++
>  package/ustream-ssl/ustream-ssl.mk | 21 +++++++++++++++++++++
>  3 files changed, 34 insertions(+)
>  create mode 100644 package/ustream-ssl/Config.in
>  create mode 100644 package/ustream-ssl/ustream-ssl.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 9399f63..89de7af 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -911,6 +911,7 @@ menu "Crypto"
>         source "package/polarssl/Config.in"
>         source "package/tinydtls/Config.in"
>         source "package/trousers/Config.in"
> +       source "package/ustream-ssl/Config.in"
>  endmenu
>
>  menu "Database"
> diff --git a/package/ustream-ssl/Config.in b/package/ustream-ssl/Config.in
> new file mode 100644
> index 0000000..540e436
> --- /dev/null
> +++ b/package/ustream-ssl/Config.in
> @@ -0,0 +1,12 @@
> +config BR2_PACKAGE_USTREAM_SSL
> +       bool "ustream-ssl"
> +       depends on !BR2_STATIC_LIBS #libubox
> +       select BR2_PACKAGE_LIBUBOX
> +       select BR2_PACKAGE_OPENSSL if !BR2_PACKAGE_MBEDTLS
> +       help
> +         ustream SSL wrapper
> +
> +         https://git.openwrt.org/?p=project/ustream-ssl.git;a=summary
> +
> +comment "ustream-ssl needs a toolchain w/ dynamic library"
> +       depends on BR2_STATIC_LIBS
> diff --git a/package/ustream-ssl/ustream-ssl.mk b/package/ustream-ssl/ustream-ssl.mk
> new file mode 100644
> index 0000000..b83d129
> --- /dev/null
> +++ b/package/ustream-ssl/ustream-ssl.mk
> @@ -0,0 +1,21 @@
> +################################################################################
> +#
> +# ustream-ssl
> +#
> +################################################################################
> +
> +USTREAM_SSL_VERSION = ec80adaa1b47f28d426fa19c692011ce60b992d6
> +USTREAM_SSL_SITE = git://git.openwrt.org/project/ustream-ssl.git
> +USTREAM_SSL_LICENSE = ISC
> +USTREAM_SSL_LICENSE_FILES = ustream-ssl.h
> +USTREAM_SSL_INSTALL_STAGING = YES
> +USTREAM_SSL_DEPENDENCIES = libubox
> +
> +ifeq ($(BR2_PACKAGE_MBEDTLS),y)
> +USTREAM_SSL_DEPENDENCIES += mbedtls
> +USTREAM_SSL_CONF_OPTS += -DMBEDTLS=ON
> +else

We usually prefer having the disable option sets here as well, i.e.:
ifeq (...)
USTREAM_SSL_CONF_OPTS += -DMBEDTLS=ON
...
else
USTREAM_SSL_CONF_OPTS += -DMBEDTLS=ON
...
endif

> +USTREAM_SSL_DEPENDENCIES += openssl
> +endif
> +
> +$(eval $(cmake-package))
> --
> 2.6.2
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

Regards,

-- 
Samuel

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 1/3] ustream-ssl: new package
  2016-10-17 20:03 ` [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Samuel Martin
@ 2016-10-18  7:41   ` Rahul Jain
  0 siblings, 0 replies; 9+ messages in thread
From: Rahul Jain @ 2016-10-18  7:41 UTC (permalink / raw)
  To: buildroot

Hi Samuel,

Please find my comment inline.

On Tuesday 18 October 2016 01:33 AM, Samuel Martin wrote:
> Hi,
>
> On Mon, Oct 17, 2016 at 1:02 PM, Rahul Jain <Rahul.Jain@imgtec.com> wrote:
>> ustream-ssl has support for polarssl and cyassl
>> but since polarssl is made obsolete by mbedtls and cyassl
>> is not present in buildroot, we have not added support for it.
>>
>> Signed-off-by: Rahul Bedarkar <Rahul.Bedarkar@imgtec.com>
>> Signed-off-by: Rahul Jain <Rahul.Jain@imgtec.com>
>> ---
>>   Changes v3 -> v4:
>>     - add USTREAM_SSL_LICENSE_FILES variable
>>   Changes v2 -> v3:
>>     - Remove entry from DEVELOPERS file
>>   Changes v1 -> v2:
>>     - No changes
>>
>>   package/Config.in                  |  1 +
>>   package/ustream-ssl/Config.in      | 12 ++++++++++++
>>   package/ustream-ssl/ustream-ssl.mk | 21 +++++++++++++++++++++
>>   3 files changed, 34 insertions(+)
>>   create mode 100644 package/ustream-ssl/Config.in
>>   create mode 100644 package/ustream-ssl/ustream-ssl.mk
>>
>> diff --git a/package/Config.in b/package/Config.in
>> index 9399f63..89de7af 100644
>> --- a/package/Config.in
>> +++ b/package/Config.in
>> @@ -911,6 +911,7 @@ menu "Crypto"
>>          source "package/polarssl/Config.in"
>>          source "package/tinydtls/Config.in"
>>          source "package/trousers/Config.in"
>> +       source "package/ustream-ssl/Config.in"
>>   endmenu
>>
>>   menu "Database"
>> diff --git a/package/ustream-ssl/Config.in b/package/ustream-ssl/Config.in
>> new file mode 100644
>> index 0000000..540e436
>> --- /dev/null
>> +++ b/package/ustream-ssl/Config.in
>> @@ -0,0 +1,12 @@
>> +config BR2_PACKAGE_USTREAM_SSL
>> +       bool "ustream-ssl"
>> +       depends on !BR2_STATIC_LIBS #libubox
>> +       select BR2_PACKAGE_LIBUBOX
>> +       select BR2_PACKAGE_OPENSSL if !BR2_PACKAGE_MBEDTLS
>> +       help
>> +         ustream SSL wrapper
>> +
>> +         https://git.openwrt.org/?p=project/ustream-ssl.git;a=summary
>> +
>> +comment "ustream-ssl needs a toolchain w/ dynamic library"
>> +       depends on BR2_STATIC_LIBS
>> diff --git a/package/ustream-ssl/ustream-ssl.mk b/package/ustream-ssl/ustream-ssl.mk
>> new file mode 100644
>> index 0000000..b83d129
>> --- /dev/null
>> +++ b/package/ustream-ssl/ustream-ssl.mk
>> @@ -0,0 +1,21 @@
>> +################################################################################
>> +#
>> +# ustream-ssl
>> +#
>> +################################################################################
>> +
>> +USTREAM_SSL_VERSION = ec80adaa1b47f28d426fa19c692011ce60b992d6
>> +USTREAM_SSL_SITE = git://git.openwrt.org/project/ustream-ssl.git
>> +USTREAM_SSL_LICENSE = ISC
>> +USTREAM_SSL_LICENSE_FILES = ustream-ssl.h
>> +USTREAM_SSL_INSTALL_STAGING = YES
>> +USTREAM_SSL_DEPENDENCIES = libubox
>> +
>> +ifeq ($(BR2_PACKAGE_MBEDTLS),y)
>> +USTREAM_SSL_DEPENDENCIES += mbedtls
>> +USTREAM_SSL_CONF_OPTS += -DMBEDTLS=ON
>> +else
>
> We usually prefer having the disable option sets here as well, i.e.:
> ifeq (...)
> USTREAM_SSL_CONF_OPTS += -DMBEDTLS=ON
> ...
> else
> USTREAM_SSL_CONF_OPTS += -DMBEDTLS=ON
> ...
> endif
>
It is about choice of cryptographic library and not adding optional 
support for it. I have referred  libcurl.mk, which do not seem to be 
doing what you suggested.

>> +USTREAM_SSL_DEPENDENCIES += openssl
>> +endif
>> +
>> +$(eval $(cmake-package))
>> --
>> 2.6.2
>>
>> _______________________________________________
>> buildroot mailing list
>> buildroot at busybox.net
>> http://lists.busybox.net/mailman/listinfo/buildroot
>
> Regards,
>

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 1/3] ustream-ssl: new package
  2016-10-17 11:02 [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Rahul Jain
                   ` (2 preceding siblings ...)
  2016-10-17 20:03 ` [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Samuel Martin
@ 2016-11-05 13:19 ` Thomas Petazzoni
  2016-12-04 14:07 ` Thomas Petazzoni
  4 siblings, 0 replies; 9+ messages in thread
From: Thomas Petazzoni @ 2016-11-05 13:19 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon, 17 Oct 2016 16:32:23 +0530, Rahul Jain wrote:
> ustream-ssl has support for polarssl and cyassl
> but since polarssl is made obsolete by mbedtls and cyassl
> is not present in buildroot, we have not added support for it.
> 
> Signed-off-by: Rahul Bedarkar <Rahul.Bedarkar@imgtec.com>
> Signed-off-by: Rahul Jain <Rahul.Jain@imgtec.com>
> ---
>  Changes v3 -> v4:
>    - add USTREAM_SSL_LICENSE_FILES variable
>  Changes v2 -> v3:
>    - Remove entry from DEVELOPERS file
>  Changes v1 -> v2:
>    - No changes

Applied to next, thanks.

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

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 2/3] uhttpd: new package
  2016-10-17 11:02 ` [Buildroot] [PATCH v4 2/3] uhttpd: " Rahul Jain
@ 2016-11-05 13:20   ` Thomas Petazzoni
  0 siblings, 0 replies; 9+ messages in thread
From: Thomas Petazzoni @ 2016-11-05 13:20 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon, 17 Oct 2016 16:32:24 +0530, Rahul Jain wrote:

> +ifeq ($(BR2_PACKAGE_LUAINTERPRETER_ABI_VERSION),"5.1")
> +ifeq ($(BR2_PACKAGE_LUA),y)
> +UHTTPD_DEPENDENCIES += lua
> +else ifeq ($(BR2_PACKAGE_LUAJIT),y)
> +UHTTPD_DEPENDENCIES += luajit

It doesn't build with LuaJIT:

[ 35%] Linking C shared module uhttpd_lua.so
/home/thomas/projets/buildroot/output/host/opt/ext-toolchain/bin/../lib/gcc/arm-buildroot-linux-uclibcgnueabi/4.8.5/../../../../arm-buildroot-linux-uclibcgnueabi/bin/ld: cannot find -llua
collect2: error: ld returned 1 exit status

So I've modified  your patch to only use Lua. If you want to re-enable
LuaJIT support, could you fix the issue and send a follow-up patch?

Applied to next with this issue fixed.

Thanks!

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

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 3/3] DEVELOPERS: add entry for uhttpd and ustream-ssl
  2016-10-17 11:02 ` [Buildroot] [PATCH v4 3/3] DEVELOPERS: add entry for uhttpd and ustream-ssl Rahul Jain
@ 2016-11-05 13:20   ` Thomas Petazzoni
  0 siblings, 0 replies; 9+ messages in thread
From: Thomas Petazzoni @ 2016-11-05 13:20 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon, 17 Oct 2016 16:32:25 +0530, Rahul Jain wrote:

> +N:	Rahul Jain <rahul.jain@imgtec.com>
> +F:	package/uhttpd
> +F:	package/ustream-ssl

We want those to have the final slash, so I've added them and applied
to next. Thanks!

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

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH v4 1/3] ustream-ssl: new package
  2016-10-17 11:02 [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Rahul Jain
                   ` (3 preceding siblings ...)
  2016-11-05 13:19 ` Thomas Petazzoni
@ 2016-12-04 14:07 ` Thomas Petazzoni
  4 siblings, 0 replies; 9+ messages in thread
From: Thomas Petazzoni @ 2016-12-04 14:07 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon, 17 Oct 2016 16:32:23 +0530, Rahul Jain wrote:
> ustream-ssl has support for polarssl and cyassl
> but since polarssl is made obsolete by mbedtls and cyassl
> is not present in buildroot, we have not added support for it.
> 
> Signed-off-by: Rahul Bedarkar <Rahul.Bedarkar@imgtec.com>
> Signed-off-by: Rahul Jain <Rahul.Jain@imgtec.com>

This new package is causing a bunch of build failures:

   http://autobuild.buildroot.net/?reason=ustream-ssl-ec80adaa1b47f28d426fa19c692011ce60b992d6

They all look like this:

In file included from /home/test/autobuild/run/instance-0/output/host/usr/mips64el-buildroot-linux-gnu/sysroot/usr/include/mbedtls/net.h:32:0,
                 from /home/test/autobuild/run/instance-0/output/build/ustream-ssl-ec80adaa1b47f28d426fa19c692011ce60b992d6/ustream-mbedtls.h:22,
                 from /home/test/autobuild/run/instance-0/output/build/ustream-ssl-ec80adaa1b47f28d426fa19c692011ce60b992d6/ustream-internal.h:25,
                 from /home/test/autobuild/run/instance-0/output/build/ustream-ssl-ec80adaa1b47f28d426fa19c692011ce60b992d6/ustream-ssl.c:25:
/home/test/autobuild/run/instance-0/output/host/usr/mips64el-buildroot-linux-gnu/sysroot/usr/include/mbedtls/ssl.h:545:5: error: unknown type name 'mbedtls_time_t'
     mbedtls_time_t start;       /*!< starting time      */

Could you have a look and submit a patch to fix this issue?

Thanks!

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

^ permalink raw reply	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2016-12-04 14:07 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-17 11:02 [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Rahul Jain
2016-10-17 11:02 ` [Buildroot] [PATCH v4 2/3] uhttpd: " Rahul Jain
2016-11-05 13:20   ` Thomas Petazzoni
2016-10-17 11:02 ` [Buildroot] [PATCH v4 3/3] DEVELOPERS: add entry for uhttpd and ustream-ssl Rahul Jain
2016-11-05 13:20   ` Thomas Petazzoni
2016-10-17 20:03 ` [Buildroot] [PATCH v4 1/3] ustream-ssl: new package Samuel Martin
2016-10-18  7:41   ` Rahul Jain
2016-11-05 13:19 ` Thomas Petazzoni
2016-12-04 14:07 ` 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.