Summary: | net-misc/asterisk-13.23.1 - Replace External libjansson Dependency with Bundled Asterisk Version | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Reuben Farrelly <reuben-gentoo-bugzilla> |
Component: | Current packages | Assignee: | Jaco Kroon <jaco> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | jaco, jstein |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Reuben Farrelly
2019-05-28 11:49:19 UTC
Hi Reuben, We'd prefer to use system-installed packages (Gentoo policy, in part to avoid installation bloat) where possible. Would you mind providing the compilation error so that we can look at that? Kind Regards, Jaco Actually it seems I had my wires crossed on this - I apologise. I was testing out an ebuild of 13.26.0 (by simply modifying the name of 13.23.1 ebuild - the contents of the ebuild itself were the same). Therefore I accept that this bug as logged is invalid in the sense that it does not apply to the current 13.23.1 ebuild in the tree - so it can be closed. However for the record now that I have spent time working this out, in these later releases of Asterisk the bundled jansson is used by default and in this case the third change I mentioned below would be required. So when the in-tree ebuild is updated in the future this will have to be taken care of one way or another - either this bug will have to be fixed, or bundled jansson be explicitly disabled. The 13.26.0 ebuild dies like this: checking sys/io.h usability... yes checking sys/io.h presence... yes checking for sys/io.h... yes checking for x86_64-pc-linux-gnu-pkg-config... /usr/bin/x86_64-pc-linux-gnu-pkg-config checking pkg-config is at least version 0.9.0... yes checking for libedit... yes checking for Testing for libedit unicode support... yes checking for uuid_generate_random in -luuid... yes checking uuid/uuid.h usability... yes checking uuid/uuid.h presence... yes checking for uuid/uuid.h... yes ./configure: line 14317: syntax error near unexpected token `else' ./configure: line 14317: `else' !!! Please attach the following file when seeking support: !!! /var/tmp/portage/net-misc/asterisk-13.26.0/work/asterisk-13.26.0/config.log Yes it really is that obscure. The configure file ends up with this around line 14336: ---------- { $as_echo "$as_me:${as_lineno-$LINENO}: *** use './configure --with-jansson-bundled'" >&5 $as_echo "$as_me: *** use './configure --with-jansson-bundled'" >&6;} exit 1 fi JANSSON_DEFINE_JSON_INT() else PBX_JANSSON=1 fi -------- Which is fixed by adding the jansson third party directory to AT_M4DIR in the ebuild as per my original report. I would also suggest if you don't want to use bundle packages you probably also should't have the third-party/pjsip directory in the AT_M4DIR list or be using that either, and should explicitly disable it too, including in 13.23.1 ;-) Once that is fixed the 13.26.0 release builds through to completion. I've tracked down the AT_M4DIR issue independently. I'm marking my update to asterisk 13.29.1 to close this. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b44a1be8b8e28517ce843e630e30ccc1ddf3fae4 commit b44a1be8b8e28517ce843e630e30ccc1ddf3fae4 Author: Jaco Kroon <jaco@uls.co.za> AuthorDate: 2019-11-10 18:29:19 +0000 Commit: Joonas Niilola <juippis@gentoo.org> CommitDate: 2019-11-30 07:48:56 +0000 net-misc/asterisk: version bump to 13.29.1 + maintainership Converted to GLEP 81 for user+group. Consolidated a few DEPEND issues. Dropped pkg_config phase function. Took maintainership. Fixed a bunch of other issues from pkgcheck (${D} and ${ROOT} not having a / following it directly). Bumped to EAPI=7 Use $ED over $D where applicable. Fix statsd integration. Update depend on virtual/mysql to db/mysql-connector-c Enable NOISY_BUILD as requested (instructed) by slyfox. Fix /usr/share/doc/${PV} being asterisk: owned. Make SSL optional. This commit enables progress on the GLSA bug: https://bugs.gentoo.org/689796 Might close: https://bugs.gentoo.org/594160 (SIGILL, may be GRSEC, or #667498) Package-Manager: Portage-2.3.76, Repoman-2.3.16 Closes: https://bugs.gentoo.org/631464 Closes: https://bugs.gentoo.org/654710 Closes: https://bugs.gentoo.org/656472 Closes: https://bugs.gentoo.org/666004 Closes: https://bugs.gentoo.org/667498 Closes: https://bugs.gentoo.org/670522 Closes: https://bugs.gentoo.org/679804 Closes: https://bugs.gentoo.org/686906 Closes: https://bugs.gentoo.org/692696 Signed-off-by: Jaco Kroon <jaco@uls.co.za> Closes: https://github.com/gentoo/gentoo/pull/13649 Signed-off-by: Joonas Niilola <juippis@gentoo.org> net-misc/asterisk/Manifest | 2 + net-misc/asterisk/asterisk-13.29.1.ebuild | 327 ++++++++++++++++++++++++++++++ net-misc/asterisk/metadata.xml | 21 +- 3 files changed, 342 insertions(+), 8 deletions(-) |