----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7597/#review13009 -----------------------------------------------------------
Overall this looks nice, and it's very useful to have this additional section in XML. docs/en-US/cloudstack.xml <https://reviews.apache.org/r/7597/#comment27995> Need to add this include statement to Installation_Guide.xml also, then verify that by building with publican-installation.cfg. docs/en-US/database-replication.xml <https://reviews.apache.org/r/7597/#comment27994> DOCTYPE should be chapter, not bookinfo docs/en-US/install-usage-server.xml <https://reviews.apache.org/r/7597/#comment27993> DOCTYPE should be chapter, not bookinfo docs/en-US/ssl.xml <https://reviews.apache.org/r/7597/#comment27991> DOCTYPE should be chapter, not bookinfo - Jessica Tomechak On Oct. 15, 2012, 12:41 p.m., Gavin Lee wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7597/ > ----------------------------------------------------------- > > (Updated Oct. 15, 2012, 12:41 p.m.) > > > Review request for cloudstack, David Nalley and Jessica Tomechak. > > > Description > ------- > > Make docbook xml for additional installation options, include usage server > installation, db replication and ssl. > > > Diffs > ----- > > docs/en-US/additional-installation-options.xml PRE-CREATION > docs/en-US/cloudstack.xml d9dca66 > docs/en-US/database-replication.xml PRE-CREATION > docs/en-US/install-usage-server.xml PRE-CREATION > docs/en-US/ssl.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/7597/diff/ > > > Testing > ------- > > Passed publican build using config file: publican-all.cfg, verified pdf/html > output. The changes won't impact admin guide doc. > > > Thanks, > > Gavin Lee > >
