Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76150 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 63928 invoked from network); 25 Jul 2014 22:30:53 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 25 Jul 2014 22:30:53 -0000 Authentication-Results: pb1.pair.com smtp.mail=tyra3l@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=tyra3l@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.54 as permitted sender) X-PHP-List-Original-Sender: tyra3l@gmail.com X-Host-Fingerprint: 209.85.192.54 mail-qg0-f54.google.com Received: from [209.85.192.54] ([209.85.192.54:44764] helo=mail-qg0-f54.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id E6/10-08559-B9AD2D35 for ; Fri, 25 Jul 2014 18:30:51 -0400 Received: by mail-qg0-f54.google.com with SMTP id z60so5941745qgd.27 for ; Fri, 25 Jul 2014 15:30:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=Ilzt6wDDyEXcCqyWtjl7cRqGilX5EhOuDjGZiaNxKys=; b=MOIjztBEcWz74jZVa0dGkNHHo0eGsGSK/OWFLL2KSXge9QTBhbcqU5XBsjBm9DxnPB MDjIVCYDJSQsOAsh8b4oevAC4vDnFRQwE5in6SVvQeO4fZmV80rTQYsvdfxDXXjZJUcW gLYSKK5QKZj5JneAFNu3FoMtUX2bZDMmILHxMDB6a4DXUBPssWC/nvmMDOyw5SjKQqKI MvSkQhyo3InZSJWRjxvlDBZQZ6UvT94IWb5guZ8KHHYnbAhAPsW3MTx5+nv9UzqZrn5W /KvjdaNB5CQDgS+2CrARcIHhpwzhbdAQvqqhTmnZy+lmEVgUZlrWRD0i9aOvDu+swh90 7sCQ== MIME-Version: 1.0 X-Received: by 10.140.80.170 with SMTP id c39mr30923524qgd.79.1406327455348; Fri, 25 Jul 2014 15:30:55 -0700 (PDT) Received: by 10.140.102.111 with HTTP; Fri, 25 Jul 2014 15:30:55 -0700 (PDT) In-Reply-To: <53D2D90B.2020005@litespeedtech.com> References: <539B8B74.6030707@sugarcrm.com> <53CB0347.2030902@sugarcrm.com> <53CEB32B.8030407@sugarcrm.com> <53D03599.20205@litespeedtech.com> <1406212844.3964.140.camel@guybrush> <53D18775.3020107@litespeedtech.com> <53D2C44C.6090106@litespeedtech.com> <53D2CA71.3090707@sugarcrm.com> <53D2D90B.2020005@litespeedtech.com> Date: Sat, 26 Jul 2014 00:30:55 +0200 Message-ID: To: George Wang Cc: Stas Malyshev , =?UTF-8?Q?Johannes_Schl=C3=BCter?= , PHP Internals Content-Type: multipart/alternative; boundary=001a11c1268c6a02b704ff0c207e Subject: Re: [PHP-DEV] Re: 5.3 final release From: tyra3l@gmail.com (Ferenc Kovacs) --001a11c1268c6a02b704ff0c207e Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi George, is there any reason why seemingly you never read my original mail about those branches beeing off-limit and those commit gonna be reverted? http://news.php.net/php.cvs/79411 ps: we prefer bottom-posting on the php.net lists, and that hasn't changed in the last couple of years. On Sat, Jul 26, 2014 at 12:24 AM, George Wang wrote: > All right, looks like it is my fault not making the reply sounds critical > enough. It pretty much make all third party php-litespeed rpms useless, > only causes trouble and confusion for LiteSpeed users. > > Maybe the release branches should be only open to the RMs, save everyone > time and efforts and force to follow the procedure. > > So, now moving forward, the code should be in PHP-5.4/5.5/5.6 branches > already, I will update NEWS files in 5.4/5.5/5.6 branches, anything else?= I > want to make sure the LiteSpeed SAPI code is up2date in next release. > > Best regards, > George Wang > > > > > > On 7/25/2014 5:21 PM, Stas Malyshev wrote: > >> Hi! >> >> I just found that my commits to PHP-5.4.31 and PHP-5.5.15 branch have >>> been voided, the result is that in final 5.4.31 and 5.5.15 release >>> package, sapi/litespeed code is still the ancient V5.5 release, it is >>> ridiculous! >>> >> No, it is not ridiculous, it is the release process. The three-number >> branches are release branches, and no commits but by the RMs should be >> done there. Moreover, even by the RMs the only commits that go there >> once the release is branched is either technical release commits >> (versions, NEWS, etc.) or urgent high-profile security fixes which could >> not be done in development branches, or other exceptional commits (like >> somebody discovering at the last moment Windows build is broken). >> >> In general, once the release branch is created, it is frozen except for >> urgent fixes, RC fixes/reverts and other exceptional cases. The decision >> of which commits to include is by the RM of the branch. >> >> The regular commits go into development branches - PHP-5.4, PHP-5.5, >> etc. I've sent you an email to verify if it is not an urgent commit, but >> both from review and from your answers it was clear that it was not, and >> there's no compelling reason to override regular release flow for it. >> Thus, these commits were not included into releases that were already >> well under way. >> >> What is the procedure to make sure our latest sapi/litespeed release >>> will be in the next release? >>> >> Creating a pull request and after review etc. committing it to the >> development branches - PHP-5.4, PHP-5.5, etc. There's more info here: >> https://wiki.php.net/vcs/gitworkflow >> >> In the future, I would recommend to ask on the list if the procedure is >> not clear. I understand there was a lot of change in how PHP release >> process works in the last couple of years, and not everybody may be up >> to date with it. Please ask and we will answer. We have docs, we have >> guides and we have this list where there are a number of people who can >> explain how to do things right. We've outgrown the situation of cowboy >> release management, we have a process now, which has been working pretty >> well so far. I am sorry that it caused your frustration, but be assured >> your changes are not lost and will be released in the next version in >> accordance to the regular release process. >> >> I would also encourage you to update NEWS files for the release branches >> from 5.4 up to reflect your changes. The changes are always made to the >> topmost release in the NEWS file. >> > > --=20 Ferenc Kov=C3=A1cs @Tyr43l - http://tyrael.hu --001a11c1268c6a02b704ff0c207e--