Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:42671 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 18888 invoked from network); 17 Jan 2009 03:02:34 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 17 Jan 2009 03:02:34 -0000 X-Host-Fingerprint: 85.21.236.169 xdmitri.dialup.corbina.ru Received: from [85.21.236.169] ([85.21.236.169:16124] helo=localhost.localdomain) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 77/AD-41390-84A41794 for ; Fri, 16 Jan 2009 22:02:32 -0500 Message-ID: <77.AD.41390.84A41794@pb1.pair.com> To: internals@lists.php.net References: <496FDDD3.9000102@zend.com> Date: Sat, 17 Jan 2009 06:02:27 +0300 Lines: 17 X-Priority: 3 X-MSMail-Priority: Normal X-Newsreader: Microsoft Outlook Express 6.00.2900.5512 X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.5579 X-RFC2646: Format=Flowed; Original X-Posted-By: 85.21.236.169 Subject: Re: [patch] build_id implementation From: dmda@yandex.ru ("jvlad") > Hi! > > Attached is the patch implementing build ID. Please comment. If no > objections are raised I'm going to commit it tomorrow. ... >+ fprintf(stderr, "Cannot load %s - it was build with configuration %s, >whereas running engine is %s\n", Why not to call zend_error instead? It would make more sense because by default zend_error falls back to the output anyway and honors php.ini settings like error_log, so admins will see the trouble in the log if it is set. -jv