From owner-freebsd-www@FreeBSD.ORG Mon Sep 18 19:27:40 2006 Return-Path: X-Original-To: freebsd-www@FreeBSD.org Delivered-To: freebsd-www@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5254616A573; Mon, 18 Sep 2006 19:27:40 +0000 (UTC) (envelope-from simon@zaphod.nitro.dk) Received: from mx.nitro.dk (zarniwoop.nitro.dk [83.92.207.38]) by mx1.FreeBSD.org (Postfix) with ESMTP id A374443E20; Mon, 18 Sep 2006 19:25:59 +0000 (GMT) (envelope-from simon@zaphod.nitro.dk) Received: from zaphod.nitro.dk (unknown [192.168.3.39]) by mx.nitro.dk (Postfix) with ESMTP id 3A925386C26; Mon, 18 Sep 2006 19:25:58 +0000 (UTC) Received: by zaphod.nitro.dk (Postfix, from userid 3000) id 1D5301141D; Mon, 18 Sep 2006 21:25:58 +0200 (CEST) Date: Mon, 18 Sep 2006 21:25:58 +0200 From: "Simon L. Nielsen" To: Shaun Amott Message-ID: <20060918192557.GE1038@zaphod.nitro.dk> References: <20060912201245.GA1915@picobyte.net> <20060917190419.GC33937@zaphod.nitro.dk> <20060917223229.GA1304@picobyte.net> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="ctP54qlpMx3WjD+/" Content-Disposition: inline In-Reply-To: <20060917223229.GA1304@picobyte.net> User-Agent: Mutt/1.5.11 Cc: freebsd-www@FreeBSD.org Subject: Re: RFC: New GNATS web (query-pr.cgi) interface X-BeenThere: freebsd-www@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: FreeBSD Project Webmasters List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 Sep 2006 19:27:40 -0000 --ctP54qlpMx3WjD+/ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 2006.09.17 23:32:29 +0100, Shaun Amott wrote: > On Sun, Sep 17, 2006 at 09:04:20PM +0200, Simon L. Nielsen wrote: > > The main thing which I think should be done before commit is use of > > cgi-style.pl. While it might seem unnecessary now, it really is much > > nicer for the people maintaing the website if there are as few places > > as possible where we need to change if modifications are made to the > > layout. Also, the script currently doesn't totally match the style of > > the rest of the FreeBSD.org web site, which I suspect is related to it > > not using cgi-style.pl. >=20 > Yes, the style is a vast improvement over the mess we have in place now. > But for the sake of consistency, I have switched to using cgi-style.pl. Great, thanks! > Hopefully, some of the improvements between the old style and mine can > be merged across the site at some point. But that's another discussion. Most of the CGI scripts could use some cleanup after having been in service for ~10 years... > > One other issue which also need to be fixed is that the script don't > > work with perl 5.0, which is still requires for a bit. >=20 > I've hacked out the modern Perl bits and pieces, it now runs on 5.0. Good, when we (hopefully soon) move to perl 5.8 on www.freebsd.org the code can be readded. > > The last thing I can think of now is that the script should use taint > > mode like shown below, just in case. >=20 > Indeed - I intended to add it anyway. Great again :-). I reviewed the changes to freefall:~shaun/public_html/stuff/gnatsweb.final/query-pr.txt and it looks good, so I it should just be committed and you can use rubber stamp (IE. Approved by me.). I also put it online on www.freebsd.org and it works fine. I'm not going to publish the URL here since I'm going to delete the script =66rom the temp location again soon. I assume you know how to add the CSS to the Makefile without breaking the build, but feel free to poke me if you have any question in that regard. --=20 Simon L. Nielsen --ctP54qlpMx3WjD+/ Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (FreeBSD) iD8DBQFFDvLFNE7ltJU9KiERArMPAJ0X8EcqjK7X6pkKO2wDoCRbqo3IkACgi51x L8aSHaPj0TtUK1Le7EWhytI= =Lah5 -----END PGP SIGNATURE----- --ctP54qlpMx3WjD+/--