Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:44357 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 74154 invoked from network); 18 Jun 2009 14:55:39 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 18 Jun 2009 14:55:39 -0000 Authentication-Results: pb1.pair.com smtp.mail=scott@macvicar.net; spf=permerror; sender-id=unknown Authentication-Results: pb1.pair.com header.from=scott@macvicar.net; sender-id=unknown Received-SPF: error (pb1.pair.com: domain macvicar.net from 97.107.131.220 cause and error) X-PHP-List-Original-Sender: scott@macvicar.net X-Host-Fingerprint: 97.107.131.220 whisky.macvicar.net Linux 2.6 Received: from [97.107.131.220] ([97.107.131.220:44266] helo=whisky.macvicar.net) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 0B/00-08167-A655A3A4 for ; Thu, 18 Jun 2009 10:55:38 -0400 Received: from [10.0.0.116] (office.vbulletin.com [217.155.246.60]) by whisky.macvicar.net (Postfix) with ESMTP id 4C0AE46957; Thu, 18 Jun 2009 10:55:35 -0400 (EDT) Message-ID: <4A3A5565.8000308@macvicar.net> Date: Thu, 18 Jun 2009 15:55:33 +0100 User-Agent: Thunderbird 2.0.0.21 (Windows/20090302) MIME-Version: 1.0 To: Gwynne Raskind CC: PHP Developers Mailing List References: <3B65D10E-3D2C-4AFB-8DC5-912D50C3BF23@darkrainfall.org> In-Reply-To: <3B65D10E-3D2C-4AFB-8DC5-912D50C3BF23@darkrainfall.org> X-Enigmail-Version: 0.95.7 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] Patch for bug #48575 From: scott@macvicar.net (Scott MacVicar) Gwynne Raskind wrote: > On Jun 18, 2009, at 10:46 AM, Gwynne Raskind wrote: >> Here is a nice simple patch for #48575 which rips out the >> mach-o/dyld.h functionality in Zend (as suggested by the original >> reporter and the Apple comment). According to my testing this not only >> doesn't break anything, but actually doesn't change anything either >> except removing some dead code and removing one configure check. Per >> Pierre's request, I'm sending it here for review :) > Looks good for me, apply to HEAD and document somewhere that we need 10.2 as the minimum version. Scott