Build failed in Jenkins: TEIP5-Test-dev #2849
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2849/changes
Changes:
[s.bauman] Addressing issue #1422: improving constraints on <f>.
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2850/changes
Changes:
[s.bauman] Several minor tweaks: moved P5 citation to top; fixed validity error of
[s.bauman] Complete re-write per meeting w/ Elli: now generates a different (we
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2851/changes
Changes:
[lou.burnard] add sentence about textnode
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2852/changes
Changes:
[s.bauman] Fix spelling of "RELAX NG" most everywhere. (Not in title of cited work,
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2853/changes
Changes:
[s.bauman] Added <signed> to 'att.written' as per issue #481
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2854/changes
Changes:
[s.bauman] At LB's suggestion, removed <g> from content of <f>, as it makes no
[s.bauman] Addressing issue #1410, added <title> to class 'att.datable'.
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
The Jenkins build of P5 is failing because of the following test in .../P5/Test/testbasic.xml: <fLib xmlns="http://www.tei-c.org/ns/1.0"> <f name="xxx">A feature may have untyped content</f> <f name="yyy"> <string>or typed</string> </f> <f name="notgood"> <string>multiple types</string> <symbol value="doubleplusungood"/> </f> <f name="alsonotgood">mixed content <symbol value="doubleplusungood"/> </f> </fLib> I think these are tests that are *supposed* to fail, and thus should be in the detest/ suite, not here in testbasic.xml. 1) Can someone (Martin?) affirm for me that testbasic.xml is supposed to be valid? 2) Can someone (Laurent?, Lou?, Piotr?) affirm for me that an <f> is supposed to have at most 1 child element (from class model.featureVal, i.e. one of <fs>, <vColl>, <vMerge>, <vNot>, <binary>, <default>, <numeric>, <string>, <symbol>, <vAlt>, or <vLabel>)? If I've got this right, I'll just move these tests out of testbasic, and we'll be up and running again.
I have always assumed that tests intended to fail should be in detest, not in the other tests. But to be honest I think the whole set of tests was assembled in a piecemeal manner without much of a guiding plan, so I wouldn't be surprised to discover that there are intentional invalidities elsewhere. But in this particular case, I see the problem. If you look at the parsed log of the last successful build (http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2848/parsed_console/), you'll see this: validateschematron: [echo] Validate using Schematron [xslt] Processing /var/lib/jenkins/jobs/TEIP5-Test-dev/workspace/P5/Test/testbasic.xml to /dev/null [xslt] Loading stylesheet /var/lib/jenkins/jobs/TEIP5-Test-dev/workspace/P5/Test/testbasic.xsl [xslt] A feature value cannot contain both text and element content (tei:* and text()[normalize-space(.) ne '']) [xslt] A feature value can contain only one child element (count(tei:*) gt 1) In other words, these tests DO fail the Schematron check; but because the Schematron error message does not contain the trigger word ERROR, the build is not marked as having failed. In other words, the wrong test is being done in the wrong place the the results are being ignored. Two takeaways from this: 1. All Schematron rules we care about should have "ERROR" in their error messages, otherwise Jenkins is not going to notice them. If that's deemed too scary for end users, we need to find another trigger string that's not so bad, and then configure the Jenkins log parser to pay attention to it. 2. We need to go through all the tests and make sure they're actually working properly. There could be many more examples of this. Cheers, Martin On 2016-05-21 03:16 PM, Syd Bauman wrote:
The Jenkins build of P5 is failing because of the following test in .../P5/Test/testbasic.xml:
<fLib xmlns="http://www.tei-c.org/ns/1.0"> <f name="xxx">A feature may have untyped content</f> <f name="yyy"> <string>or typed</string> </f> <f name="notgood"> <string>multiple types</string> <symbol value="doubleplusungood"/> </f> <f name="alsonotgood">mixed content <symbol value="doubleplusungood"/> </f> </fLib>
I think these are tests that are *supposed* to fail, and thus should be in the detest/ suite, not here in testbasic.xml.
1) Can someone (Martin?) affirm for me that testbasic.xml is supposed to be valid?
2) Can someone (Laurent?, Lou?, Piotr?) affirm for me that an <f> is supposed to have at most 1 child element (from class model.featureVal, i.e. one of <fs>, <vColl>, <vMerge>, <vNot>, <binary>, <default>, <numeric>, <string>, <symbol>, <vAlt>, or <vLabel>)?
If I've got this right, I'll just move these tests out of testbasic, and we'll be up and running again.
The key phrase here is "schematron tests we care about" of course. As noted earlirr on this thread it's not clear that XML validation of any kind is really useful in this case since proper validation entails something more complicated using the fsd. The fact that our processes depend on the presence of some arbitrary word in a message seems a bit lame however. Sent from my Honor Mobile -------- Original Message -------- Subject: Re: [tei-council] content of in testbasic From: Martin Holmes To: s.bauman@neu.edu,tei-council@lists.tei-c.org CC: Piotr Banski ,Laurent Romary I have always assumed that tests intended to fail should be in detest, not in the other tests. But to be honest I think the whole set of tests was assembled in a piecemeal manner without much of a guiding plan, so I wouldn't be surprised to discover that there are intentional invalidities elsewhere. But in this particular case, I see the problem. If you look at the parsed log of the last successful build (http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2848/parsed_console/), you'll see this: validateschematron: [echo] Validate using Schematron [xslt] Processing /var/lib/jenkins/jobs/TEIP5-Test-dev/workspace/P5/Test/testbasic.xml to /dev/null [xslt] Loading stylesheet /var/lib/jenkins/jobs/TEIP5-Test-dev/workspace/P5/Test/testbasic.xsl [xslt] A feature value cannot contain both text and element content (tei:* and text()[normalize-space(.) ne '']) [xslt] A feature value can contain only one child element (count(tei:*) gt 1) In other words, these tests DO fail the Schematron check; but because the Schematron error message does not contain the trigger word ERROR, the build is not marked as having failed. In other words, the wrong test is being done in the wrong place the the results are being ignored. Two takeaways from this: 1. All Schematron rules we care about should have "ERROR" in their error messages, otherwise Jenkins is not going to notice them. If that's deemed too scary for end users, we need to find another trigger string that's not so bad, and then configure the Jenkins log parser to pay attention to it. 2. We need to go through all the tests and make sure they're actually working properly. There could be many more examples of this. Cheers, Martin On 2016-05-21 03:16 PM, Syd Bauman wrote:
The Jenkins build of P5 is failing because of the following test in .../P5/Test/testbasic.xml:
<fLib xmlns="http://www.tei-c.org/ns/1.0"> <f name="xxx">A feature may have untyped content</f> <f name="yyy"> <string>or typed</string> </f> <f name="notgood"> <string>multiple types</string> <symbol value="doubleplusungood"/> </f> <f name="alsonotgood">mixed content <symbol value="doubleplusungood"/> </f> </fLib>
I think these are tests that are *supposed* to fail, and thus should be in the detest/ suite, not here in testbasic.xml.
1) Can someone (Martin?) affirm for me that testbasic.xml is supposed to be valid?
2) Can someone (Laurent?, Lou?, Piotr?) affirm for me that an <f> is supposed to have at most 1 child element (from class model.featureVal, i.e. one of <fs>, <vColl>, <vMerge>, <vNot>, <binary>, <default>, <numeric>, <string>, <symbol>, <vAlt>, or <vLabel>)?
If I've got this right, I'll just move these tests out of testbasic, and we'll be up and running again.
-- tei-council mailing list tei-council@lists.tei-c.org http://lists.lists.tei-c.org/mailman/listinfo/tei-council PLEASE NOTE: postings to this list are publicly archived
Can we go ahead and do this please? I’m getting bored with the build failing constantly. The reason it’s throwing an error is because what was once enforced with Schematron (there being only one child node for <f>) is now enforced by the content model. The test expects the Schematron to fail, but the content model to be valid. But the content model is no longer valid. I don’t think we really need to test that invalid content is invalid, so I’d say remove the tests. They were testing Schematron that’s now gone. Hugh
On May 21, 2016, at 18:16 , Syd Bauman
wrote: The Jenkins build of P5 is failing because of the following test in .../P5/Test/testbasic.xml:
<fLib xmlns="http://www.tei-c.org/ns/1.0"> <f name="xxx">A feature may have untyped content</f> <f name="yyy"> <string>or typed</string> </f> <f name="notgood"> <string>multiple types</string> <symbol value="doubleplusungood"/> </f> <f name="alsonotgood">mixed content <symbol value="doubleplusungood"/> </f> </fLib>
I think these are tests that are *supposed* to fail, and thus should be in the detest/ suite, not here in testbasic.xml.
1) Can someone (Martin?) affirm for me that testbasic.xml is supposed to be valid?
2) Can someone (Laurent?, Lou?, Piotr?) affirm for me that an <f> is supposed to have at most 1 child element (from class model.featureVal, i.e. one of <fs>, <vColl>, <vMerge>, <vNot>, <binary>, <default>, <numeric>, <string>, <symbol>, <vAlt>, or <vLabel>)?
If I've got this right, I'll just move these tests out of testbasic, and we'll be up and running again. -- tei-council mailing list tei-council@lists.tei-c.org http://lists.lists.tei-c.org/mailman/listinfo/tei-council
PLEASE NOTE: postings to this list are publicly archived
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2855/changes
Changes:
[elli_mylonas] fixing #563 content model of u is paraContent
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2856/changes
Changes:
[elli_mylonas] fixing #563 content model of u is paraContent
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2857/changes
Changes:
[elli_mylonas] empty commit to change msg on previouscommit - fixing corrigible error
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2858/changes
Changes:
[philomousos] Fixed a couple of problems
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
See http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/2859/changes
Changes:
[philomousos] Update tcw22.xml
------------------------------------------
[...truncated 30 lines...]
rm -f test*.doc.xml test*.rnc test*.dtd test*.compiled.* test*.xsd test*.rnc test*.rng test*.xsl test*.isosch
rm -rf LOG *~ *.xsd Schema RomaResults DTD
rm -rf *.doc.*
rm -f detest.log detest.log.all
rm -f *-examples.rng *-examples.rnc *test*.nvdl *-ex.odd
rm -f detest.rnc detest.rng detest.dtd detest.isosch
make[1]: Leaving directory `http://teijenkins.hcmc.uvic.ca/job/TEIP5-Test-dev/ws/P5/Test'
rm -rf release Guidelines Guidelines-web Schema DTD dtd Split
rm -rf Guidelines.??? Guidelines-*
rm -f *.isosch.xsl
rm -f *.stamp
rm -f *.xsd
rm -f Exemplars/stdout
rm -f Test/*.isosch
rm -f Test/detest.dtd Test/detest.rnc Test/detest.rng Test/detest.xsl
rm -f Test/stdout
rm -f Utilities/guidelines.xsl
rm -f anything buildweb.xml
rm -f p5.sch p5.isosch p5.xml p5subset.xml p5subset.json p5subset.js
rm -f p5attlist.txt
rm -f p5odds-examples.rng p5odds-examples.rnc p5odds.rng p5odds.rnc p5odds.isosch
rm -f pdfbuild.log
rm -f stripspace.xsl.model
rm -f tei-*.zip
rm -f tei-p5-*_*build
rm -f tei-p5-*_*changes
rm -f tei-p5-*_*deb
rm -f teiwebsiteguidelines.zip
rm -rf FASC-*
rm -rf catalogue.* modList
rm -rf valid v.xml ValidatorLog.xml Utilities/pointerattributes.xsl graphics.sh missfont.log v.body v.header Schematron1.xml Schematron2.xml
Checking you have running XML tools and Perl before trying to run transform...
Ant:/usr/bin/ant
Perl:/usr/bin/perl
Java:/usr/bin/java
xmllint:/usr/bin/xmllint
touch check.stamp
get latest date:
VCS is git
if [ "git" = "svn" ] ; \
then \
if [ true = "true" ] ; \
then svn info --xml svn://svn.code.sf.net/p/tei/code/trunk/P5 ; \
else svn info --xml ; \
fi > repodate.xml ; \
fi
if [ git = "git" ] ; \
then \
git log --max-count=1 --pretty=format:"
Whoa! I had not noticed yesterday that the Jenkins build is busted. At first glance, it looks like the problem is the updates I made to the content model of <f>. The error occurs when trying to validate this bit of P5/Test/testbasic.xml: --------- | <!-- extra tests for feature values --> | <fLib xmlns="http://www.tei-c.org/ns/1.0"> | <f name="xxx">A feature may have untyped content</f> | <f name="yyy"> | <string>or typed</string> | </f> | <f name="notgood"> | <string>multiple types</string> | <symbol value="doubleplusungood"/> | </f> | <f name="alsonotgood">mixed content | <symbol value="doubleplusungood"/> | </f> | </fLib> --------- Looks to me like the 3rd and 4th <f> elements are *supposed* to fail. But I'm under the impression that tests in testbasic.xml, unlike tests in detest.xml, are supposed to always succeed. Any thoughts on this? But now, matters are worse. The new content model for <f> is | <content> | <alternate minOccurs="1" maxOccurs="1"> | <macroRef key="macro.xtext"/> | <classRef key="model.featureVal"/> | </alternate> | </content> In RELAX NG compact this would be expressed as | ( macro.xtext | model.featureVal ) which boils down to | ( ( text | g )* | model.featureVal.complex | model.featureVal.single ) which then boils down to | ( ( text | g )* | fs | vColl | vMerge | vNot | binary | default | numeric | string | symbol | vAlt | vLabel ) In RELAX NG this is fine. It represents exactly what we want, and works perfectly well. But in DTD-land, where the 'text' gets converted to #PCDATA, the parens around the "#PCDATA | g" cause an error. (Because DTDs require mixed content to be expressed with "#PCDATA" first in a single paren-group, nothing but OR bars, and an asterisk after the closing paren. See production 51 in 3.2.2 of the XML 1.0 Spec, 2nd edition.) I could have sworn that at one point Sebastian updated ODD->DTD processing so that if a #PCDATA was noticed anywhere in the content, the content model was altered to meet DTDs (ridiculous) constraint on content models that represent mixed content. But apparently I'm fantasizing, or he reverted that change. Our ODD processor does not magically fix this either for PureODD or for RELAX NG content in <content>. Is anybody up to making this change to our ODD->DTD processor? I'm not at all sure I am.
I think the simplest and most appropriate solution is to replace macro.xtext by <textNode> 1. This corresponds more closely to the published ISO scheme, which knows nothing of <g> elements 2. There is no reason to suppose that a text string describing a feature value would ever need to use a <g> element -- these are not transcriptions from source texts but created fragments of documentation 3. Even if there were such a reason, the way to do it would be to wrap the proposed string in the <string> element as content of the <f> The possibility of a textnode within <f> was agreed to only as a simplification of this more general approach. We don't need to agonise over making it work for every case. Indeed, in my view, we'd do better to consider removing it again. On 18/05/16 22:44, Syd Bauman wrote:
Whoa! I had not noticed yesterday that the Jenkins build is busted. At first glance, it looks like the problem is the updates I made to the content model of <f>. The error occurs when trying to validate this bit of P5/Test/testbasic.xml: --------- | <!-- extra tests for feature values --> | <fLib xmlns="http://www.tei-c.org/ns/1.0"> | <f name="xxx">A feature may have untyped content</f> | <f name="yyy"> | <string>or typed</string> | </f> | <f name="notgood"> | <string>multiple types</string> | <symbol value="doubleplusungood"/> | </f> | <f name="alsonotgood">mixed content | <symbol value="doubleplusungood"/> | </f> | </fLib> --------- Looks to me like the 3rd and 4th <f> elements are *supposed* to fail. But I'm under the impression that tests in testbasic.xml, unlike tests in detest.xml, are supposed to always succeed. Any thoughts on this?
But now, matters are worse. The new content model for <f> is
| <content> | <alternate minOccurs="1" maxOccurs="1"> | <macroRef key="macro.xtext"/> | <classRef key="model.featureVal"/> | </alternate> | </content>
In RELAX NG compact this would be expressed as | ( macro.xtext | model.featureVal ) which boils down to | ( ( text | g )* | model.featureVal.complex | model.featureVal.single ) which then boils down to | ( ( text | g )* | fs | vColl | vMerge | vNot | binary | default | numeric | string | symbol | vAlt | vLabel )
In RELAX NG this is fine. It represents exactly what we want, and works perfectly well. But in DTD-land, where the 'text' gets converted to #PCDATA, the parens around the "#PCDATA | g" cause an error. (Because DTDs require mixed content to be expressed with "#PCDATA" first in a single paren-group, nothing but OR bars, and an asterisk after the closing paren. See production 51 in 3.2.2 of the XML 1.0 Spec, 2nd edition.)
I could have sworn that at one point Sebastian updated ODD->DTD processing so that if a #PCDATA was noticed anywhere in the content, the content model was altered to meet DTDs (ridiculous) constraint on content models that represent mixed content.
But apparently I'm fantasizing, or he reverted that change. Our ODD processor does not magically fix this either for PureODD or for RELAX NG content in <content>.
Is anybody up to making this change to our ODD->DTD processor? I'm not at all sure I am.
I think that's a very good idea. Having heard no objection, for the moment I'm going to consider <g> in <f> as a corrigible error, and just fix it. (No deprecation, that is.) If Council doesn't like it, speak up!
I think the simplest and most appropriate solution is to replace macro.xtext by <textNode> 1. This corresponds more closely to the published ISO scheme, which knows nothing of <g> elements 2. There is no reason to suppose that a text string describing a feature value would ever need to use a <g> element -- these are not transcriptions from source texts but created fragments of documentation 3. Even if there were such a reason, the way to do it would be to wrap the proposed string in the <string> element as content of the <f> The possibility of a textnode within <f> was agreed to only as a simplification of this more general approach. We don't need to agonise over making it work for every case. Indeed, in my view, we'd do better to consider removing it again.
participants (5)
-
Hugh Cayless
-
Lou Burnard
-
Martin Holmes
-
Syd Bauman
-
Victoria TEI Jenkins