[Asterisk-code-review] bundled-jansson: On OpenSuse Leap libjansson.a was placed i... (asterisk[16])

Joshua C. Colp asteriskteam at digium.com
Mon Feb 4 11:30:25 CST 2019


Joshua C. Colp has submitted this change and it was merged. ( https://gerrit.asterisk.org/10947 )

Change subject: bundled-jansson:  On OpenSuse Leap libjansson.a was placed in lib64
......................................................................

bundled-jansson:  On OpenSuse Leap libjansson.a was placed in lib64

On OpenSuse Leap, libjansson.a is installed in
third-party/jansson/dest/lib64 instead of lib (which is where
the top-level makeopts looks).  This causes a link failure.

* Updated jansson/Makefile to add an explicit --libdir to force
  the installation to third-party/jansson/dest/lib.

ASTERISK-28271
Reported by: David Wilcox

Change-Id: Ibf8af75e5da13562105fcc39ed898c6ef0b5a5f3
---
M third-party/jansson/Makefile
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Joshua C. Colp: Looks good to me, but someone else must approve; Approved for Submit
  Corey Farrell: Looks good to me, approved



diff --git a/third-party/jansson/Makefile b/third-party/jansson/Makefile
index ca57b1a..a81502c 100644
--- a/third-party/jansson/Makefile
+++ b/third-party/jansson/Makefile
@@ -81,7 +81,8 @@
 
 source/config.status: source/.unpacked Makefile.rules .rebuild_needed
 	$(ECHO_PREFIX) Configuring
-	$(CMD_PREFIX) (cd source ; ./configure $(QUIET_CONFIGURE) $(JANSSON_CONFIG_OPTS) --disable-shared --enable-static --prefix=$(JANSSON_DIR)/dest)
+	$(CMD_PREFIX) (cd source ; ./configure $(QUIET_CONFIGURE) $(JANSSON_CONFIG_OPTS) --disable-shared \
+		--enable-static --prefix=$(JANSSON_DIR)/dest --libdir=$(JANSSON_DIR)/dest/lib)
 
 configure: source/config.status
 

-- 
To view, visit https://gerrit.asterisk.org/10947
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibf8af75e5da13562105fcc39ed898c6ef0b5a5f3
Gerrit-Change-Number: 10947
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Friendly Automation (1000185)
Gerrit-Reviewer: Joshua C. Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190204/578813ea/attachment.html>


More information about the asterisk-code-review mailing list