[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Patch for Makefile.common [Was: Re: New page for devel/website]



On Tue, Aug 24, 2004 at 09:41:07PM +0200, Tobias Toedter wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On Tuesday 24 August 2004 20:36, Luk Claes wrote:
> > I think it's a good idea to put such a page on the site.
> >
> > But I also think we maybe need updated validate and tidy scripts? They
> > would ease the conversion and maintenance.
> 
> Well, I agree in principle. However, I don't think it would be a good idea
> to check for XHTML compliance right now, because poor Alfie (who's running
> the scripts in a cron job) would be overwhelmed by the sheer amount of
> errors.
> 
> I guess we should concentrate on fixing most of the outstanding issues
> before switching the scripts. For example, wml is currently "optimizing"
> the HTML code by removing the closing slash in e.g. <br />, thus printing
> <br>.
> 
> I think this should be fixed by applying the following patch to
> Makefile.common. Could someone please review this? Note that the expansion
> level of 3082 is what I concluded from the wml docs; I don't know whether
> Debian uses the default configuration of mp4h for the generation of the web
> pages.

This is not enough:
  1. DOCTYPE has to be changed and all files rebuild
  2. many source files have to be modified in order to be processed
     (basically they have to be XHTML compliant)
  3. generated files have to be checked by a validator to ensure that
     files are XHTML compliant
  4. contents of generated files have to be checked, displayed text
     should be identical to HTML ones
  5. this will trigger bugs in wml and/or mp4h which have to be fixed
  6. fixed packages have to be installed on www-master
IMO steps 1-5 should be performed by people interested in this
migration (on test machines, normally changes required in step 2 can
be committed because they should not break current processing), then
admins will be asked to install fixed packages and only then such a
patch could be applied.

Denis



Reply to: