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