cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dave Cahill" <dcah...@midokura.com>
Subject Re: Review Request: Add docs for MidoNet networking plugin [CLOUDSTACK-996]
Date Tue, 30 Apr 2013 08:39:08 GMT


> On April 30, 2013, 8:01 a.m., Jessica Tomechak wrote:
> > docs/en-US/plugin-midonet-preparations.xml, line 40
> > <https://reviews.apache.org/r/10136/diff/4/?file=286089#file286089line40>
> >
> >     Believe the DTD requires a <para> tag within each entry tag. Like <entry><para>Setting
Name</para></entry>. The same comment applies throughout the doc here.
> 
> Dave Cahill wrote:
>     Wouldn't Publican fail to build on DTD errors?
>     
>     Running this command successfully builds the doc for me:
>     publican build --config=publican-plugin-midonet.cfg --langs=en-US --format=pdf

The DocBook  site also shows this as an example of a legal entry (no para):
<entry>3.1415927</entry>

ref: http://www.docbook.org/tdg/en/html/entry.html


- Dave


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10136/#review19943
-----------------------------------------------------------


On April 30, 2013, 5:22 a.m., Dave Cahill wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10136/
> -----------------------------------------------------------
> 
> (Updated April 30, 2013, 5:22 a.m.)
> 
> 
> Review request for cloudstack and Radhika PC.
> 
> 
> Description
> -------
> 
> Adding docs for MidoNet networking plugin [CLOUDSTACK-996]
> 
> Plugin itself is awaiting review / commit here:
> [PATCH 1] https://reviews.apache.org/r/9897/
> [PATCH 2] https://reviews.apache.org/r/9898/
> 
> 
> This addresses bug CLOUDSTACK-996.
> 
> 
> Diffs
> -----
> 
>   docs/en-US/MidoNet_Plugin_Guide.ent PRE-CREATION 
>   docs/en-US/MidoNet_Plugin_Guide.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-about.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-features.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-introduction.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-preparations.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-provider.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-revisions.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-ui.xml PRE-CREATION 
>   docs/en-US/plugin-midonet-usage.xml PRE-CREATION 
>   docs/publican-plugin-midonet.cfg PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10136/diff/
> 
> 
> Testing
> -------
> 
> Built docs.
> 
> 
> Thanks,
> 
> Dave Cahill
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message