Hi Pavel, 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
It looks correct and I am ok with the change. Only very minor nit: The webrev is trying to say that a year at line 23 is already 2008, but here: http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/cmd/initpkg/umountall.sh we could see that the year is still 2007 in the gate. Is there any recent push to the file not reflected yet in the public gate at opensolaris.org? Have a nice day -- Marcel Telka Solaris RPE
