AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:
CFG:
PT:0
S:andy.sharp@lsi.com
RQ:
SSV:mhbs.lsil.com
NSV:
SSH:
R:<Ed.Kwan@lsi.com>,<larry.scheer@lsi.com>
MAID:2
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/LSI/INBOX	0	2B044E14371DA244B71F8BF2514563F50410E6CD@cosmail03.lsi.com
X-Sylpheed-End-Special-Headers: 1
Date: Mon, 16 Nov 2009 18:20:10 -0800
From: Andrew Sharp <andy.sharp@lsi.com>
To: "Kwan, Ed" <Ed.Kwan@lsi.com>
Cc: Larry Scheer <larry.scheer@lsi.com>
Subject: Re: please review code change for TED 17769 "system copy all"
 copies NFS mounts
Message-ID: <20091116182010.25d4946b@ripper.onstor.net>
In-Reply-To: <2B044E14371DA244B71F8BF2514563F50410E6CD@cosmail03.lsi.com>
References: <2B044E14371DA244B71F8BF2514563F50410E6CD@cosmail03.lsi.com>
Organization: LSI
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Fri, 13 Nov 2009 11:34:27 -0700 "Kwan, Ed" <Ed.Kwan@lsi.com> wrote:

> Change 33808

Change 33808 by edk@edk-dev on 2009/11/13 10:33:14 *pending*

        For TED 17769 "system copy all" copies NFS mounts
        Remove "mnt" from the list of files/directories to copy.
        Reviewed by

Affected files ...

... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_flash-openbsd.c#18 edit


I strongly wish to NAK this one.  NFS mounted filesystems is not a
supported configuration, hence the bug is WAD.  And, who's to say I
don't have something on /mnt locally that I think should be copied?
Also, just removing /mnt doesn't do it, there could be many NFS mounted
directories in many places, or one in some other place, etc.

CC'ing to Larry for the tie breaker since he's pretty familiar with
this code area too.
