Looks good. Just one comment. --- svccfg.man1m.original Tue Jun 16 10:47:31 2009 +++ svccfg.man1m Fri Jun 19 16:53:08 2009 @@ -103,14 +103,20 @@ Service Profile Subcommands apply file
- If file is a service profile, then service instances - specified within the file are enabled or disabled - according to it. See smf(5) for a description of service - profiles. This command requires privileges to modify the - "general/enabled" property of the service instances. See - smf_security(5) for the privileges required to modify - properties. If file is not a service profile, the sub- - command fails. + If a file is a service profile, properties, including + general/enabled, which are specified in the file are + modified in the repository. Non-existing properties and + property groups will be created. The type of + pre-existing property groups will not be changed by the + profile. Existing properties can have their type + changed by the profile. Why do we allow changing property type and not property group? I'd think type is part of definition and not configuration. Is there a use case where we'd want to override existing property type? -tony