Cloud App: Music player

[josipl]

This is a ticket related to my GSoC proposition. The plan is to write a fully featured music player which embraces the social components of the web while retaining privacy and security provided by Tahoe-LAFS. This application should be just a glimpse of what can be built on top of Tahoe-LAFS: highly interactive applications providing alternative views to files stored in Tahoe-LAFS.

​See full proposal with UI mock-ups and implementation notes.

[josipl]

Fixed initial scan, which now actually happens.

[josipl]

This file contains two patches, add-music-player and add-music-players-core-deps, where core-deps means that only files needed from dependencies are included (no READMEs, docs, etc.)

[josipl]

(regarding add-music-player-with-core-deps.dpatch)

As mentioned, only files that are actually needed are included in the second patch (add-music-players-core-deps), while irrelevant files are available in a third patch which was too big for upload here (18 MB) but can be downloaded from: ​http://drop.io/aaiodts

[davidsarah]

Review needed for GSoC mid-term evaluations.

[josipl]

Only the music player's code.

[zooko]

Unsetting review-needed. This patch is not ready to be reviewed as if it were ready to commit. However, it would probably be a good help and encouragement to Josip if anyone would look at his code, docs, or comments and give him your thoughts. :-)

[josipl]

Adds player's play/pause/next/prev interface, information about currently playing song, access to additional info about the artist, recommendations and music videos.

[josipl]

Includes new dependencies (notifications library and Last.fm API wrapper) and a fix for MooTools?' JSON parser.

[josipl]

To review the music player's code, following patches have to be applied to Tahoe-LAFS darcs repo:

• attachment:add-music-player.dpatch​
• attachment:updates01082010.dpatch​

If you wish to test the player itself, you'll also need to apply following patches (those containing 3rd party code):

• download 'add-music-players-deps.dpatch' from ​http://drop.io/aaiodts (16MB)
• attachment:new-dependencies.dpatch​

Further instructions on testing and installing the player can be found in 'NOTES' and 'INSTALL' files.

[davidsarah]

Starting review. Everyone, please test!

[francois]

I've just started playing with it and will post comments along the way.

The following command in contrib/musicplayer/INSTALL is not correct.

  $ cp -r build/ ~/.tahoe/public_html/musicplayer

You should add a trailing slash to the destination directory, without it a build directory gets created in ~/.tahoe/public_html/musicplayer/.

  $ cp -r build/ ~/.tahoe/public_html/musicplayer/

[josipl]

Replying to francois:

I've just started playing with it and will post comments along the way.

Cool, thanks!

You should add a trailing slash to the destination directory, without it a build directory gets created in ~/.tahoe/public_html/musicplayer/.

  $ cp -r build/ ~/.tahoe/public_html/musicplayer/

I presume you're using *BSD or OS X? I tested it with GNU version of cp (v8.4) and it works as expected with and without the trailing slash.

[francois]

Replying to josipl:

I presume you're using *BSD or OS X? I tested it with GNU version of cp (v8.4) and it works as expected with and without the trailing slash.

No, I tried that was on Ubuntu 10.04. The command works as expected if directory musicplay does not previsouly exists.

francois@slayer:/tmp$ mkdir test1
francois@slayer:/tmp$ touch test1/file1
francois@slayer:/tmp$ cp -r test1/ blah
francois@slayer:/tmp$ ls blah
file1
francois@slayer:/tmp$ rm -r blah
francois@slayer:/tmp$ mkdir blah
francois@slayer:/tmp$ cp -r test1/ blah
francois@slayer:/tmp$ ls blah
test1
francois@slayer:/tmp$ 

[josipl]

Replying to francois:

Replying to josipl:

I presume you're using *BSD or OS X? I tested it with GNU version of cp (v8.4) and it works as expected with and without the trailing slash.

No, I tried that was on Ubuntu 10.04. The command works as expected if directory musicplay does not previsouly exists.

Indeed, I haven't considered that case, thanks for pointing it out. Have you had any problems with the app itself?

[freestorm]

I have tested on Windows XP SP3 French with trunk branch.
When I tried to compile I got this error:

D:\tahoe-1023\contrib\musicplayer>python manage.py build
running build
error: error in C:\Python26\lib\distutils\distutils.cfg: command 'Build' has no such option 'compiler'

D:\tahoe-1023\contrib\musicplayer>

[josipl]

Adds search and playlist editor, numerous improvements to column rendering and slightly improved UI.

[josipl]

The last patch (attachment:updates14082010.dpatch​) adds some useful features and number of improvements to the old ones, so everyone - please test!

If the app will have problems with detection of your files or if songs/artists/albums will have funky characters in their names, please use attachment:id3_extractor.py​ to extract ID3 tags from your troubled files and attach the result here. (note: you will have to easy_install mutagen)

[freestorm]

Replying to freestorm:

I have tested on Windows XP SP3 French with trunk branch.
When I tried to compile I got this error:

D:\tahoe-1023\contrib\musicplayer>python manage.py build
running build
error: error in C:\Python26\lib\distutils\distutils.cfg: command 'Build' has no such option 'compiler'

D:\tahoe-1023\contrib\musicplayer>

It works with patch attachment:updates14082010.dpatch​

Tested with Firefox

[freestorm]

I've tested musicplayer on CentOS 5.5 with python 2.4.3

When I try to build I have this error:

python manage.py roll
running roll
error: build/resources: No such file or directory

I have this issue because on manage.py line 161 you delete build directory, and after you copy 'src/resources' to 'build/resources'.

It seems that Python on CentOS don't create the build directory before copy subdir resources

So I've added

os.makedirs('build')

just before:

shutil.copytree('src/resources', 'build/resources')

After that it compile with success