torsdagen den 19 januari 2006 16.59 skrev Steffen:
> Compiling on Windows and Apache 2.2.0 works with the patch.
>
> Steffen
>
> http://www.apachelounge.com
> ----- Original Message -----
> From: "Sierk Bornemann" <[EMAIL PROTECTED]>
> 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?
> >

Looks similar to my hack in Mandriva.

-- 
Regards // Oden Eriksson
Mandriva: http://www.mandriva.com
NUX: http://li.nux.se
--- src/mod_tidy.c	2005-11-11 01:47:57.000000000 +0100
+++ src/mod_tidy.c.oden	2005-12-16 02:33:27.000000000 +0100
@@ -186,7 +186,9 @@
 			cfg->tidyoptions, NULL );
 	}
 
-    APR_BRIGADE_FOREACH(e, bb) {
+    for (e = APR_BRIGADE_FIRST(bb);
+         e != APR_BRIGADE_SENTINEL(bb);
+         e = APR_BUCKET_NEXT(e)) {
         const char *str;
         apr_size_t len;
 		

Reply via email to