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 581018 - Mass Storage only supports USB devices
Mass Storage only supports USB devices
Status: RESOLVED WONTFIX
Product: banshee
Classification: Other
Component: Device - USB Mass Storage
git master
Other All
: Normal enhancement
: ---
Assigned To: Banshee Maintainers
Gabriel Burt
gnome[unmaintained]
: 632819 672005 723572 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-05-01 15:32 UTC by sean dague
Modified: 2020-03-17 08:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
handle non usb mass storage devices (1.21 KB, patch)
2012-03-13 18:41 UTC, Bertrand Lorentz
none Details | Review
Updated patch to handle non-usb mass storage devices (4.36 KB, patch)
2012-03-29 12:34 UTC, Fabrice Bellet
none Details | Review

Description sean dague 2009-05-01 15:32:58 UTC
Please describe the problem:
I have an integrated SD card reader in my laptop (Lenovo t61p).  If I create a .is_audio_player file rhythmbox will correctly enumerate the device as an audio device, but banshee will not.

It appears this is because MS mode for banshee only works with USB devices, and makes all kinds of assumptions in getting device info from that.  It would be really ideal if this worked for integrated readers as well so that I don't need a USB card reader to get banshee to sync on my laptop which has integrated SD reader.

Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 sean dague 2009-05-03 02:37:53 UTC
I've made a first attempt at fixing this in this github tree (a fork of stable) - http://github.com/sdague/banshee/tree/stable

The basic changes are:

* even if a device is not usb, if it contains .is_audio_player pass, let the device be registered
* build the UsbDevice even if it has no usb ids
* a try / catch around the Custom Device lookup, which does that by usb ids

At that point I end up being able to stick my SD card in the integrated reader and have it show up in Banshee and podcast sync worked.  The patch also includes debugging as I was looking for better properties to match on.  It's a bit hackish to just brute force the device in as a "USB Device", but it works surprisingly well, as the USB aspects of the device don't seem to be used much.

The device name in Banshee is "Computer", which comes out of HAL.  It would be nice to get that to be something more reasonable.  Not sure how to tackle that.
Comment 2 Gabriel Burt 2009-05-14 17:49:41 UTC
Hi Sean, thanks for working on this.

Does your device have a vendor/product id, even though it's obviously not USB?  I'm wondering if we could change the current, usb-spedific, vendor/product-ids-are-shorts code into something more generic - maybe just strings for them.

Can you paste or attach the hal-device output for your device (and its parents/children)?
Comment 3 sean dague 2009-05-15 14:00:19 UTC
Can you give me a command I should run, or which output you'd like to get this?  My familiarity with hal internals is low.

Thanks
Comment 4 Gabriel Burt 2009-05-15 14:30:29 UTC
You can just attach the full results of 

hal-device
Comment 5 Gabriel Burt 2009-05-15 14:30:50 UTC
Ideally while you have a sd card inserted and mounted
Comment 6 Gabriel Burt 2009-10-27 20:17:54 UTC
Bulk changing the assignee to banshee-maint@gnome.bugs to make it easier for people to get updated on all banshee bugs by following that address.  It's usually quite apparent who is working on a given bug by the comments and/or patches attached.
Comment 7 Gabriel Burt 2010-11-02 19:47:34 UTC
*** Bug 632819 has been marked as a duplicate of this bug. ***
Comment 8 Sebastian Krämer 2011-01-21 11:40:42 UTC
A solution for this would be great. Syncing via USB works most of the time, but just putting the flash card into the PC and being able to do the same thing would be really nice. Sometimes just copying by hand the media is not enough, for example in cases where certain file names or codecs are unsupported.
I haven't tried the .is_audio_player thing. Would it overwrite the settings that banshee uses in situations where I connect via android phone/USB?
Comment 9 Ryan Armstrong 2011-03-26 11:17:59 UTC
Personally, I would also like this solution to be possible for NFS drives. That way I can use NFS to treat my Eee PC as an "audio device" and automatically load it with music and playlists.
Comment 10 Andrés G. Aragoneses (IRC: knocte) 2011-12-26 10:10:49 UTC
Sean, can you provide the information asked in comment#3?

Also, if you could post a diff with the differences between your fork and current Banshee, that would be great.
Comment 11 Andrés G. Aragoneses (IRC: knocte) 2011-12-26 10:11:12 UTC
I meant, comment#4 :)
Comment 12 sean dague 2011-12-29 20:25:13 UTC
While this is still probably a worth while fix, I stopped using banshee a couple of years back. So it's probably worth closing the bug unless someone else is interested in it.
Comment 13 Andrés G. Aragoneses (IRC: knocte) 2011-12-29 20:33:41 UTC
Well, it's still a valid thing to implement, and you're not the only user that requested it (see comment#9), so I guess the bug can remain open.

That being said, I just realised that there is an extension in the Banshee-Community-Extensions repository that may actually provide this feature, it's called "DirectorySync" if I recall correctly.
Comment 14 Bertrand Lorentz 2012-03-13 18:39:57 UTC
*** Bug 672005 has been marked as a duplicate of this bug. ***
Comment 15 Bertrand Lorentz 2012-03-13 18:41:13 UTC
Created attachment 209637 [details] [review]
handle non usb mass storage devices

Patch from the duplicate, by Fabrice Bellet
Comment 16 Fabrice Bellet 2012-03-29 12:34:06 UTC
Created attachment 210865 [details] [review]
Updated patch to handle non-usb mass storage devices

The DeviceMapper.Map() method should be called, even when no usb device is found.
Comment 17 olivier dufour 2012-08-13 10:11:26 UTC
What do we need to commit this patch?
Timo have ever modify it on his own mac build and I think it is better.
Comment 18 Andrés G. Aragoneses (IRC: knocte) 2014-03-14 09:33:38 UTC
*** Bug 723572 has been marked as a duplicate of this bug. ***
Comment 19 André Klapper 2020-03-17 08:29:00 UTC
Banshee is not under active development anymore and had its last code changes more than three years ago. Its codebase has been archived.

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect
reality. Please feel free to reopen this ticket (or rather transfer the project
to GNOME Gitlab, as GNOME Bugzilla is being shut down) if anyone takes the
responsibility for active development again.
See https://gitlab.gnome.org/Infrastructure/Infrastructure/issues/264 for more info.