Liane Praza wrote: > Sean Wilcox wrote: >> The following bugs fixed here are : >> >> 6302173 startd dies after interaction with inetd / inetd service >> 6764671 Array overrun in librestart >> 6221667 method context errors should be logged in the service's instance >> log >> 6348085 Spurious "Ignoring duplicate environment variable" message >> 6411391 Empty method_context's don't work >> 5093847 importing a manifest should create method_context properties >> even if not set >> 6517270 partial method context definitions should work >> >> The full webrev is located at : >> >> http://cr.opensolaris.org/~swilcox/full_webrev/ >> >> The bfu only webrev is located at : >> >> http://cr.opensolaris.org/~swilcox/bfu_webrev/ > > I haven't done the review yet, but noticed when scanning through the file > list that you're missing the appropriate template updates in inetd.xml and > restarter.xml to reflect the fix for 6517270.
Sorry, I mis-remembered, and clearly mis-read when skimming the manifests again. I didn't actually do what I thought I did in the templates and declare that if the context was defined, all of the properties were 'required'. My mistake. No changes to those files are required. liane
