"Add To Playlist" patch

[LinusN]

I think this feature should be implemented like this:

- It should use the directory /Playlists to store the playlists. I don't believe in having the name configurable.

- It should use a synclist to present the catalog.

The only problem with this approach would be that it is harder to sort the catalog with the most recently used lists first.

[dunno]

ah, Linus to the rescue

a alphabetical listing of the catalog is good.

[lenny]

true, but the most recently used lists is real handy, if it isn't *too* hard to put in. I'd rather see the patch get in cvs than not at all though.

[Sweet Spot]

Is there any word or progress on committing this patch to the CVS ?  I haven't updated my build in well over a month, perhaps 2, because I don't want to lose the functionality which it ads. I find it invaluable really. However, I realize that I must be missing out on some other good things/bug fixes/optimizations (coldfire %40 speed increase for instance), and would really like to be up to date as far as those things are concerned.

[dunno]

nope, no further interest so far from safetydan or hardeep. Linus hasn't given it any more thought either.

[paulheu]

Isn't this the 'Party mode' now in CVS..

[dunno]

no it's not the same, playlist catalog is a totally different feature

[Sweet Spot]

That really sucks. I don't want to update my build without that feature. Shit.

[lenny]

I've been running a cvs build from the end of January because I didn't have the time to build a more recent one with this patch. I tried today and I got errors when I tried to compile. Has anyone successfully patched a recent build?

[Assimalyst]

No sadly it doesn't work anymore  :'(

[Road Runner]

Well, I'm not sure If I had used the most updated file that was given on this thread (if i recall correctly all links here are supposed to be dead) , but it's updated to cvs 4.19.06 and working nicely

[Assimalyst]

Well, I'm not sure If I had used the most updated file that was given on this thread (if i recall correctly all links here are supposed to be dead) , but it's updated to cvs 4.19.06 and working nicely

Patch tracker linky: http://www.rockbox.org/tracker/task/5179

[lenny]

the most recent cvs that compiled successfully for me was 20060423 as of today, April 30th

hopefully that'll save someone some experimentation time

[Road Runner]

lenny, the patch does work!

if you have any problems, it's probably in /rockbox/apps/langs/english.lang
- It's a frequantly changed file, and when we use diff -u to create unified patches,  it (the patch utility) doesn't know where to place the new strings.

you just have to edit (english.rej) and add the lines that are marked with a + sign to the end of (english.lang) discarding the plus sign obviously.

[lenny]

Great Road Runner, I learned something new! It worked fine