-
Notifications
You must be signed in to change notification settings - Fork 1
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
Processing terminated by xsl:message at line 2137 in css-to-obfl.xsl #203
Comments
After the last pull from pipeline-mod-braille there is no focus attribute anymore. Not sure I ran the dotify-formatter tests but I think I did. Regarding the "processing terminated" error, is that related? If not, I'm gonna need that test book. |
This is the input file. The contents are obfuscated (same as in our automated tests). This is the input book for our tests, but by the time it reaches the braille script it looks like the HTML above: https://github.com/nlbdev/produksjonssystem/raw/dab0d8ac7f12e44b7d4849116b8242f2d9b19585/tests/558237.epub |
I discovered that the dotify-formatter tests had the focus attribute while trying to find the source of the error, but I don't think it's related. |
I can't reproduce the error so I assume it has been fixed in the mean time. I did hit another issue though. When a |
I haven't managed to test in on the latest version as I'm having trouble building the repo. ( |
I can build the linux distribution (but not the zip) locally. Had to fix the cli to be able to parse the version string for openjdk 10. Now I'm getting |
The exception is because of the issue with noterefs inside tables that I mentioned above. What do you mean by "I can build the linux distribution but not the zip"? |
Oh, I must've had my head in two places when I wrote that. I meant I can build the linux distribution but not the minimal distribution. (in the Dockerfile I can't build the linux distribution either) |
I think the problem in the Dockerfile is that the
|
Hmm, annoying. It seems that Christian does not ignore the .git folder when he builds his Docker image, but that's maybe not such a nice solution either. |
On a build server, the .git folder might not be too big. I think Jenkins normally does a shallow(?) clone. |
But will the |
Don't know. |
I can run your test book without errors now. |
This started happening between 3ea2d35 and b8a23ea. Those two commits have been built and are available under http://repo.nlb.no/pipeline/ using the updater, but not the intermediate commits. I haven't tried building the commits between them yet to determine which commit caused the issue.
The bug does not occur in all books, but it does happen with the book we use for system testing in our production system (which I can provide upon request). I have not yet tried to determine whether there is some specific part of the book that causes this issue.
I don't see an easy way to create a regression test for this issue. I can get the input for
css-to-obfl.xsl
which causes the error, butcss-to-obfl.xsl
depends on some Java stuff. The NLB translator is used, which is not available from the xprocspec tests in dotify-formatter. I don't think there's an easy way to import for instancecss-to-obfl.xpl
for use in a xprocspec test in mod-nlb either.The tests in dotify-formatter has been disabled (presumably by mistake) since 039bbe2 as a result of adding a
focus
attribute to one of the tests. Three of the tests intest_format.xprocspec
fail if thatfocus
attribute is removed:volume-break-inside: -obfl-keep(...)
The text was updated successfully, but these errors were encountered: