On 07/12/10 06:32 PM, Keith Mitchell wrote:
On 07/12/10 02:33 AM, Jan Damborsky wrote:
Hi Keith,


On 07/ 9/10 06:54 PM, Keith Mitchell wrote:
Hi Jan,

Thank you for reviewing. Responses below.

On 07/ 9/10 01:08 AM, Jan Damborsky wrote:
Hi Keith,

I have only couple of nits - please see below.

Jan



cmd/auto-install/README.test
----------------------------

29 Package: pkg:/install/installadm
->
29 Package: pkg:/system/install/auto-install

81 Package: pkg:/install/installadm
->
81 Package: pkg:/system/install/auto-install

The SUNWinstalladm-tools package was renamed to pkg:/install/installadm (and I've confirmed that install/installadm contains the executable mentioned, /usr/lib/installadm/webserver). The auto-install package is client side, as I understand it.

Yep. That I agree. However, it seems that you are referring
'cmd/ai-webserver/README.test' while I commented on 'cmd/auto-install/README.test'
which mentions /usr/bin/ai_get_manifest and /usr/bin/ai_sd binaries -
those are delivered by pkg:/system/install/auto-install.

Thank you,
Jan


Oh! You're right. Apologies for the confusion. I'll address that with my adjustment to the fix for 16434.

Keith, no problem at all.

Thank you for making those changes.
Jan

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to