Return-Path: X-Original-To: apmail-httpd-dev-archive@www.apache.org Delivered-To: apmail-httpd-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 5DC579BF3 for ; Wed, 1 Feb 2012 13:20:31 +0000 (UTC) Received: (qmail 54907 invoked by uid 500); 1 Feb 2012 13:20:30 -0000 Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 54616 invoked by uid 500); 1 Feb 2012 13:20:29 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: List-Post: List-Id: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 54607 invoked by uid 99); 1 Feb 2012 13:20:29 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 01 Feb 2012 13:20:29 +0000 X-ASF-Spam-Status: No, hits=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of rainer.jung@kippdata.de designates 195.227.30.149 as permitted sender) Received: from [195.227.30.149] (HELO mailserver.kippdata.de) (195.227.30.149) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 01 Feb 2012 13:20:22 +0000 Received: from [10.0.110.6] ([192.168.2.104]) by mailserver.kippdata.de (8.13.5/8.13.5) with ESMTP id q11DK11E019875 for ; Wed, 1 Feb 2012 14:20:01 +0100 (CET) Message-ID: <4F293C00.8000005@kippdata.de> Date: Wed, 01 Feb 2012 14:20:00 +0100 From: Rainer Jung User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0) Gecko/20111222 Thunderbird/9.0.1 MIME-Version: 1.0 To: dev@httpd.apache.org Subject: Re: [PATCH] trunk/2.4 core output filter is broken References: <20120120165210.GA23839@redhat.com> <201201292053.09488.sf@sfritsch.de> <4F27A95A.4090206@gknw.net> <201201312230.27824.sf@sfritsch.de> In-Reply-To: <201201312230.27824.sf@sfritsch.de> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit On 31.01.2012 22:30, Stefan Fritsch wrote: > On Tuesday 31 January 2012, Gregg Smith wrote: >> On 1/29/2012 11:53 AM, Stefan Fritsch wrote: >>> On Wednesday 25 January 2012, Jim Jagielski wrote: >>>> Looking over the code, impl as a hook seems more "isolated", >>>> rather than the current impl which is intrusive (which is >>>> part of what we're trying to avoid, aren't we?) >>> >>> OK, patch is attached. This needs review/testing for Windows. >> >> Patched trunk at r1237447, saw same after patching 2.4-HEAD so I >> tried trunk. >> >> Regards, >> Gregg >> >> Compiling... >> service.c >> .\server\mpm\winnt\service.c(38) : error C2370: 'mpm_winnt_module' >> : redefinition; different storage class >> >> c:\build3\httpd-head_r1237447\server\mpm\winnt\mpm_winnt.h(70) >> >> : see declaration of 'mpm_winnt_module' > > Oops. Please try this in addition. Thanks for the testing. No info about Windows, but: I applied the patches on trunk and ran the test suite against a reallyall build on Solaris 10. No test failures. Regards, Rainer