Sunanda Menon wrote:
>
> The changes have been incorporated .Could you please review it again at 
> the same link http://cr.opensolaris.org/~sunandam/6640026

Detail, but why the inconsistent indentation in this section?

+      echo "file is $file"
+if [ $(basename $dirs) = 'lib' ]; then
+             perm=644
+else
+             perm=755
+fi
+         _install P $file "$file"_new ${perm}


-- 
Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems

Reply via email to