Hi Jean.
On 07/06/09 18:58, Jean McCormack wrote:
> Jack Schwartz wrote:
>> Hi Jean.
>>
>> I checked that you got all instances of "pfinstall", so that part is
>> OK...
>>
>> I'm wondering about the comment change on line 421 of
>> perform_slim_install.c. The function in which the comment appears is
>> not "pfinstall", so to me it doesn't seem correct to change
>> "pfinstall" to the name of the current function. Not sure if it
>> might make more sense just to remove the sentence altogether?
> I'll change the sentence to
>
> /*
> * The .sysIDtool.state file needs to be written before the
> * install completes. Update the state here for install
> */
Sounds good.
Thanks,
Jack
>
> Jean
>>
>> Thanks,
>> Jack
>>
>> On 07/06/09 16:03, Jean McCormack wrote:
>>> I'd like a code review for 789 Remove the pfinstall references in
>>> liborchestrator
>>>
>>> Webrev:
>>> http://cr.opensolaris.org/~jeanm/slim_789/
>>>
>>> CR:
>>> http://defect.opensolaris.org/bz/show_bug.cgi?id=789
>>>
>>> Jean
>>> _______________________________________________
>>> caiman-discuss mailing list
>>> caiman-discuss at opensolaris.org
>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>
>
