Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:57493 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 37797 invoked from network); 24 Jan 2012 21:02:45 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 24 Jan 2012 21:02:45 -0000 Authentication-Results: pb1.pair.com header.from=jason.gerfen@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=jason.gerfen@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.42 as permitted sender) X-PHP-List-Original-Sender: jason.gerfen@gmail.com X-Host-Fingerprint: 209.85.215.42 mail-lpp01m010-f42.google.com Received: from [209.85.215.42] ([209.85.215.42:45667] helo=mail-lpp01m010-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 05/78-15695-37C1F1F4 for ; Tue, 24 Jan 2012 16:02:43 -0500 Received: by lahg1 with SMTP id g1so1508031lah.29 for ; Tue, 24 Jan 2012 13:02:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=wON286zLrlhJ34G4nHhRe4xVLHxi6hGiXf8ly6Y+M8E=; b=h48lkGoi+IM2Ftul0ByrclJe84a72XuwTzXe/JiRHDZnYfycRPYP5cjRgZpQUgsnpl lx72ChOeqTFVSe3wCtUaozQRPW3INjFFZBaeY5aXTSqtzPBwz7EMMA+8nBjqFRtiCtiZ qSu8pxSF/SLGkvUnh6w9izHXtzv9+eBNAAD0o= MIME-Version: 1.0 Received: by 10.112.41.169 with SMTP id g9mr3778205lbl.94.1327438960349; Tue, 24 Jan 2012 13:02:40 -0800 (PST) Received: by 10.152.127.137 with HTTP; Tue, 24 Jan 2012 13:02:40 -0800 (PST) In-Reply-To: <4F1F190F.1090009@oracle.com> References: <4F1F190F.1090009@oracle.com> Date: Tue, 24 Jan 2012 14:02:40 -0700 Message-ID: To: Christopher Jones Cc: internals@lists.php.net Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Subject: Re: [PHP-DEV] Patch inclusions From: jason.gerfen@gmail.com (Jason Gerfen) On Tue, Jan 24, 2012 at 1:48 PM, Christopher Jones wrote: > > > On 01/24/2012 03:36 AM, Jason Gerfen wrote: >> >> I just have a general question regarding patch inclusions. When a >> patch is submitted is there a formal review and testing of the patch >> prior to inclusion? From what I have been following from this list is >> that the process is patches get applied to new/existing bug/feature >> requests, bug fixes are priority numero uno, test cases should >> accompany new functions then testing? How likely are patches which may >> receive low usage from developers still applied? >> > > What you see on this list and on IRC (#php.pecl on EFNet) is what > happens. =A0It can take some persistence for people without commit karma > to get low interest patches applied because it can be hard to find a > reviewer/committer. =A0Having a clean patch with clear testcases helps. > Good to know. When you say 'clear' test cases, I assume you mean a .phpt file which calls and evaluates any new function as well as its available options. > >> Also, besides the usual retrieving latest testing branch, compiling >> running tests etc. how else can one help? > > > There are bugs to be triaged, documentation to be updated > (https://edit.php.net), the website prototype to be completed > (http://prototype.php.net/), information about PHP development > processes to be gathered & shared (I'm thinking about git here), new > community members to be greeted... > So my best bet in contributing is to try my hand at patching bugs vs. implementing feature requests as I did in https://bugs.php.net/bug.php?id=3D38917 > Chris > > -- > Email: christopher.jones@oracle.com > Tel: =A0+1 650 506 8630 > Blog: =A0http://blogs.oracle.com/opal/ --=20 Jas