[PATCH] Fixes meinproc/docbook can't handle paths with spaces (was: Re: meinproc/docbook can't handle paths with spaces ?)
neundorf at kde.org
Thu Dec 20 22:37:28 GMT 2007
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
dir/kdelibs/doc/kde4-config/man-kde4-config.1.docbook:4: I/O warning : failed
to load external entity "dtd/kdex.dtd"
dir/kdelibs/doc/kde4-config/man-kde4-config.1.docbook:6: parser error :
Entity 'language' not defined
More information about the kde-core-devel