Playlists do not handle \r or mixed newline formats

My sample links were wrong: instead of m3 they should be m3u.

Thanks for the report, can you attach that playlist to the report for testing with it? thanks.

Thanks, it seems that the file is somewhat broken the second line said "/home/michael/Music/bach-bwv852a-grant.mp3^M" that's why totem didn't catch up the file, i don't think it's a totem bug rather a broken file, thanks.

 If there were a recognized standard for m3u files, you could say that an m3u file was broken. But since there is no official standard, it's hard to say that a file doesn't meet the standard. I think that Totem should be able to read any plain-text list of music files, separated by carriage returns and/or line feeds. The files were created with basic plain-text editing software. I don't think we can open a bug on gedit or Windows Notepad, since they seem to be doing what they're supposed to do. The mp3 files aren't broken, in case you were wondering. See http://hanna.pyxidis.org/tech/m3u.html for more about the adhoc m3u format.

Changing status back to Incomplete, since the m3u file is as valid as any file can be that has no standard to meet.

May you take a look with "vi" to that file? There's an extra ^M on the 6 line remove that character and it should work, can you test that? thanks.

Yes, it does work. But I believe that Totem should work on all m3u files created using Windows Notepad or gedit. It shouldn't be necessary to use vi to edit them. Can Totem ignore that extra ^M?

to be send upstream by someone interested on it, for forwarding instructions please take a look to https://wiki.ubuntu.com/Bugs/Upstream/GNOME ; leaving this as incomplete until it gets forwarded, thanks.

We are closing this bug report because it lacks the information we need to investigate the problem, as described in the previous comments. Please reopen it if you can give us the missing information, and don't hesitate to submit bug reports in the future. To reopen the bug report you can click on the current status, under the Status column, and change the Status back to "New". Thanks again!.

Please reopen this bug!

Many Internet radio stations change servers frequently, and the the only way to get an updated server list is to download an m3u or .pls file. When I want to listen to internet radio, it's a real pain to have to download a playlist file, open it up in a text editor, and change the carriage returns to newlines.

The cause of the problem appears to be in totem-pl-parser/plparse/totem-pl-parser-pls.c, lines 161 to 167:

 /* figure out whether we're a unix pls or dos pls */
 if (strstr(contents,"\x0d") == NULL) {
  split_char = "\n";
 } else {
  split_char = "\x0d\n";
  dos_mode = TRUE;
 }
 lines = g_strsplit (contents, split_char, 0);

This if statement should have a third branch to account for files that use \r as a line separator.

Fred - have you experienced an Internet radio station that uses '\r' as newline separators? Totem supports '\r\n' (DOS) or '\n' (Unix) as line separators.
mdmcgin - the issue you are experiencing is you have mixed newline characters in the playlist file - this is not a valid DOS or Unix text file. Have you experienced a server sending files like this? I do however agree with lack of a spec Totem should be able to handle these cases as ^M is not a valid filename character.

Fred, I noticed bug 307492 has playlist files with '\r' as line separators:
http://smoothjazz.com/streams/smoothjazz_128.pls
http://smoothjazz.com/streams/smoothjazz_64.pls
http://smoothjazz.com/streams/smoothjazz_32.pls

Yes, it looks like bug 307492 describes the same problem as this one.

FYI, the specific stations that I've had problems with were:
http://smoothlounge.com/streams/smoothlounge_128.pls
http://smoothlounge.com/streams/smoothlounge_64.pls

These stations appear to be run by the same folks that run smoothjazz.com.

Fixed upstream

Patch now in SRU in bug 372602

This bug was fixed in the package totem-pl-parser - 2.26.2-0ubuntu1

---------------
totem-pl-parser (2.26.2-0ubuntu1) karmic; urgency=low

  * New upstream release (LP: #372602)
    - Fix video/mp4 files played from YouTube showing "get_video" as the video
      name (LP: #346661)
  * debian/patches/20_handle_playlist_newlines.patch
    - Handle various newline formats in playlists (LP: #181553)

 -- Robert Ancell <email address hidden> Wed, 06 May 2009 12:09:58 +0200