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