> Date: Sun, 15 Jul 2007 23:10:02 -0700 (PDT)
> From: David.Comay at sun.com
> Subject: Re: final ksh93 code review, round three
> To: April Chin <April.Chin at sun.com>
> Cc: ksh93-integration-discuss at opensolaris.org, peter.memishian at sun.com, 
craig.mohrman at sun.com, james.d.carlson at sun.com, roland.mainz at 
nrubsig.org
> MIME-version: 1.0
> Content-transfer-encoding: 7BIT
> X-PMX-Version: 5.2.0.264296
> 
> > FULL WEBREVS
> 
> > Non-AST files:
> > http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul13-nonattfiles
> 
> I only have one remaining comment.
> 
> usr/src/lib/libshell/common/tests/sun_solaris_getconf.sh
> 
>       Line 32 - s/compatibilty/compatibility/
> 
>       (I just happen to see that but otherwise did not check these
>       scripts in detail.  You might want to spell check the comments
>       in all of the new non-ATT files just to make sure.)

Thanks...spell check came up with 4 more typos we'll fix:

./src/cmd/ksh/Makefile.com:54:# way to expliclity list each single flag.
                                       ^^^^^^^^^^
                                       explicitly
./src/lib/Makefile.asthdr:86:# Add temprary include files to the list of files 
to "clobber"
                                   ^^^^^^^^
                                   temporary
./src/lib/libshell/common/tests/sun_solaris_getconf.sh:32:# 
sun_solaris_getconf.sh - test the ksh93 getconf builtin for compatibilty 
                                                            ^^^^^^^^^^^^
                                                            compatibility
./src/lib/libc/port/regex/wordexp.c:86: * return s1-1 (position of string 
terminator ('\0') in destnation buffer).
                     ^^^^^^^^^^
                     destination
                     
Thanks,
        April

> 
> dsc


Reply via email to