Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76884 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 31146 invoked from network); 26 Aug 2014 19:55:39 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 26 Aug 2014 19:55:39 -0000 Authentication-Results: pb1.pair.com header.from=nikita.ppv@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=nikita.ppv@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.44 as permitted sender) X-PHP-List-Original-Sender: nikita.ppv@gmail.com X-Host-Fingerprint: 209.85.215.44 mail-la0-f44.google.com Received: from [209.85.215.44] ([209.85.215.44:48368] helo=mail-la0-f44.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 59/72-11622-936ECF35 for ; Tue, 26 Aug 2014 15:55:38 -0400 Received: by mail-la0-f44.google.com with SMTP id el20so15680481lab.31 for ; Tue, 26 Aug 2014 12:55:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:date:message-id:subject:from:to:content-type; bh=XTxpI1kzUETdOKw99b0BkrASVrNcnKhr8psSx1qQFzE=; b=G93aXQzsc1K3e0Ffk8OEu5AS5+vbESTmDXLTa8/FabYJl5DljtPPWiyr6Vy1uh3Wvb 5DuE6ZgWCpEeG8YUyN5zTd4f/cbOzEX6k3GyXoXAF8SO9Jqa7LBUiviOGaLb1hYNu1ZD Io1Vch6E2zHexqVc6Io0yanT82ohRRhd9J0aIs9/VvS1cvZwMTu/tWbge+8BEIt1WsCS uG0vfr51agBkkEWqO9jP7KLrVsM+g/OPfo5AmVgOEpMnMulhsmSWlFuzWeYysq3gpKLQ puHuiv21xDr4ZRREWxyunXiWcm7CI2A1W7GbVd98hTIjF4WwlER3eqN5TtlDbmo8CHa/ /j4w== MIME-Version: 1.0 X-Received: by 10.112.28.8 with SMTP id x8mr4201369lbg.104.1409082934693; Tue, 26 Aug 2014 12:55:34 -0700 (PDT) Received: by 10.25.154.209 with HTTP; Tue, 26 Aug 2014 12:55:34 -0700 (PDT) Date: Tue, 26 Aug 2014 21:55:34 +0200 Message-ID: To: PHP internals Content-Type: multipart/alternative; boundary=001a11340428c8166005018dafa3 Subject: Remove generated lexers from git? From: nikita.ppv@gmail.com (Nikita Popov) --001a11340428c8166005018dafa3 Content-Type: text/plain; charset=UTF-8 Hi internals! Currently our git repo contains files like zend_language_scanner.c, zend_ini_scanner.c, etc which are files generated by re2c. Historically these files have been included because re2c was not readily available on many platforms. In the thread on bison 3 compatibility [1] there was some discussion as to whether this limitation still applies. Quoting Adam Harvey: > +1. I don't think re2c is that onerous a requirement anyway, for the > most part: it's available through apt-get, brew, yum, and probably > most other packaging systems. Given the amount of other things a > developer has to install to build php-src from git, re2c is hardly > going to break the camel's back. So, I'd like to bring this up again. Are there any objections to removing generated lexer files from the repo? If not, does it suffice to just git rm them and add them to .gitignore, or are other changes required? Nikita [1]: http://markmail.org/message/x3p7ltekzws6ywhw --001a11340428c8166005018dafa3--