httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Steffen" <i...@apachelounge.com>
Subject Re: Problem: compiling mod_tidy with Apache 2.2
Date Thu, 19 Jan 2006 15:59:50 GMT
Compiling on Windows and Apache 2.2.0 works with the patch.

Steffen

http://www.apachelounge.com
----- Original Message ----- 
From: "Sierk Bornemann" <sierkb@gmx.de>
To: <dev@httpd.apache.org>
Sent: Thursday, January 19, 2006 3:28 PM
Subject: Problem: compiling mod_tidy with Apache 2.2


> Hi!
>
> I am the project maintainer of the Apache2 module mod_tidy
> (http://mod-tidy.sourceforge.net/), and there seems to be a problem
> compiling mod_tidy with Apache 2.2, because API has changed from
> Apache2.0 to Apache2.2:
>
> Compiler error messages:
> ------
> src/mod_tidy.c: In function 'mod_tidy_filter':
> src/mod_tidy.c:189: warning: implicit declaration of function
> 'APR_BRIGADE_FOREACH'
> src/mod_tidy.c:189: error: expected ';' before '{' token
> src/mod_tidy.c:154: warning: unused variable 'r'
> apxs:Error: Command failed with rc=65536
> ------
>
> APR_BRIGADE_FOREACH does not longer exist, so there must be a short fix
> reflecting this.
>
> I have a short patch here, which compiles well under apache2.0.55 and
> which lets to a functionable binary under Linux (OpenSuse), but I have
> no possibility yet to check, if it also compiles and works with apache2.2.
>
> For convenience, I have attached the little patch as a Unix diff to this
> email.
> The source tarball of mod_tidy is available on:
> http://mod-tidy.sourceforge.net/src/
> Is here anybody, who can help and have a look into the source and the
> patch or propose a better patch to solve the problem?
>
>
> Thanks in advance,
> Sierk Bornemann
>
> Sierk Bornemann | Hannover | Germany
> e-mail:  sierkb@gmx.de
> URL:     http://sierkbornemann.de/


--------------------------------------------------------------------------------


> 38,39c38,39
> <     $Date: 2006-01-12 22:47:25 +0100 (Thu, 12 Jan 2006) $
> <     $Revision: 96 $
> ---
>>     $Date: 2006-01-17 20:51:26 +0100 (Tue, 17 Jan 2006) $
>>     $Revision: 106 $
> 47a48
>> #include "apr_version.h"
> 189c190,197
> <     APR_BRIGADE_FOREACH(e, bb) {
> ---
>> #if APR_MAJOR_VERSION
>>     for (e = APR_BRIGADE_FIRST(bb);
>>          e != APR_BRIGADE_SENTINEL(bb);
>>          e = APR_BUCKET_NEXT(e))
>> #else
>>     APR_BRIGADE_FOREACH(e, bb)
>> #endif
>>         {
> 310c318
> < "<meta name=\"revision\" content=\"$Id: mod_tidy.c 96 2006-01-12
> 21:47:25Z bornemann $\"/>\n"
> ---
>> "<meta name=\"revision\" content=\"$Id: mod_tidy.c 106 2006-01-17
>> 19:51:26Z bornemann $\"/>\n"
> 605c613
> < "$Date: 2006-01-12 22:47:25 +0100 (Thu, 12 Jan 2006) $\n"
> ---
>> "$Date: 2006-01-17 20:51:26 +0100 (Tue, 17 Jan 2006) $\n"
>


Mime
View raw message