Pass strict validation

Bug #1275866 reported by Shaun McCance on 2014-02-03
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Documentation
Shaun McCance

Bug Description

Attaching a patch to make ubuntu-help pass strict validation with yelp-check. The command I use for validation is:

yelp-check validate --strict --allow --allow --allow *.page

Shaun McCance (shaunm-gnome) wrote :
Doug Smythies (dsmythies) wrote :

The patch does eliminate the strict validation errors.
On my 12.04 server I get 11610 of these (both before and after the patch):

Unimplemented block at ../../relaxng.c:9024

I'll try again on my 14.04 VM and with the upstream yelp-tools and yelp-xsl once I get it working.
I'm going to commit this in the meantime.

Changed in ubuntu-docs:
assignee: nobody → Shaun McCance (shaunm-gnome)
status: New → In Progress
Changed in ubuntu-docs:
status: In Progress → Fix Committed
Doug Smythies (dsmythies) wrote :

On my 14.04 Desktop VM, using the upstream yelp-tools and yelp-xsl (or so I believe), I still get several thousand

Unimplemented block at ../../relaxng.c:9054

lines printed. Lines spewing at such a high rate can potentially mask real issues.

Shaun McCance (shaunm-gnome) wrote :

My fault. I forgot that the --allow option requires a fix I did on libxml2, which appears to not be in any released version yet. I'll ping the libxml2 maintainer about getting a release out.

The patch is still fine. You just can't do the bestest in Mallard validation without building libxml2 from git master. You can use --strict without any --allow options, but it'll complain about any pages where you're using ITS or experimental features. You can also use yelp-check validate without --strict. It just might not report errors in how you've used things like conditionals or UI expanders.

Doug Smythies (dsmythies) wrote :

Thanks for the information. As a temporary solution, I'm doing this:

yelp-check validate --strict --allow --allow --allow *.page | grep -v "relaxng\.c"

And, ugh oh, a validation error was introduce in the latest, and already accepted, Merge Proposal.

Changed in ubuntu-docs:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers