Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] shairport-sync: bump to version 3.0
@ 2017-03-10 20:41 Jörg Krause
  2017-03-10 21:22 ` Thomas Petazzoni
  0 siblings, 1 reply; 4+ messages in thread
From: Jörg Krause @ 2017-03-10 20:41 UTC (permalink / raw)
  To: buildroot

Support for PolarSSL is deprecated and the usage of mbedTLS is
recommended, if not OpenSSL is used as crypto backend.

Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
---
 package/shairport-sync/Config.in           | 2 +-
 package/shairport-sync/shairport-sync.hash | 2 +-
 package/shairport-sync/shairport-sync.mk   | 6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/package/shairport-sync/Config.in b/package/shairport-sync/Config.in
index c558a64c5..2213a9663 100644
--- a/package/shairport-sync/Config.in
+++ b/package/shairport-sync/Config.in
@@ -6,7 +6,7 @@ config BR2_PACKAGE_SHAIRPORT_SYNC
 	select BR2_PACKAGE_ALSA_LIB_MIXER
 	select BR2_PACKAGE_LIBCONFIG
 	select BR2_PACKAGE_LIBDAEMON
-	select BR2_PACKAGE_POLARSSL if !BR2_PACKAGE_OPENSSL
+	select BR2_PACKAGE_MBEDTLS if !BR2_PACKAGE_OPENSSL
 	select BR2_PACKAGE_POPT
 	help
 	  Shairport Sync emulates an AirPort Express for the purpose of
diff --git a/package/shairport-sync/shairport-sync.hash b/package/shairport-sync/shairport-sync.hash
index 45bce1016..fb837a4ad 100644
--- a/package/shairport-sync/shairport-sync.hash
+++ b/package/shairport-sync/shairport-sync.hash
@@ -1,2 +1,2 @@
 # Locally calculated
-sha256  2e73416ac8fdda1657034afaa143f1710852ebed06e0aa43dafc2b7dc5eb653d  shairport-sync-2.8.6.tar.gz
+sha256  d02697810da0a2c10d3b39e8c9e0795af1e81fff065db1e27514cdb5da696108  shairport-sync-3.0.tar.gz
diff --git a/package/shairport-sync/shairport-sync.mk b/package/shairport-sync/shairport-sync.mk
index 00d6a695b..ab4fdaee2 100644
--- a/package/shairport-sync/shairport-sync.mk
+++ b/package/shairport-sync/shairport-sync.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-SHAIRPORT_SYNC_VERSION = 2.8.6
+SHAIRPORT_SYNC_VERSION = 3.0
 SHAIRPORT_SYNC_SITE = $(call github,mikebrady,shairport-sync,$(SHAIRPORT_SYNC_VERSION))
 
 SHAIRPORT_SYNC_LICENSE = MIT, BSD-3c
@@ -36,8 +36,8 @@ ifeq ($(BR2_PACKAGE_OPENSSL),y)
 SHAIRPORT_SYNC_DEPENDENCIES += openssl
 SHAIRPORT_SYNC_CONF_OPTS += --with-ssl=openssl
 else
-SHAIRPORT_SYNC_DEPENDENCIES += polarssl
-SHAIRPORT_SYNC_CONF_OPTS += --with-ssl=polarssl
+SHAIRPORT_SYNC_DEPENDENCIES += mbedtls
+SHAIRPORT_SYNC_CONF_OPTS += --with-ssl=mbedtls
 endif
 
 ifeq ($(BR2_PACKAGE_SHAIRPORT_SYNC_LIBSOXR),y)
-- 
2.12.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] shairport-sync: bump to version 3.0
  2017-03-10 20:41 [Buildroot] [PATCH] shairport-sync: bump to version 3.0 Jörg Krause
@ 2017-03-10 21:22 ` Thomas Petazzoni
  2017-03-10 21:54   ` Thomas Petazzoni
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2017-03-10 21:22 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri, 10 Mar 2017 21:41:05 +0100, J?rg Krause wrote:
> Support for PolarSSL is deprecated and the usage of mbedTLS is
> recommended, if not OpenSSL is used as crypto backend.
> 
> Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
> ---
>  package/shairport-sync/Config.in           | 2 +-
>  package/shairport-sync/shairport-sync.hash | 2 +-
>  package/shairport-sync/shairport-sync.mk   | 6 +++---
>  3 files changed, 5 insertions(+), 5 deletions(-)

Applied to master, thanks.

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] shairport-sync: bump to version 3.0
  2017-03-10 21:22 ` Thomas Petazzoni
@ 2017-03-10 21:54   ` Thomas Petazzoni
  2017-03-10 23:08     ` Arnout Vandecappelle
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2017-03-10 21:54 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri, 10 Mar 2017 22:22:45 +0100, Thomas Petazzoni wrote:

> On Fri, 10 Mar 2017 21:41:05 +0100, J?rg Krause wrote:
> > Support for PolarSSL is deprecated and the usage of mbedTLS is
> > recommended, if not OpenSSL is used as crypto backend.
> > 
> > Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
> > ---
> >  package/shairport-sync/Config.in           | 2 +-
> >  package/shairport-sync/shairport-sync.hash | 2 +-
> >  package/shairport-sync/shairport-sync.mk   | 6 +++---
> >  3 files changed, 5 insertions(+), 5 deletions(-)  
> 
> Applied to master, thanks.

And I've reverted because it causes a Config.in circular dependency:

package/openssl/Config.in:1:error: recursive dependency detected!
package/openssl/Config.in:1:	symbol BR2_PACKAGE_OPENSSL is selected by BR2_PACKAGE_MBEDTLS
package/mbedtls/Config.in:1:	symbol BR2_PACKAGE_MBEDTLS is selected by BR2_PACKAGE_OPENSSL

Try out yourself: no such warning right before your patch at commit
cd8e5b819ff31c96dbc7ad94c28a5ad2eb838631. And at commit
7670dcc43e3b023b9bd613fa464533ecde7aca70, right after your patch is
applied, we have this Config.in warning.

Best regards,

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] shairport-sync: bump to version 3.0
  2017-03-10 21:54   ` Thomas Petazzoni
@ 2017-03-10 23:08     ` Arnout Vandecappelle
  0 siblings, 0 replies; 4+ messages in thread
From: Arnout Vandecappelle @ 2017-03-10 23:08 UTC (permalink / raw)
  To: buildroot



On 10-03-17 22:54, Thomas Petazzoni wrote:
> Hello,
> 
> On Fri, 10 Mar 2017 22:22:45 +0100, Thomas Petazzoni wrote:
> 
>> On Fri, 10 Mar 2017 21:41:05 +0100, J?rg Krause wrote:
>>> Support for PolarSSL is deprecated and the usage of mbedTLS is
>>> recommended, if not OpenSSL is used as crypto backend.
>>>
>>> Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
>>> ---
>>>  package/shairport-sync/Config.in           | 2 +-
>>>  package/shairport-sync/shairport-sync.hash | 2 +-
>>>  package/shairport-sync/shairport-sync.mk   | 6 +++---
>>>  3 files changed, 5 insertions(+), 5 deletions(-)  
>>
>> Applied to master, thanks.
> 
> And I've reverted because it causes a Config.in circular dependency:
> 
> package/openssl/Config.in:1:error: recursive dependency detected!
> package/openssl/Config.in:1:	symbol BR2_PACKAGE_OPENSSL is selected by BR2_PACKAGE_MBEDTLS
> package/mbedtls/Config.in:1:	symbol BR2_PACKAGE_MBEDTLS is selected by BR2_PACKAGE_OPENSSL

 This is due to ustream-ssl:

select BR2_PACKAGE_OPENSSL if !BR2_PACKAGE_MBEDTLS

so either the direction in ustream-ssl has to be inverted, or the one in
shairport-sync.

 Regards,
 Arnout

> 
> Try out yourself: no such warning right before your patch at commit
> cd8e5b819ff31c96dbc7ad94c28a5ad2eb838631. And at commit
> 7670dcc43e3b023b9bd613fa464533ecde7aca70, right after your patch is
> applied, we have this Config.in warning.
> 
> Best regards,
> 
> Thomas
> 

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-03-10 23:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-10 20:41 [Buildroot] [PATCH] shairport-sync: bump to version 3.0 Jörg Krause
2017-03-10 21:22 ` Thomas Petazzoni
2017-03-10 21:54   ` Thomas Petazzoni
2017-03-10 23:08     ` Arnout Vandecappelle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox