Nicolas Williams wrote: > On Wed, Dec 17, 2008 at 06:31:05PM +0100, Pavel Filipensky wrote: >> I have updated the comments, new webrev is here (it also contains the >> latest umountall changeset from today): >> >> http://cr.opensolaris.org/~pavelf/6778894-v3 >> >> Can you give an explicit review of this workspace? > > You need to explicitly set NFS_LIST= the empty string, otherwise if > NFS_LIST happens to be set in the environment when unmountall runs...
Thanks for catching this. I have added: 246 NFS_LIST="" webrev is updated http://cr.opensolaris.org/~pavelf/6778894-v3 --Pavel > > I think I have a better way to deal with the whitespace issues too, > including newlines. I'll send a reply on the other thread sometime > after lunch.
