After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 585112 - WebOS Support
WebOS Support
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: Device - USB Mass Storage
git master
Other Linux
: Normal enhancement
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-06-08 05:50 UTC by Jeff Wheeler
Modified: 2009-07-19 12:20 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Adds support for the Palm Pre (12.36 KB, patch)
2009-06-10 07:09 UTC, Jeff Wheeler
needs-work Details | Review
Add support for Palm Pre (12.81 KB, patch)
2009-07-02 22:55 UTC, Jeff Wheeler
needs-work Details | Review
Add support for webOS devices (17.95 KB, patch)
2009-07-10 01:43 UTC, Jeff Wheeler
committed Details | Review

Description Jeff Wheeler 2009-06-08 05:50:42 UTC
The Palm Pre works very similarly to the G1/Android, and I think supporting it, too, should be relatively trivial due to Android's support.

I'm trying to learn about Banshee, so I'll try to create a simple patch.
Comment 1 Jeff Wheeler 2009-06-08 05:54:44 UTC
And I suppose I should have said webOS, not Palm Pre.
Comment 2 Jeff Wheeler 2009-06-10 07:09:20 UTC
Created attachment 136257 [details] [review]
Adds support for the Palm Pre

I've attached my patch, mostly based on the support for the G1.

This is my first contribution to Banshee, so if there's anything else I can do to help this go through, please let me know. :)
Comment 3 Gabriel Burt 2009-06-10 17:25:10 UTC
Hi Jeff, thanks for the patch, looks pretty good.

1) minor whitespace/indentation inconsistency in the .addins.xml changes

2) Are the audio_folders and playback_mime_types 100% complete/accurate, or do they need more research/changes?  It really does have AmazonMP3 support, for instance?

3) Should probably add yourself to the Copyright/Authors list for WebOSDevice.cs
Comment 4 Jeff Wheeler 2009-06-10 18:01:21 UTC
Thanks for the feedback.

1) Oh, I see the trailing whitespace on the first line. Is that the only problem?

2) The audio_folders are set to what I believe is reasonable. The ringtones and AmazonMP3 folders exist on the system by default (and are used by the Amazon app and the phone app, respectively), but I have arbitrarily chosen the Music and Videos directories.

I do need to verify the playback_mime_types; I haven't yet checked these.

3) I'll include this in an updated patch.
Comment 5 Gabriel Burt 2009-07-02 19:56:45 UTC
Re 1) I don't know what I was referring to, so ignore that. :)
Comment 6 Jeff Wheeler 2009-07-02 22:55:38 UTC
Created attachment 137753 [details] [review]
Add support for Palm Pre

I've updated the above patch, per your suggestions.
Comment 7 Jeff Wheeler 2009-07-02 22:59:25 UTC
Note that the supported audio/video formats comes from the file format file that, in JSON, matches mimetypes and file extensions to their respective apps. The file is /usr/palm/command-resource-handlers.json, and this is only accessible via the leaked ROM. I'm not sure whether this matters at all . . .
Comment 8 Jeff Wheeler 2009-07-09 17:07:11 UTC
Is there anything further I need to do before this can be applied?

I'm hoping it gets applied sooner than later, so that conflicts do not arise. :)
Comment 9 Gabriel Burt 2009-07-09 18:44:37 UTC
Why do you have

amazon_base_dir = System.IO.Path.Combine (Source.Volume.MountPoint, audio_folders[1]);

but audio_folders[1] is "Videos/"?  Do you mean to use audio_folders[3]?

Please update the copyright header too, adding yourself.

Also, would you factor out the AmazonMp3GroupSource so it's not duplicated?
Comment 10 Jeff Wheeler 2009-07-10 01:43:27 UTC
Created attachment 138165 [details] [review]
Add support for webOS devices

I hope this takes care of the issues you've mentioned. If the formatting of the copyright is not correct, please correct it to your preferred format. (There is no suggested style in the style guide.)
Comment 11 Gabriel Burt 2009-07-13 17:43:20 UTC
Thanks Jeff, I slightly modified the copyright lines (putting them on their own lines, putting yours as 2009).  Committed!
Comment 12 Jeff Wheeler 2009-07-15 19:47:48 UTC
Wonderful! Thank you very much. :)

I didn't include it because I'm a crappy artist, but if somebody wants to make an icon like the G1 icon, the support for the two phones would be just about equal.