Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76891 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 44637 invoked from network); 26 Aug 2014 21:17:10 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 26 Aug 2014 21:17:10 -0000 Authentication-Results: pb1.pair.com header.from=smalyshev@sugarcrm.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=smalyshev@sugarcrm.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain sugarcrm.com designates 108.166.43.83 as permitted sender) X-PHP-List-Original-Sender: smalyshev@sugarcrm.com X-Host-Fingerprint: 108.166.43.83 smtp83.ord1c.emailsrvr.com Linux 2.6 Received: from [108.166.43.83] ([108.166.43.83:51743] helo=smtp83.ord1c.emailsrvr.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id A2/10-43626-259FCF35 for ; Tue, 26 Aug 2014 17:17:10 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by smtp3.relay.ord1c.emailsrvr.com (SMTP Server) with ESMTP id 9189C18041A; Tue, 26 Aug 2014 17:17:04 -0400 (EDT) X-Virus-Scanned: OK Received: by smtp3.relay.ord1c.emailsrvr.com (Authenticated sender: smalyshev-AT-sugarcrm.com) with ESMTPSA id 1FD8F18039F; Tue, 26 Aug 2014 17:17:04 -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.2 with cipher DHE-RSA-AES128-SHA) by 0.0.0.0:465 (trex/5.2.10); Tue, 26 Aug 2014 21:17:04 GMT Message-ID: <53FCF94F.1090702@sugarcrm.com> Date: Tue, 26 Aug 2014 14:17:03 -0700 Organization: SugarCRM User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:31.0) Gecko/20100101 Thunderbird/31.0 MIME-Version: 1.0 To: Andrea Faulds , Nikita Popov CC: PHP internals References: <23EA2BFD-AAA8-4D5E-92D6-8D7B57332291@ajf.me> In-Reply-To: <23EA2BFD-AAA8-4D5E-92D6-8D7B57332291@ajf.me> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [PHP-DEV] Remove generated VM from git? From: smalyshev@sugarcrm.com (Stas Malyshev) Hi! > Why would a non-internals person want to build PHP from git? That’s Why not? It's an open-source project, isn't it? People may prefer using git, many integration systems (including PHP's own composer) rely on git. > just making things harder on themselves. Besides PHP itself (should > my proposal succeed), you need other extra dependencies, including a > lexer generator if Nikita’s proposal succeeds. It's like "since we depend on gcc, adding more dependencies is no problem". Doesn't make any sense to me, adding dependencies makes it harder, so it is more problems. Especially with recursive dependencies. > 1. We avoid git tracking generated files that don’t provide > meaningful diffs and that can have massive changes just from changing > the source code or the generation script. > 2. It’s no longer necessary to manually generate the VM every time an > opcode is modified. (Less debugging pain if you forget.) That's not an advantage. Having extra file in git is no problem at all, we don't pay per byte, and somebody who can't handle regenerating the VM file should not be messing with the VM (for one, they would notice the problem immediately on running the test for the change locally, and if they don't test the changes locally we probably don't want these changes). And it's not like we change the VM every day. -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/