tahoe-lafs
/
trac-2024-07-25
2
0
Issues 1.1k Wiki

Cloud App: Music player #1023

New Issue
Open
opened 2010-04-14 14:18:15 +00:00 by josipl · 20 comments
josipl commented 2010-04-14 14:18:15 +00:00
Owner
Copy Link

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.

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](http://docs.google.com/Doc?docid=0AYv4KObVHf8OZGZrYmhkd2RfMTQ0aGt4OWZ6eGQ&hl=en_GB).
tahoe-lafs added the
contrib
major
task
1.6.1
 labels 2010-04-14 14:18:15 +00:00
tahoe-lafs added this to the 1.8.0 milestone 2010-04-14 14:18:15 +00:00
josipl commented 2010-07-08 22:47:53 +00:00
Author
Owner
Copy Link

Attachment add-music-player.2.dpatch (2913671 bytes) added

Fixed initial scan, which now actually happens.

**Attachment** add-music-player.2.dpatch (2913671 bytes) added Fixed initial scan, which now actually happens.
add-music-player.2.dpatch
2.8 MiB
josipl commented 2010-07-09 22:26:21 +00:00
Author
Owner
Copy Link

Attachment add-music-player-with-core-deps.dpatch (9274320 bytes) added

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.)

**Attachment** add-music-player-with-core-deps.dpatch (9274320 bytes) added 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.)
add-music-player-with-core-deps.dpatch
8.8 MiB
josipl commented 2010-07-09 23:06:50 +00:00
Author
Owner
Copy Link

(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

(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 commented 2010-07-09 23:48:36 +00:00
Author
Owner
Copy Link

Review needed for GSoC mid-term evaluations.

Review needed for GSoC mid-term evaluations.
josipl commented 2010-07-10 19:23:33 +00:00
Author
Owner
Copy Link

Attachment add-music-player.dpatch (235537 bytes) added

Only the music player's code.

**Attachment** add-music-player.dpatch (235537 bytes) added Only the music player's code.
add-music-player.dpatch
230 KiB
zooko commented 2010-07-23 05:42:57 +00:00
Copy Link

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. :-)

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 commented 2010-08-01 00:25:13 +00:00
Author
Owner
Copy Link

Attachment updates01082010.dpatch (411379 bytes) added

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

**Attachment** updates01082010.dpatch (411379 bytes) added Adds player's play/pause/next/prev interface, information about currently playing song, access to additional info about the artist, recommendations and music videos.
updates01082010.dpatch
402 KiB
josipl commented 2010-08-01 00:26:46 +00:00
Author
Owner
Copy Link

Attachment new-dependencies.dpatch (45658 bytes) added

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

**Attachment** new-dependencies.dpatch (45658 bytes) added Includes new dependencies (notifications library and Last.fm API wrapper) and a fix for [MooTools](wiki/MooTools)' JSON parser.
new-dependencies.dpatch
45 KiB
josipl commented 2010-08-09 16:46:38 +00:00
Author
Owner
Copy Link

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

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

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

To review the music player's code, following patches have to be applied to Tahoe-LAFS darcs repo: * [add-music-player.dpatch](/tahoe-lafs/trac-2024-07-25/attachments/000078ac-1388-25ed-8f41-07e39dac97b5) * [updates01082010.dpatch](/tahoe-lafs/trac-2024-07-25/attachments/000078ac-1388-25ed-8f41-9954f6d62ba6) 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) * [new-dependencies.dpatch](/tahoe-lafs/trac-2024-07-25/attachments/000078ac-1388-25ed-8f41-9525647d50e5) Further instructions on testing and installing the player can be found in 'NOTES' and 'INSTALL' files.
davidsarah commented 2010-08-09 18:54:33 +00:00
Author
Owner
Copy Link

Starting review. Everyone, please test!

Starting review. Everyone, please test!
zooko modified the milestone from 1.8.0 to 1.9.0 2010-08-09 22:16:20 +00:00
francois commented 2010-08-10 11:41:13 +00:00
Author
Owner
Copy Link

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/
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 commented 2010-08-10 13:45:15 +00:00
Author
Owner
Copy Link

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.

Replying to [francois](/tahoe-lafs/trac-2024-07-25/issues/1023#issuecomment-76956): > 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 commented 2010-08-10 17:05:10 +00:00
Author
Owner
Copy Link

Replying to [josipl]comment:10:

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$
Replying to [josipl]comment:10: > 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 commented 2010-08-10 17:15:51 +00:00
Author
Owner
Copy Link

Replying to [francois]comment:11:

Replying to [josipl]comment:10:

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?

Replying to [francois]comment:11: > Replying to [josipl]comment:10: > > > 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 commented 2010-08-11 21:19:55 +00:00
Author
Owner
Copy Link

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>
I have tested on Windows XP SP3 French with trunk branch.<br> 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 commented 2010-08-14 18:12:57 +00:00
Author
Owner
Copy Link

Attachment updates14082010.dpatch (272868 bytes) added

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

**Attachment** updates14082010.dpatch (272868 bytes) added Adds search and playlist editor, numerous improvements to column rendering and slightly improved UI.
updates14082010.dpatch
266 KiB
josipl commented 2010-08-14 18:45:09 +00:00
Author
Owner
Copy Link

Attachment id3_extractor.py (329 bytes) added

**Attachment** id3_extractor.py (329 bytes) added
id3_extractor.py
329 B
josipl commented 2010-08-14 18:53:04 +00:00
Author
Owner
Copy Link

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 id3_extractor.py to extract ID3 tags from your troubled files and attach the result here. (note: you will have to easy_install mutagen)

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 [id3_extractor.py](/tahoe-lafs/trac-2024-07-25/attachments/000078ac-1388-25ed-8f41-253a76e8fb55) to extract ID3 tags from your troubled files and attach the result here. (note: you will have to `easy_install mutagen`)
freestorm commented 2010-08-16 23:17:51 +00:00
Author
Owner
Copy Link

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 updates14082010.dpatch

Tested with Firefox

Replying to [freestorm](/tahoe-lafs/trac-2024-07-25/issues/1023#issuecomment-76960): > I have tested on Windows XP SP3 French with trunk branch.<br> > 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 [updates14082010.dpatch](/tahoe-lafs/trac-2024-07-25/attachments/000078ac-1388-25ed-8f41-36f89d983b1f) Tested with Firefox
freestorm commented 2010-08-19 16:13:41 +00:00
Author
Owner
Copy Link

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

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'.<br> It seems that Python on CentOS don't create the build directory before copy subdir resources<br> So I've added ``` os.makedirs('build') ``` just before: ``` shutil.copytree('src/resources', 'build/resources') ``` After that it compile with success
zooko modified the milestone from 1.9.0 to soon 2011-04-24 04:05:36 +00:00
zooko modified the milestone from soon to 1.10.0 2011-04-24 04:07:09 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
0.2.0

   
0.3.0

   
0.4.0

   
0.5.0

   
0.5.1

   
0.6.0

   
0.6.1

   
0.7.0

   
0.8.0

   
0.9.0

   
1.0.0

   
1.1.0

   
1.10.0

   
1.10.1

   
1.10.2

   
1.10a2

   
1.11.0

   
1.12.0

   
1.12.1

   
1.13.0

   
1.14.0

   
1.15.0

   
1.15.1

   
1.2.0

   
1.3.0

   
1.4.1

   
1.5.0

   
1.6.0

   
1.6.1

   
1.7.0

   
1.7.1

   
1.7β

   
1.8.0

   
1.8.1

   
1.8.2

   
1.8.3

   
1.8β

   
1.9.0

   
1.9.0-s3branch

   
1.9.0a1

   
1.9.0a2

   
1.9.0b1

   
1.9.1

   
1.9.2

   
1.9.2a1

   
LeastAuthority.com automation

   
blocker

   
cannot reproduce

   
cloud-branch

   
code

catch-all for anything that can be fixed by writing code but isn't classified better

  
code-dirnodes

anything involving tahoe directories

  
code-encoding

immutable-file encoding format/layout, cap format, encoding/decoding algorithms

  
code-frontend

general frontend things: new user-facing interfaces, FTP/SFTP servers, FUSE-like interfaces

  
code-frontend-cli

CLI tools

  
code-frontend-ftp-sftp

   
code-frontend-magic-folder

   
code-frontend-web

webapi interfaces, user-facing "WUI" interfaces

  
code-mutable

anything involving mutable files: encoding/decoding, share format

  
code-network

Introducer, locating storage servers, establishing/maintaining connections to them

  
code-nodeadmin

node startup/shutdown, tools to create nodes, logging tools, management/monitoring tools, manhole, disk-space-limiting tools

  
code-peerselection

server selection/sorting algorithm, rebalancing code, allocation policy, swarming-download

  
code-storage

storage-server implementation, share-storage directory layout, client-to-server share transfer protocol

  
contrib

problems with non-core things that live in the source:contrib/ directory

  
critical

   
defect

   
dev-infrastructure

things that help us build Tahoe: buildbot, automated speed/memory/code-coverage tests and results-displayers, IRC channels, mailing lists, this Trac instance

  
documentation

docs items. Also use 'docs' keyword on tickets in more specific components.

  
duplicate

   
enhancement

   
fixed

   
invalid

   
major

   
minor

   
n/a

   
normal

   
operational

things involved in running a tahoe grid, external tools to monitor/maintain a grid (like "nodeadmin" but larger-scale). Was also used for issues with the allmydata.com grid.

  
packaging

setup.py script, library dependencies, binary packages, licensing, installation instructions

  
somebody else's problem

   
supercritical

   
task

   
trivial

   
unknown

Uncategorized. All tickets default to this component until someone updates it.

  
was already fixed

   
website

things about the tahoe-lafs.org website (and sometimes this Trac instance, shared with 'dev-infrastructure')

  
wontfix

   
worksforme
No Label
0.2.0
0.3.0
0.4.0
0.5.0
0.5.1
0.6.0
0.6.1
0.7.0
0.8.0
0.9.0
1.0.0
1.1.0
1.10.0
1.10.1
1.10.2
1.10a2
1.11.0
1.12.0
1.12.1
1.13.0
1.14.0
1.15.0
1.15.1
1.2.0
1.3.0
1.4.1
1.5.0
1.6.0
1.6.1
1.7.0
1.7.1
1.7β
1.8.0
1.8.1
1.8.2
1.8.3
1.8β
1.9.0
1.9.0-s3branch
1.9.0a1
1.9.0a2
1.9.0b1
1.9.1
1.9.2
1.9.2a1
LeastAuthority.com automation
blocker
cannot reproduce
cloud-branch
code
code-dirnodes
code-encoding
code-frontend
code-frontend-cli
code-frontend-ftp-sftp
code-frontend-magic-folder
code-frontend-web
code-mutable
code-network
code-nodeadmin
code-peerselection
code-storage
contrib
critical
defect
dev-infrastructure
documentation
duplicate
enhancement
fixed
invalid
major
minor
n/a
normal
operational
packaging
somebody else's problem
supercritical
task
trivial
unknown
was already fixed
website
wontfix
worksforme
Milestone
Clear milestone
No items
No Milestone
soon
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Reference: tahoe-lafs/trac-2024-07-25#1023
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?