Your message dated Sat, 22 May 2004 11:23:31 -0700
with message-id <[EMAIL PROTECTED]>
and subject line Bug#249805: www.debian.org: tidy reports trimming empty <dd> 
for security pages
has caused the attached Bug report to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--------------------------------------
Received: (at submit) by bugs.debian.org; 19 May 2004 10:27:23 +0000
>From [EMAIL PROTECTED] Wed May 19 03:27:23 2004
Return-path: <[EMAIL PROTECTED]>
Received: from postfix3-2.free.fr [213.228.0.169] 
        by spohr.debian.org with esmtp (Exim 3.35 1 (Debian))
        id 1BQOIF-0006m4-00; Wed, 19 May 2004 03:27:23 -0700
Received: from bertol (lns-th2-4f-81-56-240-158.adsl.proxad.net [81.56.240.158])
        by postfix3-2.free.fr (Postfix) with ESMTP id 62B58D142
        for <[EMAIL PROTECTED]>; Wed, 19 May 2004 12:27:20 +0200 (CEST)
Received: from nico by bertol with local (Exim 3.35 #1 (Debian))
        id 1BQOIA-0000u1-00; Wed, 19 May 2004 12:27:18 +0200
Date: Wed, 19 May 2004 12:27:18 +0200
From: Nicolas Bertolissio <[EMAIL PROTECTED]>
To: Debian Bug Tracking System <[EMAIL PROTECTED]>
Subject: www.debian.org: tidy reports trimming empty <dd> for security pages
Message-ID: <[EMAIL PROTECTED]>
Mime-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;
        protocol="application/pgp-signature"; boundary="sXc4Kmr5FA7axrvy"
Content-Disposition: inline
X-Reportbug-Version: 2.58
User-Agent: Mutt/1.5.5.1+cvs20040105i
Delivered-To: [EMAIL PROTECTED]
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2004_03_25 
        (1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Status: No, hits=0.5 required=4.0 tests=BAYES_20,HTML_MESSAGE 
        autolearn=no version=2.60-bugs.debian.org_2004_03_25
X-Spam-Level: 


--sXc4Kmr5FA7axrvy
Content-Type: multipart/mixed; boundary="t0UkRYy7tHLRMCai"
Content-Disposition: inline


--t0UkRYy7tHLRMCai
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Package: www.debian.org
Severity: normal
Tags: patch

Hi,

looking at
   http://people.debian.org/~alfie/tidy/fr
tidy reports some warnings about empty <dd> tags, here is a patch that
should solve this problem for security pages. The warning appears when
<moreinfo> is empty, so I added a test on this. BTW this patch also
removes the "More info:" line if there is no more info.

It's written in perl as I'm a better at perl than at wml :)


Regards


Nicolas Bertolissio

-- System Information:
Debian Release: testing/unstable
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)
Kernel: Linux 2.6.4
Locale: [EMAIL PROTECTED], [EMAIL PROTECTED]

--=20

--t0UkRYy7tHLRMCai
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="security.wml.diff"

Index: security.wml
===================================================================
RCS file: /cvs/webwml/webwml/english/template/debian/security.wml,v
retrieving revision 1.117
diff -u -r1.117 security.wml
--- security.wml        23 Apr 2004 13:33:24 -0000      1.117
+++ security.wml        19 May 2004 10:18:21 -0000
@@ -74,8 +74,13 @@
        </dd>
 <dt><gettext domain="security">Security database references</gettext>:</dt>
        <dd><:= security_references("<secrefs/>") :></dd>
+<perl>
+  if ("<moreinfo/>" =~ /\S/) {
+    print "
 <dt><gettext domain="security">More information</gettext>:</dt>
        <dd><moreinfo/></dd>
+"; }
+</perl>
 #<ifneq "{#securitybody#}" "" "
 <ifneq "<isvulnerable/>" "no" "
 <dt><gettext domain="security">Fixed in</gettext>:</dt>

--t0UkRYy7tHLRMCai--

--sXc4Kmr5FA7axrvy
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFAqzaGPnQ0OX3ZAY0RAjEsAJ47M70aEoFJ1UPAnuBuuRh3F8qgjgCgpFVO
j+NxqUJ31a8ZlPOcRcH8q48=
=FhSQ
-----END PGP SIGNATURE-----

--sXc4Kmr5FA7axrvy--

---------------------------------------
Received: (at 249805-done) by bugs.debian.org; 22 May 2004 18:22:18 +0000
>From [EMAIL PROTECTED] Sat May 22 11:22:18 2004
Return-path: <[EMAIL PROTECTED]>
Received: from zoot.lafn.org [206.117.18.6] 
        by spohr.debian.org with esmtp (Exim 3.35 1 (Debian))
        id 1BRb8U-0003s3-00; Sat, 22 May 2004 11:22:18 -0700
Received: from localhost (host-66-81-16-238.rev.o1.com [66.81.16.238])
        by zoot.lafn.org (8.12.3p3/8.12.3) with ESMTP id i4MIM8O1006056
        (version=TLSv1/SSLv3 cipher=RC4-SHA bits=128 verify=NO);
        Sat, 22 May 2004 11:22:14 -0700 (PDT)
        (envelope-from [EMAIL PROTECTED])
Received: from kraai by localhost with local (Exim 4.32)
        id 1BRb9g-0000Mp-4k; Sat, 22 May 2004 11:23:32 -0700
Date: Sat, 22 May 2004 11:23:31 -0700
From: Matt Kraai <[EMAIL PROTECTED]>
To: Nicolas Bertolissio <[EMAIL PROTECTED]>
Cc: [EMAIL PROTECTED]
Subject: Re: Bug#249805: www.debian.org: tidy reports trimming empty <dd> for 
security pages
Message-ID: <[EMAIL PROTECTED]>
References: <[EMAIL PROTECTED]> <[EMAIL PROTECTED]> <[EMAIL PROTECTED]>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <[EMAIL PROTECTED]>
User-Agent: Mutt/1.5.6i
Sender: Matt Kraai <[EMAIL PROTECTED]>
X-Virus-Scanned: clamd / ClamAV version 0.70, clamav-milter version 0.70j
Delivered-To: [EMAIL PROTECTED]
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2004_03_25 
        (1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Status: No, hits=-6.0 required=4.0 tests=BAYES_00,HAS_BUG_NUMBER 
        autolearn=no version=2.60-bugs.debian.org_2004_03_25
X-Spam-Level: 

On Thu, May 20, 2004 at 08:55:48AM +0200, Nicolas Bertolissio wrote:
> Le mercredi 19 mai 2004, Matt Kraai ?crit :
> > On Wed, May 19, 2004 at 12:27:18PM +0200, Nicolas Bertolissio wrote:
> > > looking at
> > >    http://people.debian.org/~alfie/tidy/fr
> > > tidy reports some warnings about empty <dd> tags, here is a patch that
> > > should solve this problem for security pages. The warning appears when
> > > <moreinfo> is empty, so I added a test on this. BTW this patch also
> > > removes the "More info:" line if there is no more info.
> > > 
> > > It's written in perl as I'm a better at perl than at wml :)
> > 
> > Thanks for the patch, but how about simply removing the <moreinfo>
> > tags from the offending pages?
> 
> Well, I've not analysed all templates, and maybe this tag is used
> somewhere else, even if empty.
> 
> Moreover, some DSA have <moreinfo>, and some don't, so we need to
> display the <moreinfo> when it is available.  And, of course, we need
> some code to do so.
> This code should not be executed when the tag doesn't exist (I've not
> tried a "<dd><moreinfo/></dd>" if <moreinfo> doesn't exist, but I assume
> this issue a warning at least, or maybe prevents the page from building
> at all) and so we need to test whether the tag exists or not.
> In my patch I test whether it's empty or not, so I think we need some
> code in both case.  My patch simply avoid changing all existing DSA with
> empty <moreinfo>.

It should be fixed now.

-- 
Matt Kraai            [EMAIL PROTECTED]            http://ftbfs.org/

Reply via email to