Bug#986013: asterisk: codec_amr not built

[Athos Ribeiro]

Package: asterisk
Version: 1:16.16.1~dfsg-2
Followup-For: Bug #986013
User: ubuntu-devel at lists.ubuntu.com
Usertags: origin-ubuntu impish ubuntu-patch
Control: tags -1 patch

Dear Maintainer,

Here is a patch to enable and test the AMR codec. The changes include:

  * Build-Depend on libvo-amrwbenc-dev for AMR codec support (LP: #1845765)
  * d/t/amr: Test if AMR codec support is available

Thanks for considering the patch.

*** /tmp/tmpqgrbz6u6/asterisk_1:16.16.1~dfsg-2ubuntu1.debdiff
diff -Nru asterisk-16.16.1~dfsg/debian/control asterisk-16.16.1~dfsg/debian/control
--- asterisk-16.16.1~dfsg/debian/control	2021-08-06 10:35:20.000000000 -0300
+++ asterisk-16.16.1~dfsg/debian/control	2021-09-29 16:02:29.000000000 -0300
@@ -61,6 +61,7 @@
  libtonezone-dev [linux-any],
  libunbound-dev,
  liburiparser-dev,
+ libvo-amrwbenc-dev,
  libvorbis-dev,
  libvpb-dev [linux-any],
  libxml2-dev,
diff -Nru asterisk-16.16.1~dfsg/debian/tests/amr asterisk-16.16.1~dfsg/debian/tests/amr
--- asterisk-16.16.1~dfsg/debian/tests/amr	1969-12-31 21:00:00.000000000 -0300
+++ asterisk-16.16.1~dfsg/debian/tests/amr	2021-09-29 16:02:29.000000000 -0300
@@ -0,0 +1,14 @@
+#!/bin/sh
+
+set -e
+
+# Veryfy amr support is enabled as per
+# https://www.mail-archive.com/asterisk-dev@lists.digium.com/msg45213.html
+# by verifying if the amr codecs are listed and by checking if amr transcoding
+# is available
+
+asterisk -x 'core show codecs audio' | grep -E '\bamr\b'
+asterisk -x 'core show codecs audio' | grep -E '\bamrwb\b'
+
+asterisk -x 'core show translation' | grep -E '\bamr\b'
+asterisk -x 'core show translation' | grep -E '\bamrwb\b'
diff -Nru asterisk-16.16.1~dfsg/debian/tests/control asterisk-16.16.1~dfsg/debian/tests/control
--- asterisk-16.16.1~dfsg/debian/tests/control	2021-08-06 10:35:20.000000000 -0300
+++ asterisk-16.16.1~dfsg/debian/tests/control	2021-09-29 16:02:29.000000000 -0300
@@ -1,3 +1,7 @@
 Tests: asttestmods
 Restrictions: needs-root, isolation-container
 Depends: asterisk, asterisk-voicemail, asterisk-tests, asterisk-core-sounds-en-gsm, libxml2-utils
+
+Tests: amr
+Restrictions: needs-root
+Depends: asterisk, asterisk-modules, grep


-- System Information:
Debian Release: 11.0
  APT prefers impish
  APT policy: (500, 'impish')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 5.13.0-16-generic (SMP w/16 CPU threads)
Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_OOT_MODULE
Locale: LANG=en_US.UTF-8, LC_CTYPE=pt_BR.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

-- 
Athos Ribeiro