From owner-cvs-all@FreeBSD.ORG Wed Oct 17 12:28:38 2007 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 1BBB616A41B; Wed, 17 Oct 2007 12:28:38 +0000 (UTC) (envelope-from bright@elvis.mu.org) Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by mx1.freebsd.org (Postfix) with ESMTP id 0415513C47E; Wed, 17 Oct 2007 12:28:37 +0000 (UTC) (envelope-from bright@elvis.mu.org) Received: by elvis.mu.org (Postfix, from userid 1192) id A4C571A4D7C; Wed, 17 Oct 2007 05:28:37 -0700 (PDT) Date: Wed, 17 Oct 2007 05:28:37 -0700 From: Alfred Perlstein To: Ceri Davies Message-ID: <20071017122837.GM31826@elvis.mu.org> References: <200710152209.l9FM9Jv3017150@repoman.freebsd.org> <20071016102013.GE23043@submonkey.net> <4714F06B.7070702@FreeBSD.org> <20071016225906.GK23043@submonkey.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20071016225906.GK23043@submonkey.net> User-Agent: Mutt/1.4.2.3i Cc: cvs-src@FreeBSD.org, Stefan Esser , cvs-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: cvs commit: src/usr.bin/locate/locate updatedb.sh X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Oct 2007 12:28:38 -0000 * Ceri Davies [071016 15:59] wrote: > On Tue, Oct 16, 2007 at 07:10:03PM +0200, Stefan Esser wrote: > > Ceri Davies schrieb: > > > On Mon, Oct 15, 2007 at 10:09:19PM +0000, Stefan Esser wrote: > > >> se 2007-10-15 22:09:19 UTC > > >> > > >> FreeBSD src repository > > >> > > >> Modified files: > > >> usr.bin/locate/locate updatedb.sh > > >> Log: > > >> Make the updatedb script installed as /usr/libexec/locate.updatedb > > >> inspect all local file systems, not only ufs and ext2fs. A number > > >> of local file systems has been added over time, and at least zfs > > >> has the potential to become a popular choice. Without this change > > >> a ZFS root file system causes the script to ignore all file-systems > > >> and leads to an empty locate db. (An alternative is to add all the > > >> relevant file systems individually, which means that at least zfs, > > >> xfs, ntfs, ntfs-3g, msdosfs should be added, probably more). > > > > > > This now includes /dev, /dev/fd, /proc, /compat/linux/proc, CD-ROMs and DVDs > > > as well as duplicating everything in nullfs mounts, all of which are probably > > > undesirable -- well, they are undesirable to me; whether they are to > > > others is the question ;-). > > > > Ok, thank you for the information. I did not notice this in my tests. > > > > I'm not sure what the "local" pseudo fs-type is meant to be used for. > > Currently it appears to be identical to "not-remote" (which is what > > the locate db update tries to avoid, due to high network load and > > scan time). > > > > I plan to fix this within the next 24 hours (if anybody thinks it needs > > to be backed-out immediately, I'm willing to do this, but I'd rather > > put in the correct fix instead. > > > > Two possibilities: We could have another pseudo fs-type in find, which > > selects "real" file systems (and skips devfs, procfs, fdescfs, nullfs > > and probably more). Scanning of removable devices is dubious, too, > > independently of their file system. By selecting "local" and "disk > > based" file systems (which would omit not only the pseudo file-systems > > and nullfs, but also cd9660 and udf), no specific file systems needed > > to be put into the script (and find would have the knowledge). > > While userland can tell if a filesystem is pseudo or not, (see lsvfs), > I'm not sure that we can reliably tell whether a device is removable or > not (cf. my camera, USB key, iPod, etc.). fstab(5) gives some hints, "noauto" would be a giveaway as well as value of 0 for 'fs_passno' is also a hint. -- - Alfred Perlstein