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

Re: Tidy validation failed


Shlomi Fish <shlomif@shlomifish.org> wrote:
> Hi Boyuan Yang,
> On Fri, 14 Feb 2020 13:55:02 -0500
> Boyuan Yang <byang@debian.org> wrote:
> > Hi debian-www members,
> > 
> > 在 2020-02-14五的 15:49 +0000,Debian Webmaster写道:
> > > *** /srv/www.debian.org/www/distrib/pre-installed.zh-cn.html
> > > line 666 column 3 - Warning: plain text isn't allowed in <tr> elements
> > > line 659 column 3 - Info: <tr> previously mentioned
> > > 
> > > --
> > >  You received this mail for the language code zh-cn.
> > >  Please edit webwml/english/devel/website/validation.data if this is not
> > > accurate.
> > >  Please also update webwml/english/devel/website/ with the new
> > > coordinator(s) data.  
> > 
> > I have been receiving this error for days but have no idea on how to resolve
> > the issue. The rendered HTML is indeed wrong but the source seems to be using
> > valid grammar. Can anyone take a look into it?

@Boyuan Yang:
I did not receive your mail from 14-02-2020 in my mailbox (don't know why), so 
I wasn't aware of that problem until today. Sorry

The problem is going down to a wrong formatting in one of my commits:

which I corrected a day later with

But for some reason, the Chinese variant of that page does not get newly
build now, it still has the wrong formatting and the superfluous
	Poland" />
within the Polish listing.

I remember that this happened in the past as well in some special
circumstances (however I did not found that in the debian-www list archive
The point is: that page includes "english/distrib/pre-installed.data"
to inject the list content from the English folder into all languages
pages. Now a change happened in the English part (pre-installed.data) and the
Chinese page did not get the newly build trigger.

To trigger a new build of the Chinese page, we could do another commit for the
Chinese source code, and indeed I found something, that looks incorrect to me
in chinese/distrib/pre-installed.wml
(This spurious line is not the reason for the missing-build situation from 
above, but we should fix it, if it's wrong, anyway.)


diff --git a/chinese/distrib/pre-installed.wml b/chinese/distrib/pre-installed.wml
index 9de72418a35..c0387fa4f97 100644
--- a/chinese/distrib/pre-installed.wml
+++ b/chinese/distrib/pre-installed.wml
@@ -28,21 +28,22 @@
 #<h2 id="submit">Information for hardware vendors</h2>
 <h2 id="submit">硬體廠商的<tw資訊></h2>
 #If you wish to be listed on this page, send mail to
 #<a href="mailto:webmaster@debian.org";>webmaster@debian.org</a>
 #providing the following information:
 <a href="mailto:preinstallvendors@debian.org";>preinstallvendors@debian.org</a>
 #<li>company name
 <li>網站 URL</li>
 #<li>e-mail address
 #<li>phone number (if accept international orders, include international dialing prefix)

Holger Wansing <hwansing@mailbox.org>
PGP-Fingerprint: 496A C6E8 1442 4B34 8508  3529 59F1 87CA 156E B076

Reply to: