Bernd Kuemmerlen wrote: > > I have a question regarding the planned DocBook 4.2 support in the next > version of XXE. > > Before you provided the downloadable DocBook 4.2 files, I had already > included my own version in my local installation. I have used a > different approach though, and I would like to discuss this here on the > list. > > The patch you provide makes a distinction between 4.1.2 and 4.2, these > are two totally different configurations. As far as I see currently, > this makes customization a bit harder than necessary, because I would > think that a customization I apply is not specific to the DocBook > version I use. > > My question is: Is it really necessary (there might be some internal > reasons for this) to separate DocBook 4.1.2 and DocBook 4.2 like you do > in your patch? > > I used the following approach to have DocBook 4.2 support in XXE: > - moved the existing config/docbook/dtd to config/docbook/dtd/4.1.2 > - put DocBook 4.2 in config/docbook/dtd/4.2 > - adjusted entity paths > - adjusted catalog.xml to the new paths > > Anything else stayed the same. I am quite sure that I have missed > something, but at least this setup worked for me... > > ... after having written this, I had a better look at the docbook42 > directory you provide, and it seems that the 4.2 configuration includes > large parts of the docbook 4.1.2 configuration anyway. It seems to me a > bit confusing to have two "toplevel" configurations (namely docbook and > docbook42) when this could also be achieved with one configuration. > > I realize that you are working on this anyway, so what we got with the > patch might not be what is in the next version, but anyway, these are > just my two cents, so please feel free to comment.
Your approach is better and unless, after trying it, we find something wrong with it (I don't see what), we'll use it in next version of XXE.
