Alan Coopersmith wrote:
> Pg. 7: "Relevance" - Do you mean 2007, not 2008?
fixed.
> 
> Pg. 14: x64 is not just AMD any more - change "AMD x64" to just "x64"?
agreed.
> 
> Pg. 17: OpenGrok - should Mercurial be added to the list?
good point, I've added a section about source repositories.
> 
> Pg. 20: ZFS - haven't they dropped the "Zettabyte" name?   (If not,
>       shouldn't it be spelled the same as on pg. 24?)
changed both to ZFS.
> 
> Pg. 50: URL for NWAM info has visible formatting tags in it.
fixed.

> 
> Pg. 57: "Prebuilt OpenSolaris distributions are limited to SX:CR"
>       - old name for SX:CE
>       - what about Belenix?  Nexenta?  Schillix?  Martux?  SX:DE?
added.
> 
> Pg. 57: Memory requirements - should it mention the 768 Mb requirement
>       for the Developer installer?
yes, fixed.
> 
> Module 8:  Much of what is described is specific to the ON community,
>       not the rest of OpenSolaris - it should be clearly labeled as such.
labeled.
> 
> Pg. 66:  The last 2 paragraphs about Perl's POSIX.pod seems out of
>       place - what does it have to do with the kernel threading model?
I commented out these two paras.
> 
> 
> Pg. 106: Mozilla has been removed from Solaris/OpenSolaris - should this
>       example use firefox instead?
I'll need help to update the entire example with different variables. 
Max Bruning authored this in a training piece, maybe he has a newer 
example that we can use instead? I'll dig around.
> 
Thanks so much for this feedback, Alan! When I finish incorporating the 
changes submitted by other reviewers, I'll post new copy with change 
bars. Much appreciated!

-Michelle

Reply via email to