Skip to contentSkip to conversation

php-amqp: selects openssl

PATCH New Fabrice Fontaine 3 participants 4 messages Jun 4, 2018 – Jul 17, 2018

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 php-amqp: selects openssl

Since version 1.8.0, php-amqp needs a rabbitmq-c with openssl support:
https://github.com/pdezwart/php-amqp/issues/310

SSL support is disabled in rabbitmq-c if BR2_STATIC_LIBS is set however don't add an unneeded !BR2_STATIC_LIBS dependency in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS
(see package/Config.in)

Fixes:
- http://autobuild.buildroot.net/results/b7c89bbbd0ca1df08dd7cbfc90c7b45dcf1fad05

---
Changes v1 -> v2 (after review of Baruch Siach and Thomas Petazzoni):
- Remove unneeded !BR2_STATIC_LIBS in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS (see package/Config.in)

 package/php-amqp/Config.in | 1 +
 1 file changed, 1 insertion(+)
package/php-amqp/Config.in +1 -0
index 0aa2021f5a..dc779f0ee0 100644
@@ -2,6 +2,7 @@ config BR2_PACKAGE_PHP_AMQP
 	bool "php-amqp"
 	depends on BR2_PACKAGE_PHP
 	depends on BR2_TOOLCHAIN_HAS_THREADS # rabbitmq-c
+	select BR2_PACKAGE_OPENSSL # rabbitmq-c: amqp_ssl_socket.h
 	select BR2_PACKAGE_RABBITMQ_C
 	help
 	  Communicate with any AMQP compliant server.
Signature · 1 line
2.14.1
#1 lore raw

PATCH v2 1/1 php-amqp: selects openssl

Hello,

Quoted 8 lines · Fabrice Fontaine

Since version 1.8.0, php-amqp needs a rabbitmq-c with openssl support:
https://github.com/pdezwart/php-amqp/issues/310

SSL support is disabled in rabbitmq-c if BR2_STATIC_LIBS is set however don't add an unneeded !BR2_STATIC_LIBS dependency in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS
(see package/Config.in)

Quoted 8 lines

Fixes:
- http://autobuild.buildroot.net/results/b7c89bbbd0ca1df08dd7cbfc90c7b45dcf1fad05

---
Changes v1 -> v2 (after review of Baruch Siach and Thomas Petazzoni):
- Remove unneeded !BR2_STATIC_LIBS in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS (see package/Config.in)

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
#2 lore raw

PATCH v2 1/1 php-amqp: selects openssl

In reply to Fabrice Fontaine
Quoted 10 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Fabrice" == Fabrice Fontaine <[email protected]> writes:

Since version 1.8.0, php-amqp needs a rabbitmq-c with openssl support:
https://github.com/pdezwart/php-amqp/issues/310

SSL support is disabled in rabbitmq-c if BR2_STATIC_LIBS is set however don't add an unneeded !BR2_STATIC_LIBS dependency in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS
(see package/Config.in)

Quoted 8 lines

Fixes:
- http://autobuild.buildroot.net/results/b7c89bbbd0ca1df08dd7cbfc90c7b45dcf1fad05

---
Changes v1 -> v2 (after review of Baruch Siach and Thomas Petazzoni):
- Remove unneeded !BR2_STATIC_LIBS in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS (see package/Config.in)

Committed to 2018.02.x, thanks.

Signature · 1 line
Bye, Peter Korsgaard
#3 lore raw

PATCH v2 1/1 php-amqp: selects openssl

In reply to Fabrice Fontaine
Quoted 10 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Fabrice" == Fabrice Fontaine <[email protected]> writes:

Since version 1.8.0, php-amqp needs a rabbitmq-c with openssl support:
https://github.com/pdezwart/php-amqp/issues/310

SSL support is disabled in rabbitmq-c if BR2_STATIC_LIBS is set however don't add an unneeded !BR2_STATIC_LIBS dependency in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS
(see package/Config.in)

Quoted 8 lines

Fixes:
- http://autobuild.buildroot.net/results/b7c89bbbd0ca1df08dd7cbfc90c7b45dcf1fad05

---
Changes v1 -> v2 (after review of Baruch Siach and Thomas Petazzoni):
- Remove unneeded !BR2_STATIC_LIBS in Config.in as all PHP External Extensions depends on !BR2_STATIC_LIBS (see package/Config.in)

Committed to 2018.05.x, thanks.

Signature · 1 line
Bye, Peter Korsgaard