AMR audio codec not supported

Bug #1386553 reported by Alfonso Sanchez-Beato
20
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Medium
Pat McGowan
gst-libav1.0 (Ubuntu)
Invalid
Undecided
Unassigned
gst-plugins-ugly1.0 (Ubuntu)
Fix Released
Undecided
Unassigned
ubuntu-touch-meta (Ubuntu)
Fix Released
Undecided
Łukasz Zemczak
ubuntu-touch-meta (Ubuntu RTM)
Fix Released
Undecided
Łukasz Zemczak

Bug Description

AMR audio codec is not supported currently. This is relevant for MMS videos, as AMR is the default for audio.

With gstreamer debug env var GST_DEBUG=*:3 I see:

0:00:39.390824849 23581 0xb3f053b0 WARN uridecodebin gsturidecodebin.c:939:unknown_type_cb:<uridecodebin4> warning: No decoder available for type 'audio/AMR, codec_data=(buffer)0000001164616d726170706c0100800005, rate=(int)8000, channels

$ system-image-cli -i
current build number: 131
device name: krillin
channel: ubuntu-touch/ubuntu-rtm/14.09-proposed
last update: 2014-10-27 15:32:17
version version: 131
version ubuntu: 20141027.1
version device: 20141015-32e0f27
version custom: 1413941794

Revision history for this message
Jim Hodapp (jhodapp) wrote :

Have you tried installing gstreamer1.0-libav to see if that codec is covered by libav? I'd be surprised if there wasn't a codec in that library for it.

Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

Right, installing gstreamer1.0-libav fixes the problem. If that is the solution we want, we have to include it in the images.

Also, I would like to know if we do not plan to use Android audio codecs, and the reasons for using/not using them.

Changed in media-hub:
assignee: nobody → John McAleely (john.mcaleely)
status: New → Confirmed
Changed in media-hub:
assignee: John McAleely (john.mcaleely) → nobody
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Two things are needed that are in progress but htis may not make the release milestone
- enable video attachments via MMS
- repackage the amr decoder plugin for gstreamer

Jim Hodapp (jhodapp)
Changed in media-hub:
assignee: nobody → Iain Lane (laney)
affects: media-hub → gst-libav1.0 (Ubuntu)
Changed in gst-libav1.0 (Ubuntu):
assignee: Iain Lane (laney) → nobody
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

This should be gst-plugins-ugly1.0 instead of gst-libav1.0. The reason is that plugins-ugly can be more easily split into packages that support different codecs, as each codec has its own shared library. On the other hand, libav is a single big shared library.

Changed in gst-plugins-ugly1.0 (Ubuntu):
status: New → Confirmed
Revision history for this message
Iain Lane (laney) wrote :

laney@iota> apt-cache showsrc gst-plugins-ugly1.0 ~
Package: gst-plugins-ugly1.0
Binary: gstreamer1.0-plugins-ugly-doc, gstreamer1.0-plugins-ugly, gstreamer1.0-plugins-ugly-amr, gstreamer1.0-plugins-ugly-dbg

Changed in gst-libav1.0 (Ubuntu):
status: Confirmed → Invalid
Changed in gst-plugins-ugly1.0 (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Iain Lane (laney) wrote :

I think someone needs to backport it to rtm.

Revision history for this message
Iain Lane (laney) wrote :

Actually I don't see ugly in rtm at all. You can probably just copy the Vivid package over.

I'm not sure what policies you guys apply for uploads to the RTM distribution around binary copies, versioning and such, so I'd prefer to leave it to someone else - should be a simple task assuming the package builds.

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

The proposal is to refactor that package to just pull in the support we need, we do not want all of gst-ugly

Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

We need to seed the already existing gstreamer1.0-plugins-ugly-amr in the phone images to get this one closed.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Looking at the gstreamer1.0-plugins-ugly-amr it seems that including it and the additional dependencies in our images would require ~450 KB of disk space as its addition to the seeds results in pulling in libopencore-amrnb0 and libopencore-amrwb0. That's not super much, but considering our recent image size issues, I suppose this needs to be approved by the product team. I also need to see how much free space we have in each of our supported images right now.

Changed in ubuntu-touch-meta (Ubuntu RTM):
assignee: nobody → Łukasz Zemczak (sil2100)
Changed in ubuntu-touch-meta (Ubuntu):
assignee: nobody → Łukasz Zemczak (sil2100)
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

ok to add to the seed

Changed in canonical-devices-system-image:
assignee: nobody → Pat McGowan (pat-mcgowan)
importance: Undecided → Medium
milestone: none → ww40-2015
status: New → Confirmed
Changed in ubuntu-touch-meta (Ubuntu):
status: New → In Progress
Changed in ubuntu-touch-meta (Ubuntu RTM):
status: New → In Progress
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

This bug was fixed in the package ubuntu-touch-meta 1.221vivid11 in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay

---------------

ubuntu-touch-meta (1.221vivid11) vivid; urgency=medium

  * Add gstreamer1.0-plugins-ugly-amr to touch (LP: #1386553)

 -- Łukasz 'sil2100' Zemczak <email address hidden> Thu, 17 Sep 2015 18:05:47 +0200

Changed in ubuntu-touch-meta (Ubuntu RTM):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
status: Confirmed → Fix Committed
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
Changed in ubuntu-touch-meta (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.