Bug 913 - Load playlist with Umlaut doesn't work
: Load playlist with Umlaut doesn't work
Status: RESOLVED WORKSFORME
Product: Logitech Media Server
Classification: Unclassified
Component: Playlists
: 6.0.0
: Other SuSE Linux
: P3 normal (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-03-02 04:56 UTC by Felix Mueller
Modified: 2005-03-10 00:07 UTC (History)
0 users

See Also:
Category: ---


Attachments
Playlist with Umlaut (939 bytes, text/plain)
2005-03-02 15:11 UTC, Felix Mueller
Details
Patch for locales and playlists. (3.84 KB, patch)
2005-03-02 18:19 UTC, Dan Sully
Details | Diff
New playlist with umlauts from revision 2301 (1.20 KB, text/plain)
2005-03-03 11:07 UTC, Felix Mueller
Details
Logs from SlimServer rev 2301 with d_source (1.68 KB, text/plain)
2005-03-03 11:10 UTC, Felix Mueller
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Mueller 2005-03-02 04:56:46 UTC
I have some albums with Umlauts in names and pathes.
Selecting such an album for playing works without problem.
I then save the current playlist (with this album) and
upon loading it again I get the 'Problem: Can't open file for'
error on the player.
Also the database seems to get disturbed. The album in
question gets added multiple times to the 'Browse
album' list.
Maybe the problem already occurs when saving the playlist.
Comment 1 Dan Sully 2005-03-02 14:51:45 UTC
Felix - could you upload the playlist to the bug?

Also - what is your LC_CTYPE environment variable set to?

Thanks.
Comment 2 Felix Mueller 2005-03-02 15:11:44 UTC
Created attachment 299 [details]
Playlist with Umlaut
Comment 3 Felix Mueller 2005-03-02 15:14:27 UTC
LC_CTYPE is not set 
Comment 4 Dan Sully 2005-03-02 18:19:45 UTC
Created attachment 300 [details]
Patch for locales and playlists.

Felix - could you give this patch a go?

Thanks.
Comment 5 Dan Sully 2005-03-02 19:47:11 UTC
I've checked this in as subversion change 2296.
Comment 6 Felix Mueller 2005-03-03 11:07:36 UTC
Created attachment 303 [details]
New playlist with umlauts from revision 2301
Comment 7 Felix Mueller 2005-03-03 11:10:01 UTC
Created attachment 304 [details]
Logs from SlimServer rev 2301 with d_source

I still get the 'Can't open file for' error. Attached is the new playlist and
the output of SlimServer with d_source in a good case, where the file plays and
the bad case, where it doesn't.
Do I need to set LC_CTYPE to make it work?
Comment 8 Dan Sully 2005-03-03 11:13:10 UTC
Well, in the case of not having an LC_CTYPE set, we default to ISO-8859-1.

My change makes both the reading and writing of pls/m3u files happen in the
locale you're in. So if there's no LC_CTYPE, we assume and read in ISO-8859-1.
Comment 9 Dan Sully 2005-03-05 12:11:59 UTC
Also - is this a playlist you are creating by hand, or SS is creating?

Comment 10 Felix Mueller 2005-03-05 17:44:59 UTC
The playlist is SS created.

I have two SuSE installations. A production server and a test server.
The output I sent is from my test server where it still does not work.

But yesterday I installed v6 for SB2 on my production server too and
a quick check shows that it works there.
I checked for LC_CTYPE and it's also not set.

Test server is SuSE 8.2, production server is SuSE 9.0

I will try to setup another testserver with an newer SuSE installation
and see what happens.

Until then I guess you can close this bug.