Skip to contentSkip to conversation

zeromq: patch configure.ac to check for ldaddr on when checking libunwind

PATCH New Asaf Kahlon 2 participants 2 messages Apr 15, 2018

lore
Share & tools
lore

Asaf Kahlon

author
#0 lore raw

PATCH 1/1 zeromq: patch configure.ac to check for ldaddr on when checking libunwind

Fixes:
http://autobuild.buildroot.net/results/2b1aaf7e35651f04a9e9d0269e49c1b0bd87ae29/

configure.ac can fail the build when there's no dynamic library support, since it cannot find dladdr which is needed only when we have libunwind. Therefore, configure.ac was patched to check for dladdr only in the libunwind check section. As a consequence of the patch, ZEROMQ_AUTORECONF was set.

The patch was applied to the zeromq master branch, see:
https://github.com/zeromq/libzmq/commit/c971445025535b9f989ab0b78f5ddd1c026f1878

Signed-off-by Asaf Kahlon <[email protected]>
---
 DEVELOPERS                                    |  1 +
 ...-serach-for-dladdr-only-on-libunwind.patch | 44 +++++++++++++++++++
 package/zeromq/zeromq.mk                      |  2 +
 3 files changed, 47 insertions(+)
 create mode 100644 package/zeromq/0001-configure.ac-serach-for-dladdr-only-on-libunwind.patch
DEVELOPERS +1 -0
index 05693d3024..aa70ba5264 100644
@@ -191,6 +191,7 @@ F:	package/stress/
 N:	Asaf Kahlon <[email protected]>
 F:	package/python-jsonmodels/
 F:	package/python-yieldfrom/
+F:	package/zeromq/
 
 N:	Ash Charles <[email protected]>
 F:	package/pru-software-support/
package/zeromq/0001-configure.ac-serach-for-dladdr-only-on-libunwind.patch new file +44 -0

44 lines not on this page · raw message

package/zeromq/zeromq.mk +2 -0
index debaf413b4..2682a567de 100644
@@ -11,6 +11,8 @@ ZEROMQ_DEPENDENCIES = util-linux
 ZEROMQ_CONF_OPTS = --without-documentation
 ZEROMQ_LICENSE = LGPL-3.0+ with exceptions
 ZEROMQ_LICENSE_FILES = COPYING COPYING.LESSER
+# 0001-configure.ac-serach-for-dladdr-only-on-libunwind.patch touches configure.ac
+ZEROMQ_AUTORECONF = YES
 
 # Assume these flags are always available. It is true, at least for
 # SOCK_CLOEXEC, since linux v2.6.27.
Signature · 1 line
2.17.0
Applied
#1 lore raw

PATCH 1/1 zeromq: patch configure.ac to check for ldaddr on when checking libunwind

Hello,

Quoted 11 lines · Asaf Kahlon

Fixes:
http://autobuild.buildroot.net/results/2b1aaf7e35651f04a9e9d0269e49c1b0bd87ae29/

configure.ac can fail the build when there's no dynamic library support, since it cannot find dladdr which is needed only when we have libunwind. Therefore, configure.ac was patched to check for dladdr only in the libunwind check section. As a consequence of the patch, ZEROMQ_AUTORECONF was set.

The patch was applied to the zeromq master branch, see:
https://github.com/zeromq/libzmq/commit/c971445025535b9f989ab0b78f5ddd1c026f1878

Quoted 7 lines
Signed-off-by Asaf Kahlon <[email protected]>
---
 DEVELOPERS                                    |  1 +
 ...-serach-for-dladdr-only-on-libunwind.patch | 44 +++++++++++++++++++
 package/zeromq/zeromq.mk                      |  2 +
 3 files changed, 47 insertions(+)
 create mode 100644 package/zeromq/0001-configure.ac-serach-for-dladdr-only-on-libunwind.patch

Applied to master, thanks.

Thomas

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