William,

thank you very much for review.
I have incorporated all your suggested changes.

Thanks !
Jan


On 02/12/09 16:08, William Schumann wrote:
> Jan,
> Small changes in grammar and word choice. It's acceptable that English 
> articles are not present in short technical document.
> Test procedures seem fine.
>
> Here's my patch:
> 29c29
> < Provides test steps for testing liblogsvc library
> ---
> > Provides steps for testing liblogsvc library
> 37c37
> < Since at time of being there is no dedicated test driver developed
> ---
> > Since at the time being there is no dedicated test driver
> 44,45c44,45
> < [1] Install Target Discovery test driver - it is available
> < in official IPS repository
> ---
> > [1] Install Target Discovery test driver - available
> > from official IPS repository
> 84c84
> < [4] Test, if setting logging level works
> ---
> > [4] Test setting logging level
> William
>
> jan damborsky wrote:
>> Hi,
>>
>> could I please ask for reviewing the fix for following bug ?
>>
>> 6591 test procedure for liblogsvc library desired
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=6591
>>
>> webrev is available at:
>> http://cr.opensolaris.org/~dambi/bug-6591
>>
>> Thank you,
>> Jan
>>
>> _______________________________________________
>> caiman-discuss mailing list
>> caiman-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss


Reply via email to