On Mon, 2009-10-12 at 16:10 -0400, Anurag S. Maskey wrote: > >>>>>>> http://zhadum.east/export/ws/am223141/temp/nwam1-work/webrev/ > >>>>>>> > >>>>>>> ... > > Or better yet, leave DLADM_FILES with absolute paths and simply call > > chgrp with the list of files on the command-line instead of using a > > loop: > > > > chgrp netadm $DLADM_FILES > > > accepted and updated webrev.
Style nit: On line 201, continuation lines should be indented by 4 spaces. Otherwise, looks good. -Seb
