Rockbox

Tasklist

FS#7542 - This patch moves all the playlist creation from theroot dir to the PLAYLISTS directory.

Attached to Project: Rockbox
Opened by Gerritt Gonzales (GRaTT) - Sunday, 05 August 2007, 19:00 GMT
Last edited by Jonathan Gordon (jdgordon) - Wednesday, 20 July 2011, 14:15 GMT
Task Type Patches
Category Playlists
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

This patch moves all the playlist creation etc... from the
root dir to the PLAYLISTS directory.
root.m3u8 and dynamic.m3u8 will be created in /PLAYLISTS.
The playlist catalog already uses /PLAYLISTS as default.
The OF creates this dir by default so it is always there
might as well use it to clean up the root.
GRaTT
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Wednesday, 20 July 2011, 14:15 GMT
Reason for closing:  Out of Date
Comment by Paul Louden (Llorean) - Sunday, 05 August 2007, 22:58 GMT
Seeing as you can copy and paste files, shouldn't the playlist be created in the folder that you created it in, rather than placing it in an arbitrary folder that may not even exist?

And not every player's OF creates the folder. Rockbox needs to, on an application level, act functionally similar on all players.
Comment by Gerritt Gonzales (GRaTT) - Monday, 06 August 2007, 04:32 GMT
The database uses /Playlists/ by default.
But all of my playlists regardless are created in the root.
My preference is to have all my playlists in one location
and since the database default is /playlists and the Sansa
OF creates this I felt this made sense.
GRaTT
Comment by Paul Louden (Llorean) - Monday, 06 August 2007, 04:34 GMT
The Rockbox database doesn't use playlists at all. What are you talking about? Do you mean the "Playlist Catalog" feature?
Comment by Gerritt Gonzales (GRaTT) - Monday, 06 August 2007, 04:36 GMT
Yes I mean the catalog feature.
GRaTT
Comment by Nick Brackley (darksaboteur) - Monday, 06 August 2007, 06:46 GMT
It is a very good idea, as I would also like all my playlists in the one directory. If putting playlists in /PLAYLISTS/ is not within guidelines maybe it could at least be an option which is disabled by default? Also why is there no download available for download this patch?
Darky
Comment by Gerritt Gonzales (GRaTT) - Monday, 06 August 2007, 18:16 GMT
here is the patch.
GRaTT
Comment by Johnathon Mihalop (Soul-Slayer) - Tuesday, 07 August 2007, 02:46 GMT
Why does it only enable for the Sansa?
Comment by Gerritt Gonzales (GRaTT) - Tuesday, 07 August 2007, 13:19 GMT
I know for sure the sansa has the "Playlists"
folder by default. The patch is small enough
that if you want to enable it for your player
just edit the #ifdef SANSA_E200.
GRaTT
Comment by Jonathan Gordon (jdgordon) - Tuesday, 07 August 2007, 13:51 GMT
it only has it if you go into the OF
Comment by Johnathon Mihalop (Soul-Slayer) - Tuesday, 07 August 2007, 14:30 GMT
Making it make the folder when built if it doesn't already exist would be logical.
Comment by x (vmh) - Tuesday, 07 August 2007, 19:14 GMT
- check if folder 'Playlists' already exists
- merge with playlist_catalog
Comment by Gerritt Gonzales (GRaTT) - Wednesday, 08 August 2007, 05:04 GMT
Thanks vmh.
@jdgordon: once rockbox has usb for PP targets then sansa owners will be free.
But I understand what you are saying. "If" this will get into svn then it
must address all targets. From previous forum posts and the first comment to this
patch I did not think this was wanted behavior. However I think this patch
makes the Playlist Catalog and playlists in general more user friendly,
and now thanks to vmh it does address all targets.
GRaTT
Comment by x (vmh) - Wednesday, 08 August 2007, 06:52 GMT
simplify and comment the added function playlist_check_dir()
Comment by Jacob Brooks (jac0b) - Saturday, 02 February 2008, 01:33 GMT
Patch is out of sync with SVN
Comment by x (vmh) - Saturday, 02 February 2008, 18:06 GMT
synchronized to r16201.
I don't know if it still works correctly. I don't have time to test it.
Comment by x (vmh) - Friday, 18 April 2008, 20:05 GMT
- synchronized to r17158
- simplified (using dir_exists()) and renamed playlist_check_dir to check_mkdir
Comment by Stephane Doyon (sdoyon) - Monday, 28 April 2008, 00:58 GMT
Shouldn't it obey the "playlist catalog directory" setting?
Comment by x (vmh) - Monday, 28 April 2008, 19:14 GMT
> Shouldn't it obey the "playlist catalog directory" setting?
You're right.

- use global_settings.playlist_catalog_dir (maybe it should be renamed to 'playlist_dir', but I don't want to make changes in too many files)
- remove some unnecessary codes in playlist_catalog.c
Comment by Jacob Brooks (jac0b) - Wednesday, 07 May 2008, 02:08 GMT
Resync

Comment by Janis Meybohm (jayme) - Sunday, 09 November 2008, 21:34 GMT
Resync to rev. 19053
Comment by Jacob Brooks (jac0b) - Friday, 24 July 2009, 01:04 GMT
resync
Comment by Jacob Brooks (jac0b) - Friday, 24 July 2009, 01:41 GMT
Opps the above patch is incorrect this one should work

Loading...