From owner-freebsd-doc@FreeBSD.ORG Tue Aug 22 09:04:10 2006 Return-Path: X-Original-To: doc@freebsd.org Delivered-To: freebsd-doc@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C847E16A59A for ; Tue, 22 Aug 2006 09:04:10 +0000 (UTC) (envelope-from danger@FreeBSD.org) Received: from virtual.micronet.sk (smtp.micronet.sk [84.16.32.237]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5488443D7E for ; Tue, 22 Aug 2006 09:03:44 +0000 (GMT) (envelope-from danger@FreeBSD.org) Received: from localhost (localhost [127.0.0.1]) by virtual.micronet.sk (Postfix) with ESMTP id 6464F10E5B0 for ; Tue, 22 Aug 2006 11:03:35 +0200 (CEST) X-Virus-Scanned: by amavisd-new at virtual.micronet.sk Received: from virtual.micronet.sk ([127.0.0.1]) by localhost (virtual.micronet.sk [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id gsM9RMg-40xN for ; Tue, 22 Aug 2006 11:03:34 +0200 (CEST) Received: from danger.mcrn.sk (danger.mcrn.sk [84.16.37.254]) by virtual.micronet.sk (Postfix) with ESMTP id E8FE810E56E for ; Tue, 22 Aug 2006 11:03:33 +0200 (CEST) Date: Tue, 22 Aug 2006 11:03:42 +0200 From: Daniel Gerzo Organization: The FreeBSD Project X-Priority: 3 (Normal) Message-ID: <1418203821.20060822110342@rulez.sk> To: doc@freebsd.org MIME-Version: 1.0 Content-Type: multipart/signed; protocol="application/pgp-signature"; micalg="pgp-sha1"; boundary="----------473519D314CD620" Cc: Subject: merge query-pr.cgi and query-pr-summary.cgi, fix -T mode with new perl X-BeenThere: freebsd-doc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Daniel Gerzo List-Id: Documentation project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Aug 2006 09:04:10 -0000 ------------473519D314CD620 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: quoted-printable Hello doc, I have produced a patch [1] which combines two cgi scripts -- query-pr.cgi and query-pr-summary.cgi. It merges functionality of query-pr.cgi into -summary.cgi, but if we want to commit this I would recommend to rename query-pr-summary.cgi to query-pr.cgi (maybe symlink would be even better?) once commited. Patched version is available at [2]. I made it to display the query form by default and not the list all open PRs. This should at least a bit unburten GNATS. (note that the html we generate now has around 1.3MB when one is accessing query-pr-summary.cgi). I also fixed this script to be able to run in tainted mode. (I'm not sure if we should call die() if insecure data are passed, or just ignore the given filter) I would like you to review this patch. The only drawback I can see is that the query-pr-summary.cgi?prnumber does not work with new version. It is required to do query-pr-summary.cgi?pr=3Dprnumber (which was standard for original version). [1] http://people.freebsd.org/~danger/query-pr.diff [2] http://people.freebsd.org/~danger/cgi/query-pr-summary.cgi --=20 Best regards, Daniel mailto:danger@FreeBSD.org ------------473519D314CD620 Content-Type: application/pgp-signature -----BEGIN PGP MESSAGE----- Version: GnuPG v1.4.5 (MingW32) iD8DBQFE6shupjhloOLa1gsRAhq3AJ0ZWXQLDjG1h8A7w7Pc5XU1tQ5IJwCfdXZP ozZGAZWYVt7uhAsc8sM+q0w= =9SG/ -----END PGP MESSAGE----- ------------473519D314CD620--