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 612741 - Include AAC profile in Banshee
Include AAC profile in Banshee
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: GStreamer
git master
Other Linux
: Normal enhancement
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
Depends on: 634015
Blocks:
 
 
Reported: 2010-03-12 21:24 UTC by Jakub 'Livio' Rusinek
Modified: 2013-06-27 07:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
renable aac profile (655 bytes, patch)
2010-08-30 18:16 UTC, David Nielsen
none Details | Review
the shame (656 bytes, patch)
2010-08-30 18:35 UTC, David Nielsen
committed Details | Review
Taglib Exception (4.65 KB, text/plain)
2010-08-30 19:26 UTC, David Nielsen
  Details

Description Jakub 'Livio' Rusinek 2010-03-12 21:24:08 UTC
Currently Banshee is able to play AAC file, but even though media players can play them, Banshee won't propose converting FLACs into AAC, but only MP3.

It's not good.
Comment 1 Jakub 'Livio' Rusinek 2010-03-12 21:30:33 UTC
https://bugzilla.novell.com/show_bug.cgi?id=588048
Comment 2 Gabriel Burt 2010-03-12 21:47:02 UTC
We have an aac profile, but it was removed from the build on purpose:

commit 5794cb70bdaa09fbe6aa55cec2973aba89475c34
Author: Aaron Bockover <abock@gnome.org>
Date:   Sat Aug 4 15:38:23 2007

        * data/audio-profiles/Makefile.am: Do not install the aac/faac profile
        as the plugins it requires are very broken in GStreamer; this profile
        only confuses users

We'd want to make sure those issues have been resolved before re-enabling.
Comment 3 Jakub 'Livio' Rusinek 2010-03-12 21:55:49 UTC
Thank you for letting me know. I'd be thankful for leaving this bug open until it's (whatever it is) resolved and AAC profile comes back as eg. Sound Converter creates good AAC(+) files.
Comment 4 Gabriel Burt 2010-03-12 22:37:02 UTC
What is Sound Convertor?  Feel free to do research on this, see if RB / Sound Juicer etc have added/changed/commented on an AAC profile (I think they might use the gnome-media one?), or figure out what Aaron meant by "the plugins it requires [that] are very broken" and see if they've been improved.
Comment 5 Jakub 'Livio' Rusinek 2010-03-12 23:34:37 UTC
http://soundconverter.berlios.de/

Open source app which's the best at converting any input format into any I tried. And I tried a lot of them.

RPM, I installed from, doesn't contain any profiles, so probably gnome-media profiles are utilised... But let me show you something.

> $ rpm -qa|grep -i gnome|sort
> gnome-icon-theme-2.28.0-2.2.noarch
> gnome-keyring-2.28.0-3.5.i586
> gnome-vfs2-2.24.1-3.4.i586
> libgnome-2.28.0-2.4.i586
> libgnomecanvas-2.26.0-2.4.i586
> libgnomecanvas-lang-2.26.0-2.4.noarch
> libgnomeui-2.24.2-2.3.i586
> libgnomeui-lang-2.24.2-2.3.noarch
> python-gnome-2.28.0-2.3.i586

No gnome-media package and sondconverter still works.

> the plugins it requires [that] are very broken

Maybe he meant gstreamer-plugins-bad (or ugly) are somehow broken, buggy, or sth?
Or there's something wrong with faac/faad? I don't know.
Comment 6 David Nielsen 2010-08-30 17:09:33 UTC
That profile deactivation appears to have been made in 2007, and it doesn't mention any upstream bugs to track. Does anyone know if the situation has improved sufficiently upstream in gstreamer?
Comment 7 Jakub 'Livio' Rusinek 2010-08-30 17:15:33 UTC
As I told you comment before: AAC support in GStreamer seems fine.
Comment 8 David Nielsen 2010-08-30 18:16:14 UTC
Created attachment 169110 [details] [review]
renable aac profile

I think this is sufficient to reenable the aac profile, I sadly have no test content. Jakub can you try this?
Comment 9 David Nielsen 2010-08-30 18:35:01 UTC
Created attachment 169113 [details] [review]
the shame

That was shameful, now let's spell xml correctly.
Comment 10 Jakub 'Livio' Rusinek 2010-08-30 18:38:41 UTC
I'd have to compile Banshee on my own. I'll mark it as my task for tomorrow.
Comment 11 David Nielsen 2010-08-30 19:26:30 UTC
Created attachment 169118 [details]
Taglib Exception

Okay, my N900 supports AAC so I tested this. It works and the files transcode to a playable state. We do get the attached TagLib exception every time. I will file this separately.

Consider this a request to merge the patch attached above.
Comment 12 Jakub 'Livio' Rusinek 2010-08-30 19:34:33 UTC
Can you give us a sample AAC? I'd try it out too.
Comment 13 Gabriel Burt 2010-09-20 19:59:11 UTC
The gnome-media AAC profile doesn't use the outputformat property (it leaves it set to the default of 0).  I wonder if we should follow suit with ours?  Can somebody test the profile with and without the outputformat property, and see if it makes a difference with GST, VLC, and iPod playback?

I don't think we can re-enable this until after 1.8.0 due to the string freeze.
Comment 14 Gabriel Burt 2010-09-20 20:00:43 UTC
I'm guessing Aaron disabled it due to bug #490060 and bug #498667, for the record.
Comment 15 David Nielsen 2010-11-01 16:37:05 UTC
*** Bug 633552 has been marked as a duplicate of this bug. ***
Comment 16 Ryan Armstrong 2010-11-01 21:11:22 UTC
Just to clarify on my bug report (633552) that was marked as duplicate for this bug: Banshee will also transcode AACs to other formats even when the iPod is capable of playing the AACs as is. Obviously that's no good, and I'd say that it is worse than just not being able to encode AACs..
Comment 17 David Nielsen 2010-11-01 22:37:48 UTC
Ryan, it is the same problem. Right now, as is noted above the AAC profile is disabled meaning you cannot encode nor transcode into AAC presently.

I'll look at it when I return from vacation unless someone beats me to it.
Comment 18 Ryan Armstrong 2010-11-01 23:05:51 UTC
Let me try again. I am not talking about encoding or transcoding TO AAC. I am talking about Banshee not understanding that existing AAC files are compatible with the iPod. Thus it transcodes AAC files into MP3s when it does not need to. 

Hopefully that's clear, I don't want to clutter up the bug discussion.
Comment 19 Gabriel Burt 2010-11-02 00:12:51 UTC
Ryan, I re-opened your bug, thanks for being patient.
Comment 20 Gabriel Burt 2010-11-02 00:15:12 UTC
Review of attachment 169113 [details] [review]:

Hey David,

Can you do the testing I mentioned in comment #13?  Also, did you file that taglib# bug?  I'd like to have it fixed before committing this.
Comment 21 David Nielsen 2010-11-04 18:52:30 UTC
I forgot about that but this gave me a change to try the very latest AAC profile in Banshee git.

The report is here
https://bugzilla.gnome.org/show_bug.cgi?id=634015
Comment 22 Alexander Kojevnikov 2012-06-29 17:01:11 UTC
Committed, thanks David!

To test, please use taglib-sharp from git master, or wait a few days for a new release.