Re: Fixing gvfs-backend-afc for iPhone/iPod Touch access

[Bastien Nocera <hadess hadess net>]

On Wed, 2009-08-05 at 13:59 +0200, Martin S. wrote:
> On Wed, 2009-08-05 at 10:46 +0100, Bastien Nocera wrote:
> > On Wed, 2009-08-05 at 10:05 퍭㐱ꫭ詴낮譞:
> > > Once we get some icons for multimedia-player-apple-ipod-touch and
> > > phone-apple-iphone in the distros (gnome-icon-theme I guess) things will
> > > look neat aswell:
> > > http://media.sukimashita.com/temp/gvfs-backend-afc-screenshot-1.png
> > 
> > Did you file a bug against gnome-icon-theme?
> 
> Yes, http://bugzilla.gnome.org/show_bug.cgi?id=590835
> 
> > > Stuff that is open and related questions:
> > > 
> > > - Nautilus does not "browse" the device on automount anymore (This
> > > worked before we took the shadowing out. What enables it again?)
> > 
> > In the File Management preferences, select the Media tab, and tick
> > "Browse media when inserted".
> 
> I never unchecked that option. This is related to some detail that got
> missing after the shadow mount stuff was removed. I can't find the
> difference there though. Perhaps nautilus-2.27.4 is the "bad guy" here.

Did you restart nautilus?

> > > - Run "indent xy" to apply GVFS coding style (What indent rules does
> > > GVFS use?)
> > 
> > Don't know, but my vim rules will help. I'll try and do that today.
> 
> Superb, pushed it.

Attached is a patch to fix a few more of the problems I found (including
fixing the build, huh).

> <snip>
> I agree that making the backends compile standalone would be the best
> solution (but requires adjusting the API to be public).
> 
> Unless of course the GVFS crew loves to maintain a series of custom
> protocol backends (and their individual quirks). :)

You don't have to maintain the whole of gvfs. Only a few people care
about the obexftp backend (which I maintain), or the FTP backend (which
Benjamin maintains), etc

Ask for a GNOME account after the backend gets committed, and you can
start pushing fixes through bugzilla/people with commit rights in the
meanwhile.

Cheers