Arch Linux User Repository

PSA: I've started hosting this and all its dependencies as prebuilt packages (x86_86 only) in my repository for those that want to install them using pacman without messing around with building from the AUR.

Thanks for the heads up @Thulinma, updated package coming shortly.

Hey there! Asterisk no longer installs headers by default, but dependent packages expects them to be installed. To re-enable headers, simply add the line:

make DESTDIR="${pkgdir}" install-headers

after the existing line

make DESTDIR="${pkgdir}" install

And that'll do the trick.

@nkukard Have you ever messed with fixing this build so it works with Lua 5.3 instead of 5.1? Asterisk supports it, their configure.ac just detects it in the wrong order. I messed with patching it some but didn't get it quite working. I wondered if you'd tried to go down that road before.

I've not tired that myself, I'd prefer to stick with what Asterisk detects, but as you say its in the wrong order, so I think it should be fine to change it if you want.

Also I don't think that should be an optdepend because it cannot be added or removed by installing Lua after the fact, it has to be build in, which means it has to be there at build time. I think it should be a makedepend, and the Arch configuration should just support it out of the box. This is consistent with the way many many other packages that optionally have Lua support handle are handled. Would you mind if I flip that around like that?

Agreed, feel free to flip that around :)

@nkukard Have you ever messed with fixing this build so it works with Lua 5.3 instead of 5.1? Asterisk supports it, their configure.ac just detects it in the wrong order. I messed with patching it some but didn't get it quite working. I wondered if you'd tried to go down that road before.

Also I don't think that should be an optdepend because it cannot be added or removed by installing Lua after the fact, it has to be build in, which means it has to be there at build time. I think it should be a makedepend, and the Arch configuration should just support it out of the box. This is consistent with the way many many other packages that optionally have Lua support handle are handled. Would you mind if I flip that around like that?

Thanks for the add @nkukard. I've also added you to pjproject as it is the only other required dependency for this that is in the AUR. That way any future absence I make have won't hold up updates to this package ;-)

PSA: I've started hosting this and all its dependencies as prebuilt packages (x86_86 only) in my repository for those that want to install them using pacman without messing around with building from the AUR.

@nkukard Would you consider adding me as a co-maintainer on this. I was the maintainer at some point until a period of extended travel where I wasn't around to bump it got it orphaned. I am the maintainer of pjproject (which this depends on) and I would now like to start hosting prebuild x86_64 packages for asterisk in my package repository. It would be nice to be able to bump this and fix issue without having to keep a fork around and possibly having miss-matches between the version in my repository and the AUR.

@isundil, can you perhaps post the error you're getting?

Hello,

There is a missing dependency reference to community/bcg729

Hi,

Here's a patch:

0001-Fix-most-illegal-instructions-when-shipping-asterisk.patch:

From 0b8eb034c73e858af14db5b9b3dd28da9114bd6d Mon Sep 17 00:00:00 2001
From: Denis 'GNUtoo' Carikli GNUtoo@cyberdimension.org
Date: Tue, 8 Jan 2019 16:00:50 +0100
Subject: [PATCH] Fix most illegal instructions when shipping asterisk.

While this works in AUR, because users are building the PKGBUILD,
distributions wanting to use this PKGBUILD as-is and ship packages
produced with it will result in illegal instructions at runtime.
Without that fix, if you build this packages on a machine with
a processor that supports avx (Advanced Vector Extensions),
libpjsip.so.2 ended up with vpxor instructions that were also
executed on machines with a processor that did not have such
extension, which resulted in an illegal instruction at asterisk
startup if the machine running it didn't have AVX.
Though for some reason, with this patch, building asterisk on a
machine with avx still results in res_pjsip_send_to_voicemail.so
having one vpxor instruction.
Hopefully even with that, asterisk can still start on machines
that do not have AVX.
To fix most illegal instructions:
- Native builds were disabled by using the ChangeLog documentation:
  "Those who need different -march= values, please, go for
   ./configure
   make menuselect.makeopts or make menuselect
  ./menuselect/menuselect --disable BUILD_NATIVE"
- The build configuration was changed to use an external
  pjproject library. The AUR PKGBUILD for pjproject also
  had several fixes to prevent it from causing illegal
  instructions.
On x86_64 I verified that most asterisk ELF files did not
have vpxor instructions anymore with:
  $ pacman -Q -l asterisk | \
                 awk '{print $2}' | xargs file | grep ELF | \
                 sed 's#:.*##' | xargs objdump -D | grep vpxor
Signed-off-by: Denis 'GNUtoo' Carikli GNUtoo@cyberdimension.org
PKGBUILD | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/PKGBUILD b/PKGBUILD
index d2fa84e..037bdcd 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -3,10 +3,11 @@
 # Contributor: Xavier Devlamynck magicrhesus@ouranos.be
 # Contributor: Alessio Biancalana dottorblaster@gmail.com
 # Contributor: Maik Broemme mbroemme@libmpq.org
+# Contributor: Denis 'GNUtoo' Carikli GNUtoo@cyberdimension.org
pkgname=asterisk
 pkgver=16.4.0
-pkgrel=1
+pkgrel=2
 pkgdesc="A complete PBX solution"
 arch=('i686' 'x86_64' 'aarch64' 'armv7h')
 backup=('etc/asterisk/acl.conf'
@@ -120,7 +121,7 @@ backup=('etc/asterisk/acl.conf'
         'etc/asterisk/xmpp.conf')
 url='http://www.asterisk.org'
 license=('GPL')
-depends=('alsa-lib' 'speex' 'popt' 'libvorbis' 'curl' 'libxml2' 'jansson' 'libxslt' 'opus' 'libedit')
+depends=('alsa-lib' 'speex' 'popt' 'libvorbis' 'curl' 'libxml2' 'jansson' 'libxslt' 'opus' 'libedit' 'pjproject')
 makedepends=('sqlite3' 'gsm')
 optdepends=('lua51' 'libsrtp' 'postgresql' 'unixodbc' 'libpri' 'libss7' 'openr2' 'iksemel' 'radiusclient-ng' 'dahdi')
 install=${pkgname}.install
@@ -135,7 +136,9 @@ sha256sums=('d870711299089d0b1f01708534b0e139c4e83d828cbb79dfc90ec59774be83fa'
build() {
   cd ${pkgname}-${pkgver}
-  ./configure --prefix=/usr --sysconfdir=/etc --localstatedir=/var --sbindir=/usr/bin
+  ./configure --prefix=/usr --sysconfdir=/etc --localstatedir=/var --sbindir=/usr/bin --with-pjproject-bundled=no
+  make menuselect.makeopts
+  ./menuselect/menuselect --disable BUILD_NATIVE
   make
 }
-- 
2.21.0