Below are the locations of the new webrev with Roland's latest changes, based
on review comments from round two of the final code review (July 9th version).
The base directory for the latest code review is:
http://cr.grommit.com/~chin/ksh93-webrev-jul13
FULL WEBREVS
All files:
http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul13-allfiles
Makefiles:
http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul13-makefiles
Non-AST files:
http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul13-nonattfiles
AST files:
http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul13-attfiles
INCREMENTAL WEBREVS
Current code (round three) vs. the codereview sent out July 11th (final review
round two, labelled July 9). Note that all three sets of incremental webrevs
are identical.
All files:
http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul9vjul13-allfiles
Makefiles:
http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul9vjul13-makefiles
non-AST files:
http://cr.grommit.com/~chin/ksh93-webrev-jul13/jul9vjul13-nonattfiles
NOTES
Same notes as listed in the pre-reviews:
https://www.opensolaris.org/jive/thread.jspa?threadID=32411&tstart=0
and previous rounds of final code review:
http://www.opensolaris.org/jive/thread.jspa?threadID=34912&tstart=0
Plus this additional fix for round 3:
RFE: Update ksh93-integration tree per second round of
Peter Memishian's review comments
http://bugs.grommit.com/show_bug.cgi?id=296
The files usr/src/lib/libcmd/{amd64,i386,sparc,sparcv9}/Makefile
show up in the incremental webrevs only; they revert back to the
version in the Nevada gate and will no longer be part of the
delivery for this project.
Also, usr/src/lib/Makefile.astinclude has been renamed to
usr/src/lib/Makefile.asthdr
Thanks,
April