[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG

bugzilla at redhat.com bugzilla at redhat.com
Tue Nov 30 21:29:29 UTC 2010


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=655601

--- Comment #10 from Ville Skyttä <ville.skytta at iki.fi> 2010-11-30 16:29:28 EST ---
Created attachment 463835
  --> https://bugzilla.redhat.com/attachment.cgi?id=463835
For reference: port of test suite generation to new Saxon, XSLT 2.0

http://scop.fedorapeople.org/packages/jing-trang.spec
http://scop.fedorapeople.org/packages/jing-trang-20091111-6.fc14.src.rpm

- Patch test suite generation to use Xalan.
- Include license texts in jing-javadoc.
- Make datatype-sample buildable out of the box, drop prebuilt jar.

Porting the test suite generation turned out to be surprisingly painful. 
Porting it to "new" Saxon did not work out - it worked with the upstream
saxon9.jar but no longer does with the newer Saxon open source "HE" versions
which is what we have in Fedora due to missing features in it, patch (which
ports it to XSLT 2.0) attached just for future reference.  Using EXSLT didn't
work either - new Saxon HE does not have support for that stuff, and Xalan
implements only a subset of it which isn't enough.  Switching it to XSLT 1.1 +
Xalan extensions worked though, but I had to disable one test case which
triggers an apparent Xalan bug; see comments in the xalan patch.  Xalan doesn't
currently support XSLT 2.0.

I decided not to do anything about the license stuff yet because I don't have a
good feeling about modifying upstream license texts (copying.html).  dtdinst
schema+xml moving from %doc and cataloguing was also postponed, will take a
look later.  Both are TODO comments in the specfile.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.


More information about the package-review mailing list