Evan Layton wrote: > 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. OK. Done.
Thanks for the review. Jean > > -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 >
