Hi Jan,

Just some minor comments on the updated webrev:

setup-sparc
-----------
92 (nit)
referred -> referenced
or you could also use
referred -> pointed to

240-245 Perhaps change the wording of:

"Service $srv_dfl is currently used by SPARC" \
      "clients not explicitly associated with another" \
      "service by the use of the 'create-client' subcommand"

to

"Service $srv_dfl is currently being used by SPARC" \
      "clients which have not explicitly been associated with another" \
      "service via the 'create-client' subcommand."

and (nit) add a comma after clients in:

"To select service $svc_name for those SPARC clients" \
                       "use the following commands:"


setup-service
-------------
259,268,273 Sparc-> SPARC

Thanks,
Sue



On 04/21/09 04:39, jan damborsky wrote:
> Hi,
> 
> could I please ask two people for reviewing changes for following blocker ?
> 
> 8262 'installadm create-service' shouldn't overwrite 
> /etc/netboot/wanboot.conf for Sparc, but notify user instead
> 
> webrev:
> http://cr.opensolaris.org/~dambi/bug-8262
> 
> Thank you very much,
> Jan
> 
> 
> modules affected:
> -----------------
> * installadm tools (Sparc platform)
> 
> testing done -  please see attached test procedures
> ---------------------------------------------------
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss


Reply via email to