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 334674 - Support for Motorola iTunes ROKR
Support for Motorola iTunes ROKR
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Removable Media
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-03-15 18:08 UTC by Lincoln Lordello
Modified: 2006-05-06 15:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
iTunes database from Rokr (2.18 KB, application/x-7z-compressed)
2006-03-15 18:11 UTC, Lincoln Lordello
  Details
lshal from my machine (66.37 KB, text/plain)
2006-03-16 18:07 UTC, Lincoln Lordello
  Details
SLVR phone lshal output (101.60 KB, text/plain)
2006-04-21 00:49 UTC, Joe Barnett
  Details
preliminary patch (3.12 KB, patch)
2006-05-04 20:45 UTC, Joe Barnett
none Details | Review
fix bad dbus usage errors (3.13 KB, patch)
2006-05-05 20:36 UTC, Joe Barnett
none Details | Review

Description Lincoln Lordello 2006-03-15 18:08:55 UTC
I want to send informations about the mobile phone called ROKR, with
Itunes.
I hope that in the future I can sync my phone using Rhythmbox ;-)


Phone: Motorola Itunes ROKR
hal storage.model: Motorola Phone

Once it is sync with iTunes, it creates a iTunes folder, with some files
that I want to atach.
(the songs will be left out ;-))
Comment 1 Lincoln Lordello 2006-03-15 18:11:52 UTC
Created attachment 61311 [details]
iTunes database from Rokr

This is the compressed iTunes folder from the phone TransFlash card, created by iTunes from Windows.
Comment 2 Christophe Fergeau 2006-03-15 19:27:56 UTC
I think libgpod will handle that phone in the future. All rhythmbox needs to do is to detect this phone as an iPod (ie the hal detection needs to take it into account).
Comment 3 Lincoln Lordello 2006-03-15 20:02:48 UTC
I think not, the iTunesDB file from phone is different from the iPod.
I tried to sync my phone with gtkpod, moving the files to correct places into the phone, but didn't works. The iPod has almost the same folder structure then the phone, except by the main folder, that's called iTunes_Control into the phone.
I'm helping YamiPod (with information and tests) to support this phone, it's almost working.
Comment 4 Christophe Fergeau 2006-03-15 20:15:09 UTC
So YamiPod can get support for this phone, but libgpod won't be able to get support for that phone for whatever reason ???
For what it's worth,  here is what was committed to libgpod CVS this week-end...

« * itdb.h, itdb_itunesdb.c, itdb_private.h: added support for
	  mobile phone reversed-endian iTunesDB. »
Comment 5 Lincoln Lordello 2006-03-15 20:26:21 UTC
Correcting something, YamiPod 0.94 finally supports Motorola ROKR Phone
;-)

I want Rhtyhmbox support for ROKR, if I didn't, why I'm submitting this bug?
All my songs are under Rhytmbox database ;-)

Rhythmbox needs YOU!
http://gnomedesktop.org/node/2609
Comment 6 Christophe Fergeau 2006-03-15 20:41:43 UTC
Yep, and my initial comment in this bug report was just describing the amount of work that will have to be done to achieve that goal, ie there are only a few lines to change, and if libgpod CVS is installed, everything should be ok.
Comment 7 Christophe Fergeau 2006-03-16 10:59:18 UTC
Hmm, in hal you don't have anything more specific than Motorola Phone? I guess this string can be found on non-ROKR motorola phones?
Comment 8 Bastien Nocera 2006-03-16 11:03:06 UTC
Using the USB ID to match the phone would be a better idea than its name.
Could you attach the output of lshal to this bug?
Comment 9 Lincoln Lordello 2006-03-16 18:07:32 UTC
Created attachment 61377 [details]
lshal from my machine

This is the lshal from my machine with Rokr connected.
Comment 10 Joe Barnett 2006-04-21 00:49:57 UTC
Created attachment 63998 [details]
SLVR phone lshal output

Motorola SLVR phone also supported.  Attaching lshal w/ SLVR connected to get it's ids.
Comment 11 Joe Barnett 2006-05-04 20:45:47 UTC
Created attachment 64833 [details] [review]
preliminary patch

The attached patch at least lets rb read from the db on a mobile phone if you have libgpod > 0.3.3.  haven't tried syncing...  can someone look over this and comment on it?  i'm sure it has some issues, as it prints out the following messages:

2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
2362: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file dbus-message.c line 793.
This is normally a bug in some application using the D-BUS library.
libhal.c 1203 : Couldn't allocate D-BUS message
Comment 12 Joe Barnett 2006-05-05 20:36:36 UTC
Created attachment 64885 [details] [review]
fix bad dbus usage errors

don't try to get properties if the spider_udi is NULL
Comment 13 James "Doc" Livingston 2006-05-06 05:40:41 UTC
Thanks for working on this. I've committed it to cvs, with a small fix so that it compiles with older versions of libgpod.


2006-05-06  James Livingston  <doclivingston@gmail.com>

	patch by: Joe Barnett  <thejoe@gmail.com>

	* configure.ac: check if we're using a version of libgpod that
	supports the Motorola ROKR

	* sources/rb-ipod-source.c: (rb_ipod_get_itunesdb_path): ask libgpod
	for the db path, if we using the newer version.

	(rb_ipod_volume_has_ipod_db): detect Motorola ROKR phones.
Comment 14 Joe Barnett 2006-05-06 15:25:29 UTC
awesome, thanks for looking at this.  I haven't had a chance to check it out yet, but it looks like it will fail to compile (missing a } before the last #endif around line 707 of rb-ipod-source.c)...