Hi Jean, This looks pretty good.
I do have one nit. The comment on line 799 of installadm.c is a bit confusing maybe something like the following would be clearer: We need to remove the existing entry in /etc/vfstab before adding the new entry and updating the smf information. -evan Jean McCormack wrote: > I need 2 reviewers for > > 5813 installadm delete-service does not remove entry from vfstab > http://defect.opensolaris.org/bz/show_bug.cgi?id=5813 > > The webrev is at: > http://cr.opensolaris.org/~jeanm/slim_5813/ > > Jean > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
