GNOME Bugzilla – Bug 648458
Banshee doesn't import .pls playlist files
Last modified: 2011-05-01 16:36:04 UTC
I could not import any .pls format playlist files. They imported just fine in Rhythmbox 0.13.3. Distribution: Ubuntu 11.04 Beta 2.
Thanks for taking the time to report this bug. This bug report isn't very useful because it doesn't describe the bug well. If you have time and can still reproduce the bug, please read https://bugzilla.gnome.org/bug-HOWTO.html and add a more useful description to this bug.
Why was this moved to Rhythmbox? If I understand the report correctly, things work fine in Rhythmbox, it's importing .pls in Banshee that causes problems. I'm moving this back to Banshee (sorry if I'm misunderstanding something), however it would still be helpful if you could provide a log, Michael. Please see http://live.gnome.org/Banshee/CommonQuestions/Logs
Created attachment 186547 [details] debug log
This is certainly a Banshee bug. I tried importing two different playlists and nothing appeared to happen.
Thanks for the log -- I don't see anything unusual in there. I exported a playlist from Banshee as .pls, deleted the playlist in Banshee, and re-imported it without problems. What program did you use to create the .pls files? Also, it might be helpful if you could attach one of .pls playlists that Banshee won't let you import. Thanks!
I used an Amarok 2.x beta build, maybe some subversion of 2.4. I've attached a .pls. I don't see anything funny except perhaps spaces in the absolute file paths of the playlist entries.
Created attachment 186695 [details] sample playlist
Created attachment 186700 [details] [review] should allow importing the playlist It looks like the pls playlist exported by Amarok can't be imported into Banshee because the first line is '[Playlist]' and Banshee is expecting the lowercase '[playlist]'. This patch fixes the issue for me.
Applies cleanly, and compiles on git master. I can confirm it fixes the issue. I created a playlist, exported it and removed it. Then I manually capitalized the P to match Amarok's style and reimported the playlist successfully.
Fantastic.
Review of attachment 186700 [details] [review]: Committed int git master and the stable-2.0 branch, thanks ! Small nitpick for the next patches : make sure the lines in the commit message don't exceed 72 characters.
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.