After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 638711 - [mp4mux] missing track count tag and cover image when transcoding to aac
[mp4mux] missing track count tag and cover image when transcoding to aac
Status: RESOLVED INCOMPLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
0.10.30
Other Linux
: Normal enhancement
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-01-04 23:45 UTC by Gautier Portet
Modified: 2014-11-26 16:09 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Gautier Portet 2011-01-04 23:45:46 UTC
Now that #622484 is fixed, one soundconverter user found another aac related problem.

When transcoding to aac (now with mp4mux), the track count tag and the cover image are missing. I also found that replaygain tags disappear, but one problem at a time.

I used the following command to convert to aac:
gst-launch-0.10 filesrc location=TheseAreDays.flac ! queue ! decodebin2 ! audioconvert ! faac profile=2 ! mp4mux ! filesink location=TheseAreDays.mp4

and the result:

gautier@quad-damage:~/Downloads$ gst-launch-0.10 filesrc location=TheseAreDays.mp4 ! decodebin2 ! fakesink -t
Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
FOUND TAG      : found by element "qtdemux0".
     audio codec: MPEG-4 AAC audio
   language code: en
FOUND TAG      : found by element "qtdemux0".
           title: These Are Days
          artist: 10,000 Maniacs
           album: MTV Unplugged
            date: 1993-01-01
    track number: 1
           genre: Rock
container format: ISO MP4/M4A
FOUND TAG      : found by element "faad0".
     audio codec: MPEG-4 AAC audio
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstSystemClock
Got EOS from element "pipeline0".
Execution ended after 1027517318 ns.
Setting pipeline to PAUSED ...
Setting pipeline to READY ...
Setting pipeline to NULL ...
Freeing pipeline ...

Original file:

gautier@quad-damage:~/Downloads$ gst-launch-0.10 filesrc location=TheseAreDays.flac ! decodebin2 ! fakesink -t
Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
FOUND TAG      : found by element "flacdec0".
           image: buffer of 29659 bytes, type: image/jpeg, width=(int)301, height=(int)300, image-type=(GstTagImageType)GST_TAG_IMAGE_TYPE_FRONT_COVER
FOUND TAG      : found by element "flacdec0".
     audio codec: FLAC
           title: These Are Days
          artist: 10,000 Maniacs
           album: MTV Unplugged
            date: 1993-01-01
    track number: 1
     track count: 14
           genre: Rock
replaygain reference level: 89.000000
replaygain track gain: -5.520000
replaygain track peak: 0.933594
replaygain album gain: -6.460000
replaygain album peak: 0.950195
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstSystemClock
Got EOS from element "pipeline0".
Execution ended after 567960021 ns.
Setting pipeline to PAUSED ...
Setting pipeline to READY ...
Setting pipeline to NULL ...
Freeing pipeline ...

I can attach an example file if needed.
Comment 1 Mark Nauwelaerts 2011-10-31 15:35:48 UTC
Commit below should fix the missing track count.  However, could not spot something wrong with cover image handling, so might need some sample or logs there (e.g. qtmux:5).  As for the replaygain tags, don't know how "standard" those are or about a spec for putting those into mp4.

commit 57684fee3a83fe5e463ca4a5c56f755f74aac676
Author: Mark Nauwelaerts <mark.nauwelaerts@collabora.co.uk>
Date:   Mon Oct 31 16:18:32 2011 +0100

    qtmux: avoid shortcut evaluation when adding paired mp4 tag
    
    Fixes (part of) #638711.
Comment 2 Thiago Sousa Santos 2014-06-20 02:11:20 UTC
Is this still valid? The samples I have around of flac with cover art seem to transcode with those tags to aac/mp4.

Do you have a sample failing?
Comment 3 Tim-Philipp Müller 2014-11-26 16:09:25 UTC
Not sure what to do this without a sample file that fails, seeing that other files appear to work.

Please re-open if this is still an issue with recent 1.x and you can make a sample file available, thanks!