[PATCH] Fixes meinproc/docbook can't handle paths with spaces (was: Re: meinproc/docbook can't handle paths with spaces ?)

[Alexander Neundorf]

On Thursday 20 December 2007, Ingo Klöcker wrote:
> On Thursday 20 December 2007, Alexander Neundorf wrote:
> > On Wednesday 19 December 2007, Ingo Klöcker wrote:
> > > On Tuesday 18 December 2007, Alexander Neundorf wrote:
> > > > On Tuesday 18 December 2007, Ingo Klöcker wrote:
> > > > > FWIW, below you'll find the function that parses the result of
> > > > > getenv("SGML_CATALOG_FILES"). The code is pretty
> > > > > straightforward and it's obvious that nobody thought about
> > > > > spaces in a catalog's path.
> > > >
> > > > Can you file a bug report for libxml ?
> > >
> > > Instead of doing this I have converted the SGML catalogs in
> > > kdoctools to XML catalogs and made meinproc write the catalog paths
> > > (encoded as file URLs) to the environment variable
> > > XML_CATALOG_FILES.
> > >
> > > Now kdelibs builds even if the source dir contains spaces.
> > >
> > > Is it okay to commit this patch and to svn add the new XML catalogs
> > > (see attached tar ball)?
> >
> > I can't comment on that.
> > IMO it is a bug fix, not a feature.
>
> Does the patch work for you?

The build worked now, but I still got warnings:

[ 65%] Generating kde4-config.1
/home/alex/src/kde4-svn/KDE 
dir/kdelibs/doc/kde4-config/man-kde4-config.1.docbook:4: I/O warning : failed 
to load external entity "dtd/kdex.dtd"
]>
  ^
/home/alex/src/kde4-svn/KDE 
dir/kdelibs/doc/kde4-config/man-kde4-config.1.docbook:6: parser error : 
Entity 'language' not defined
<refentry lang="&language;">
                          ^

Alex