[libvirt] [PATCH] build: fix HTML errors in nwfilter docs

Eric Blake eblake at redhat.com
Wed May 26 13:59:38 UTC 2010


On 05/26/2010 02:44 AM, Daniel P. Berrange wrote:
>> * docs/formatnwfilter.html.in: Fix invalid HTML constructs.
>> Reported by Eduardo Otubo.
>> ---
>>
>> Don't know why a build on Fedora didn't catch these, so I
>> won't push until I get an ack that this fixes things on Ubuntu.
> 
> Are you missing the 'xhtml1-dtds' RPM on your machine ?

$ rpm -q xhtml1-dtds
xhtml1-dtds-1.0-20020801.5.noarch

But either that didn't catch it; or it did catch it but we have a bug in
the makefile that ignored the failure, such that I didn't notice it
scroll by.  Just to test things, I reverted to an older version, and
tried again; sure enough:

Generating formatnwfilter.html.tmp
formatnwfilter.html.in:390: HTML parser error : Unexpected end tag : p
    </p>
        ^
formatnwfilter.html.in:705: HTML parser error : Unexpected end tag : code
         <td>End of range of valid source ports</code></td>
                                                      ^
formatnwfilter.html.in:710: HTML parser error : Unexpected end tag : code
         <td>Start of range of valid destination ports</code></td>
                                                             ^

It's still the right thing to skip a test if xhtml1-dtds is not
installed, but it looks like we could improve the makefile to bail out
when we detect bad html.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100526/bf9e99c0/attachment-0001.sig>


More information about the libvir-list mailing list