On 2017-11-16 12:07, Rainer Müller wrote: > On 2017-11-16 06:47, Eitan Adler wrote: >> There is a chance that I ran into a problem, "fixed" it by using >> xml:id, and then found the proper fix >> >> [64796 21:45:50.877 eax@FlyingEagle ~/svn/macports/guide]∴git push >> grimreaper && git pull-request >> .... >> https://github.com/macports/macports-guide/pull/11 >> >> Seems to just work > > Maybe I was a bit too fast when I merged the pull request... > > Although it builds fine and the resulting HTML looks correct now, the > 'make validate' step failed when trying to deploy the changes: > > $ make validate > /opt/local/bin/xmllint --xinclude --loaddtd --postvalid --noout > guide/xml/guide.xml > intro.xml:6: element chapter: validity error : No declaration for > attribute id of element chapter > installing.xml:8: element chapter: validity error : No declaration for > attribute id of element chapter > installing.xml:17: element section: validity error : No declaration for > attribute id of element section > ... > > I guess this was the reason why you had changed it before to xml:id="". > DocBook 5 only has xml:id="" [1], but why is this not converted into the > expected id="" for HTML output? Are we missing some option for docbook-xsl?
Okay, the problem was actually the regex in the Makefile for post-processing, where we still expect to match id="" in the HTML. https://github.com/macports/macports-guide/commit/7fa83b3565339e53de947b747bff088a7c5bafef Fixed and deployed. Rainer
