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/