[MPlayer-DOCS] Build system [patch]

[Diego Biurrun]

Torinthiel writes:
 > On Wed, Dec 24, 2003 at 07:16:43PM -0300, Diego Biurrun wrote:
 > > Torinthiel writes:
 > >  > On Wed, Dec 24, 2003 at 08:08:05PM +0100, Diego Biurrun wrote:
 > >  > > Torinthiel writes:
 > >  > > I have some small suggestions below.  Also please test it under Cygwin
 > >  > > as that was the original motivation for the build system change.  If
 > >  > > you cannot test it yourself, send it to the mplayer-cygwin list,
 > >  > > somebody there should be able to help you out.  I'd test it myself,
 > >  > > but I do not currently have access to a Windows box I can fiddle with.
 > >  > 
 > >  > Ok, applied almost all, except for one. Sent here, and I will send it to
 > >  > -cygwin as soon as I subscribe it, as I don't use it. And I'll even
 > >  > subscribe (for a brief period of time) to -os2 and -users, to ask if
 > >  > anybody has a different DTD location. Is there a list for MinGW? I think
 > >  > I should also ask there.
 > > 
 > > mplayer-cygwin also covers MinGW.
 > 
 > Thanks. Shouldn't it find itself into the docs?

Sure, feel free to add it.

 > >  > Maybe I should rename the root file to something indicating it's
 > >  > not usable without changing, but I don't have any sensible
 > >  > idea. Anyway, this is only cosmetics.
 > > 
 > > template_main.xml sounds reasonable, but there should be a way to do
 > > this directly without the intermediate template_main.xml.  I'll leave
 > > it as an exercise to the reader ;-)
 > 
 > I don't see any sensible way. The configure script does not know (and
 > should not) anything about documentation languages. So all the code
 > would have to go to Makefile or Makefile.inc. And I don't like the idea
 > of searching for DTD in any of them. It belongs into configure.

Well, you do have the directory names, which are the language code, so
you could parse those, or you could do something like

LANGS="en es pl ru"
for i in $LANGS; do
	create_main.xml
done

I'd still prefer that to a "stray" copy of main.xml lying around.

Diego