Skip site navigation (1)Skip section navigation (2)
Date:      10 Sep 2002 18:51:42 -0400
From:      Joe Marcus Clarke <marcus@FreeBSD.org>
To:        Maxim Sobolev <sobomax@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: ports/x11-fm/nautilus Makefile ports/x11-fm/nautilus/files patch-libnautilus-private::nautilus-volume-monitor.c
Message-ID:  <1031698302.43819.30.camel@gyros.marcuscom.com>
In-Reply-To: <20020910220212.GC3821@vega.vega.com>
References:  <200209102024.g8AKO7kA047896@freefall.freebsd.org>  <20020910220212.GC3821@vega.vega.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 2002-09-10 at 18:02, Maxim Sobolev wrote:
> On Tue, Sep 10, 2002 at 01:24:07PM -0700, Joe Marcus Clarke wrote:
> > marcus      2002/09/10 13:24:07 PDT
> > 
> >   Modified files:
> >     x11-fm/nautilus      Makefile 
> >     x11-fm/nautilus/files 
> >                           patch-libnautilus-private::nautilus-volume-monitor.c 
> >   Log:
> >   Port sobomax's nautilus2 volume patches to nautilus, and take things a step
> >   further.  Now, removable media such as CDs, Zip drives, etc. are correctly
> >   handled by nautilus with pretty little icons.  Bump PORTREVISION.
> 
> WOW, wonderfull! Any plans to backport (forwardport maybe) those changes
> for nautilus2?
> 

Working on it now.  I had to write some code to get the CD names to pop
up correctly.  Funny you should write, I have a question.

You changed the code for detecting removable media to look for /mnt/ in
the mount point name.  I don't think this is right for FreeBSD.  hier(7)
says /mnt is used as a temporary mount point, and FreeBSD uses /cdrom. 
I've, therefore, changed the code to always return TRUE when asked if
the volume is removable.  This seems okay since the function only passes
removable volumes into it anyway.

The other solution is to go back to looking for "noauto" in fs_mntops. 
What do you think?

Joe

> -Maxim
> 
-- 
Joe Marcus Clarke
FreeBSD GNOME Team	::	marcus@FreeBSD.org
http://www.FreeBSD.org/gnome


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1031698302.43819.30.camel>