Glenn Lagasse wrote: > * Sundar Yamunachari (sundar.yamunachari at sun.com) wrote: > >> Glenn Lagasse wrote: >> >>> All, >>> >>> I've got what I believe to be the required changes for 6699. I've >>> addressed what I believe was the intent in the bug originally. Which is >>> to say that I've removed any code and reference to installadm displaying >>> a version number. >>> >>> They are located at: >>> >>> http://cr.opensolaris.org/~glagasse/slim_6699 >>> >>> http://defect.opensolaris.org/bz/show_bug.cgi?id=6699 >>> >>> That said, as part of some due diligence on my part I trolled around for >>> other 'version' references related to AI. I don't *think* what I've >>> found is related, but I'd like to get feedback from others who are >>> probably more familiar. >>> >>> Instances: >>> >>> usr/src/cmd/installadm/docs/manifest.html >>> 283: service name='ai_properties' version='1' type='service'> >>> >>> >> This is official document. We will look at it when we review the >> document. These doc files will be replaced for June release. >> >>> usr/src/cmd/auto-install/svc/auto-installer.xml >>> 38: version='1'> >>> >>> >> This is the version of the SMF service and it has nothing to do with >> installadm version. >> >>> usr/src/cmd/auto-install/ai_get_manifest.py >>> 783: <Version Number="0.5"> >>> >>> >> This is just a comment. >> >>> usr/src/cmd/ai-webserver: >>> *.py files reference a version number of some sort which appears >>> different than what installadm was reporting (1.0 for installadm, 0.5 >>> for the .py files) >>> >>> >> 0.5 appears to be the version of the module OptionParser. It has nothing >> to do with installadm version. Can you check with Clay about >> OptionParser? >> >>> Are any of these additional references relevant to the bug at hand? >>> They don't appear to be from my casual observation. >>> >>> >> No. >> > > Thanks Sundar. So just to be clear, you're fine with the changes I've > proposed in the webrev then? > I look at the changes to installadm.c and installadm.h. They looked fine. I was waiting for the final webrev. Like Sue mentioned in the other mail, you don't need to change installadm.html. This is the document delivered by Barbara and it will change for June release.
- Sundar > Thanks, > > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.opensolaris.org/pipermail/caiman-discuss/attachments/20090313/21514aac/attachment.html>
