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

Reply via email to