Fix and Validate Docbook XMLBRL-CAD
Status: ClosedTime to complete: 48 hrs Mentors: SeanTags: docbook, xml, text editing

Some of BRL-CAD's existing documentation has been converted to DocBook format, but not all of the new files pass strict XML validation (particularly the new GCI files). We want to make our existing pages conform strictly and convert cleanly. The simplest way to find documents with errors is to add a CMake configuration flag when preparing to build BRL-CAD that enables a strict check when make is run. Adding this flag will cause a build failure if there's a syntax error:

-DBRLCAD_EXTRADOCS_VALIDATE=ON

Generally, the resulting error message will provide hints that can be used to identify and fix the problem. Also, once it is clear how to address one particular problem, it is likely that that problem will occur multiple times in different files. This will make the process faster once initial errors are solved.

This task involves preparing a BRL-CAD compilation environment with validation enabled, then cleaning up the DocBook XML files under the doc/docbook directory, fixing whatever errors arise and submitting a patch. You can build test your changes by compiling just a subset of the documentation. In your build directory (not source), you can run this:

cd doc/docbook  make

Once you're done, make and submit your patch: 

svn diff ~/brlcad.svn/doc/docbook  xmlfixes.patch

References:

Code:

  • doc/docbook/*/*/*.xml

 

Uploaded Work
File name/URLFile sizeDate submitted
man5final.patch13.0 KBDecember 02 2012 19:37 UTC
man5full.patch25.9 KBDecember 03 2012 01:31 UTC
man1full.patch24.9 KBDecember 03 2012 01:31 UTC
system_warnings.patch147.2 KBDecember 03 2012 14:42 UTC
system_contrib_fix.patch76.4 KBDecember 03 2012 16:57 UTC
Comments
Cezaron December 2 2012 19:21 UTCTask Claimed

I would like to work on this task.

Sean on December 2 2012 19:36 UTCTask Assigned

This task has been assigned to Cezar. You have 48 hours to complete this task, good luck!

Cezaron December 2 2012 19:37 UTCReady for review

The work on this task is ready to be reviewed.

Cezaron December 3 2012 01:35 UTCCommit r53957

Commit r53957

Sean on December 3 2012 03:41 UTCall committed?

If these are all committed, I'm still getting a slew of warnings throughout the compile.  Messages like this:


[  9%] Generating ../../../../../share/man/man1/anim_track.1


Note: meta manual : no titled ancestor of refentry                 anim_track


Note: meta manual : no refentry/refmeta/refmiscinfo@class=manual   anim_track


Note: meta manual : see http://docbook.sf.net/el/refmiscinfo       anim_track


Warn: meta manual : using "BRL-CAD" for "manual"                   anim_track


Note: Writing anim_track.1


[  9%] Built target anim_track_man1_man


[  9%] Generating ../../../../../share/doc/html/man1/en/anim_turn.html


[  9%] Built target anim_turn_man1_html


[  9%] Generating ../../../../../share/man/man1/anim_turn.1


Note: meta manual : no titled ancestor of refentry                 anim_turn


Note: meta manual : no refentry/refmeta/refmiscinfo@class=manual   anim_turn


Note: meta manual : see http://docbook.sf.net/el/refmiscinfo       anim_turn


Warn: meta manual : using "BRL-CAD" for "manual"                   anim_turn


Note: Writing anim_turn.1


 

Lots and lots.

Sean on December 3 2012 03:41 UTCTask Needs More Work

One of the mentors has sent this task back for more work. Talk to the mentor(s) assigned to this task to satisfy the requirements needed to complete this task, submit your work again and mark the task as complete once you re-submit your work.

Cezaron December 3 2012 06:54 UTCWarnings

Ah, wasn't aware we cared about the warnings, so I didn't fix those. Okay then, I'm on it.

Cezaron December 3 2012 14:43 UTCReady for review

The work on this task is ready to be reviewed.

Cezaron December 3 2012 14:51 UTCCommit r53961

`system_warnings.patch` should get rid of all the warnings inside doc/docbook/system/*


 


Commit r53961

Sean on December 3 2012 17:01 UTCTask Closed

Congratulations, this task has been completed successfully.

Cezaron December 3 2012 17:08 UTCFix contrib tag

I removed the contrib tag where the author is an organization.


Commit r53964


Patch file: system_contrib_fix.patch