Hi Sue,
please see my comments below.
Thank you,
Jan
setup-sparc.sh
--------------
51 - since install.conf file containing service name is
stored in image area, I assume it implies that the image
can't be configured for more than one service - not sure
if this might be the issue, just checking since I think
this limitation doesn't exists for x86 case.
59 - I might recommend to use "install_service" keyword
instead of "service", so that it is the same as in x86 case.
125-127 - It seems that it is assumed here that network
is always C-class type (with 24-bit network prefix).
What happens if server is configured with different length
of network address ?
63-65 - I think those might be simplified
/usr/bin/mv ${tmpconf} ${installconf}
Similarly, I think 105-107 might be replaced with
/usr/bin/mv ${tmpconf} ${wanbootconf}
installadm-common.sh
--------------------
195-197 - since 'install_boot' option is not
used by AI client, I think those lines could be removed.
Susan Sohn wrote:
> Please review the changes for:
>
> 4194 need to make installadm tool changes for SPARC
> http://defect.opensolaris.org/bz/show_bug.cgi?id=4194
>
> which are posted at:
>
> http://cr.opensolaris.org/~sohn/4194
>
> Thanks,
> Sue
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
