[MPlayer-dev-eng] [PATCH] Autoconfiguration for XML DOCS building system

Dmitry Baryshkov lumag at qnc.ru
Sun Jun 15 18:24:18 CEST 2003


Hello!
В ?? 14.06.2003, в 20:28, Diego Biurrun пишет:
> OK, I finally got around to checking this, I like but I did some
Thank you.
> modifications, let's discuss it first before I commit.
> 
> Changes are spelling and cosmetics (you had a mixture of tabs and
> spaces), as well as some more substantive things.
They are OK
> The Makefile now builds HTML by default now instead of printing a help
> message.
Suppose it's OK.
> You use things like
> 
>   if test "x$_catalog" != "x"
>   if test "x$_chunk_xsl" = "x"
> 
> which is OK, but 
[snip]
> are simpler IMHO.  The creation of html.xsl etc are also simplified
> IMHO.
OK
> Let me know what you think (and whether it still works for you ;-)),
It worked :)
But: I noticed that it could end up in such situation: Saxon processor &
 _java="". Attached contains quick fix.
Also I noticed, that final gij-3.3 from Debian sid can't work with
installed on my box saxon-6.4.4.jar (It worked with gij-3.2). So I
disabled it.
Also added .cvsignore
> otherwise I will commit this during the next days.
OK. But don't forget to remove 3 generated files before commit
(you included them in patch, so that's why I'm mentioning it).

Also, be aware, that "make test" is broken now, so I removed them from
"make help".
> 
> Diego
[from another letter]
> Hmm, another thought..  Should we put this into the main configure
instead?
I asked this in this list before starting this work.
Here is answer from gabucino at mplayerhq.hu:
IMHO this shit shouldn't be in configure.

So, that's why I put it into new script.

Also, I made some 'timing' (
real/user times from "time make distclean all", using various
processors.):
XSLTproc:               3m30.435s/3m6.500s
Saxon (SUN J2SE1.4):    3m34.360s/2m36.790s
Jade:                   0m50.425s/0m43.210s
OpenJade:               6m0.737s/4m55.710s

So we must look into better support of Jade (Currently It can't be made
default one, because it doesn't use .xsl styling. Maybe we must look
into writing proper (equivalent to html-common.xsl) .dsl .
-- 
With best wishes
Dmitry Baryshkov <lumag at qnc.ru>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch
Type: text/x-patch
Size: 11621 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20030615/68237235/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: ??? ????? ????????? ????????? ???????? ????????
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20030615/68237235/attachment.pgp>


More information about the MPlayer-dev-eng mailing list