Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:44366 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 71639 invoked from network); 19 Jun 2009 00:21:33 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 19 Jun 2009 00:21:33 -0000 Authentication-Results: pb1.pair.com smtp.mail=gwynne@darkrainfall.org; spf=permerror; sender-id=unknown Authentication-Results: pb1.pair.com header.from=gwynne@darkrainfall.org; sender-id=unknown Received-SPF: error (pb1.pair.com: domain darkrainfall.org from 208.97.132.5 cause and error) X-PHP-List-Original-Sender: gwynne@darkrainfall.org X-Host-Fingerprint: 208.97.132.5 mailbigip.dreamhost.com Received: from [208.97.132.5] ([208.97.132.5:47008] helo=homiemail-a5.g.dreamhost.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id B4/1A-08167-A0ADA3A4 for ; Thu, 18 Jun 2009 20:21:32 -0400 Received: from Moonstar.home (pool-71-174-84-161.bstnma.fios.verizon.net [71.174.84.161]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by homiemail-a5.g.dreamhost.com (Postfix) with ESMTP id E0821BC786; Thu, 18 Jun 2009 17:21:27 -0700 (PDT) Cc: PHP Developers Mailing List Message-ID: To: Scott MacVicar In-Reply-To: <4A3A5565.8000308@macvicar.net> Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes Content-Transfer-Encoding: 7bit Mime-Version: 1.0 (Apple Message framework v935.3) Date: Thu, 18 Jun 2009 20:21:26 -0400 References: <3B65D10E-3D2C-4AFB-8DC5-912D50C3BF23@darkrainfall.org> <4A3A5565.8000308@macvicar.net> X-Mailer: Apple Mail (2.935.3) Subject: Re: [PHP-DEV] Patch for bug #48575 From: gwynne@darkrainfall.org (Gwynne Raskind) On Jun 18, 2009, at 10:55 AM, Scott MacVicar 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. Correction: 10.3 as the minimum :). -- Gwynne