-
Notifications
You must be signed in to change notification settings - Fork 126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
classatts.xsl producing "internal" Saxon error #276
Comments
I have to admit I did not read the > 2000 words of those forum posts thoroughly, but the take home message I got was not to add a conditional (which in theory should not do anything — if there are no descendant What version of oXygen are you running? And can you provide a small ODD so I could test it in my version? |
I'm using oXygen 19.0. And sure! This test ODD:
Run with this ANT code:
Produces this error:
|
I have oXygen 19.0 build 2017033118 on my (GNU/Linux) desktop. It uses Saxon 9.7.0.15. I’ve confirmed @joeytakeda’s bug happens to me, too. (I’m using the standard 3.1.0 and 7.42.0 frameworks.) However, I generated RELAX NG from the same ODD without problem using commandline So I’m thinking this is a solved Saxon problem, not a TEI problem. Thus the solution for oXygen to update to the newest version of Saxon. Once someone agrees with me I’ll send it along to SyncRO Soft. |
This might be a bug arising out of Oxygen's use of special optimizations when it calls saxon9ee, which we've seen before: The odd-to-relax transformation uses saxon9ee explicitly. I believe this can be prevented by replacing it with saxon9he, and/or commenting out the oxygen.jar file. I'm going to test that and see if the problem is solved with a test file. I'll make the change, let the bleeding-edge plugin build itself, then see if the test ODD works in Oxygen. |
Converting the test file with the bleeding-edge version of the plugin, updated today, with Oxygen 19.0 2017-04-20and TEI Version 3.3.0a last updated on, 7th August 2017, revision 33e8a19, the RNG file is produced with no problem. I'm going to get the latest Oxy build and test again to see if it's a regression. |
No problem with Oxy build 2017062918. @joeytakeda and @sydb can you confirm that if you call the TEI ODD to RelaxNG (XML syntax) built-in transformation on the test file in Oxygen, you still see the error? |
Sorry for the lateness of the reply! I'm using the same Oxy build (2017062918) and TEI Version 3.3.25, build number 2017-10-13-103823 and I am still seeing the almost exactly the same error; the only difference, as far as I can tell, is that the line number has changed from line 155 to 153.
|
I just tesetd in XML Developer 19.1, build 2017092911, and had no problems. |
@joeytakeda Can you update your Oxygen to the latest version and see if you can reproduce it? We can't make it happen on Mac or Linux with the current Oxygen. |
Updated to Oxygen 19.1 build 2017121318 and no more error! Thanks |
Closing this, then. There should be a category "Not fixed; went away." |
I'm using the bleeding edge TEI framework for oXygen to create an RNG from an ODD in an ANT build. Using oXygen and Saxon 9 HE, this line:
Produces this error:
According to this post in the Saxon forum (https://saxonica.plan.io/boards/3/topics/6543), this problem is because of "a predicate of the form /*[name()=$x] where $x is empty." Following the advice in the forum, I've squashed the error messages locally by simply adding an
<xsl:if/>
to classatts.xsl like so:The result RNG seems unchanged by my modifications, but the ODD I'm working with is quite simple, so it might not be the best test case.
The text was updated successfully, but these errors were encountered: