Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:75369 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 68029 invoked from network); 11 Jul 2014 08:58:24 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 11 Jul 2014 08:58:24 -0000 Authentication-Results: pb1.pair.com smtp.mail=smalyshev@sugarcrm.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=smalyshev@sugarcrm.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain sugarcrm.com designates 108.166.43.107 as permitted sender) X-PHP-List-Original-Sender: smalyshev@sugarcrm.com X-Host-Fingerprint: 108.166.43.107 smtp107.ord1c.emailsrvr.com Linux 2.6 Received: from [108.166.43.107] ([108.166.43.107:36235] helo=smtp107.ord1c.emailsrvr.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id A7/94-28582-F27AFB35 for ; Fri, 11 Jul 2014 04:58:23 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by smtp14.relay.ord1c.emailsrvr.com (SMTP Server) with ESMTP id EE1EE18059D; Fri, 11 Jul 2014 04:58:20 -0400 (EDT) X-Virus-Scanned: OK Received: by smtp14.relay.ord1c.emailsrvr.com (Authenticated sender: smalyshev-AT-sugarcrm.com) with ESMTPSA id 832001802F5; Fri, 11 Jul 2014 04:58:20 -0400 (EDT) X-Sender-Id: smalyshev@sugarcrm.com Received: from Stass-MacBook-Pro.local (108-66-6-48.lightspeed.sntcca.sbcglobal.net [108.66.6.48]) (using TLSv1 with cipher DHE-RSA-AES128-SHA) by 0.0.0.0:465 (trex/5.2.4); Fri, 11 Jul 2014 08:58:20 GMT Message-ID: <53BFA72B.1080008@sugarcrm.com> Date: Fri, 11 Jul 2014 01:58:19 -0700 Organization: SugarCRM User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Ferenc Kovacs CC: Tjerk Meesters , PHP Internals References: <2264B27B-F9BE-4540-AD1C-5046143F18D2@gmail.com> <53BFA075.9000201@sugarcrm.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] Travis CI for pull requests From: smalyshev@sugarcrm.com (Stas Malyshev) Hi! > mentioned a few times now), and I think this will cause a significant > amount of pain for the people who wanna merge pull requests after the > phpng (or other similar major rewrite) is merged to the master, as they > will be required to backport the changes. When we will have the major rewrite merged (if it will be merged into master at all and not used as separate branch), we'll deal with it, but it didn't happen yet, so I was talking about the situation now. Right now I think we should use the same model many other projects use to run development - that is, one active development branch (master) and stable branches where patches are merged into on one-by-one basis. This is also a workflow described here for features: https://wiki.php.net/vcs/gitworkflow > Ofc. when we have a diverging master we can't really save the porting, > but we could offload that work to the PR authors, and porting forward is > better documented (UPGRADING.INTERNALS) than porting backwards. Right now it is not needed since master is not that different. When it'd be needed you'd probably need separate pulls for both branches anyway, since the porting effort may be significant. If you need to test your patch on multiple branches, you also can always fork on gihtub and hook Travis CI to your private fork, and run the builds on those too. Running CI does not require a pull. -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/