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... 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.
