Hi John, Why the call to sys.exit(1)?
Everything else looks ok to me. - Keith On 02/25/10 02:06 PM, John Fischer wrote: > Keith, > > You reviewed the previous rev of this fix. Slight modification > to be PEP8 compliant for the function name: > > http://cr.opensolaris.org/~johnfisc/13123-tftpboot > > for: > > 13123 installadm needs to reference SMF for tftp's > base directory, not inetd.conf > > http://defect.opensolaris.org/bz/show_bug.cgi?id=13123 > > The fix is to use /usr/bin/svcprop to get the entry from SMF > via a pipe. If there is a problem then it falls back to /tftpboot. > > I've tested it with installadm list and installadm delete-service > which are the consumers of the interface. > > Thanks, > > John > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
