Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 23 Feb 2006 05:54:32 -0600
From:      "Larry Rosenman" <ler@lerctr.org>
To:        "'Clement Laforet'" <sheepkiller@cultdeadsheep.org>
Cc:        apache@FreeBSD.org, 'Sean Chittenden' <sean@gigave.com>
Subject:   RE: mod_auth_pgsql for Apache22?
Message-ID:  <016301c6386f$f034b830$66c8a8c0@lerctr.org>

next in thread | raw e-mail | index | archive | help
Larry Rosenman wrote:
> Clement Laforet wrote:
>> On Wed, Feb 22, 2006 at 03:23:15PM -0600, Larry Rosenman wrote:
>>> USE_APACHE=     22
>>> 
>>> LATEST_LINK=    mod_auth_pgsql2
>>> 
>>> AP_INC?=        ${LOCALBASE}/include
>>> Makefile: 33 lines, 826 characters
>>> # make
>>> ===>  mod_auth_pgsql-2.0.2b1_2 : Error from bsd.apache.mk. Illegal
>>> use of USE_APACHE. *** Error code 1
>>> 
>>> Stop in /usr/ports/www/mod_auth_pgsql2.
>>> #
>>> 
>>> nope. :(
>> 
>> Congrats! you find a bug in bsd.apache.mk :) I'll fix it ASAP.
>> 
>> To make the port work with apache 2.[012].x., USE_APACHE should be
>> set to "20+". I don't have a tinderbox up so I can't test if it
>> builds properly. Please ping me if you get it built, so I can update
>> the port. 
>> 
>> clem
> 
> Well, I got a little further:
> # vi Makefile
> # New ports collection makefile for:    mod_auth_pgsql2
> # Date created:                         Mon Oct  6
> # Whom:                                 Clement Laforet
> <sheepkiller@cultdeadshe 
> ep.org>
> #
> # $FreeBSD: ports/www/mod_auth_pgsql2/Makefile,v 1.9 2005/12/04
>  21:08:46 clement Exp $
> 
> PORTNAME=       mod_auth_pgsql
> PORTVERSION=    2.0.2b1
> PORTREVISION=   2
> CATEGORIES=     www
> MASTER_SITES=   http://www.giuseppetanzilli.it/mod_auth_pgsql2/dist/
> 
> MAINTAINER=     apache@FreeBSD.org
> COMMENT=        Allows users to use PostgreSQL databases for user
> authentication 
> 
> USE_PGSQL=      yes
> USE_APACHE=     20+
> 
> LATEST_LINK=    mod_auth_pgsql2
> 
> AP_INC?=        ${LOCALBASE}/include
> Makefile: 33 lines, 827 characters
> # make
> => mod_auth_pgsql-2.0.2b1.tar.gz doesn't seem to exist in
> /usr/ports/distfiles/. => Attempting to fetch from
> http://www.giuseppetanzilli.it/mod_auth_pgsql2/dist/. 
> mod_auth_pgsql-2.0.2b1.tar.gz                 100% of   15 kB   34
> kBps ===>  Extracting for mod_auth_pgsql-2.0.2b1_2
> => MD5 Checksum OK for mod_auth_pgsql-2.0.2b1.tar.gz.
> => SHA256 Checksum OK for mod_auth_pgsql-2.0.2b1.tar.gz.
> ===>  Patching for mod_auth_pgsql-2.0.2b1_2
> ===>   mod_auth_pgsql-2.0.2b1_2 depends on file: /usr/local/sbin/apxs
> - found ===>   mod_auth_pgsql-2.0.2b1_2 depends on file:
> /usr/local/sbin/apxs - found ===>   mod_auth_pgsql-2.0.2b1_2 depends
> on shared library: pq.4 - found ===>  Configuring for
> mod_auth_pgsql-2.0.2b1_2 ===>  Building for mod_auth_pgsql-2.0.2b1_2
> ===>  Generating apache plist
> /usr/local/build-1/libtool --silent --mode=compile cc -prefer-pic -O2
> -fno-strict-aliasing -pipe    -I/usr/local/include/apache22 
> -I/usr/local/include/apr-1   -I/usr/local/include/apr-1
> -I/usr/local/include -I/usr/local/include -I/usr/local/include  -c -o
> mod_auth_pgsql.lo mod_auth_pgsql.c && touch mod_auth_pgsql.slo
> mod_auth_pgsql.c:91:24: apr_compat.h: No such file or directory    
> mod_auth_pgsql.c: In function `auth_pg_md5':
> mod_auth_pgsql.c:452: error: `MD5_DIGESTSIZE' undeclared (first use
> in this function) 
> mod_auth_pgsql.c:452: error: (Each undeclared identifier is reported
> only once 
> mod_auth_pgsql.c:452: error: for each function it appears in.)
> mod_auth_pgsql.c: In function `do_pg_query':
> mod_auth_pgsql.c:634: warning: cast to pointer from integer of
> different size 
> mod_auth_pgsql.c: In function `auth_pg_md5':
> mod_auth_pgsql.c:453: error: storage size of `md5hash' isn't known
> apxs:Error: Command failed with rc=65536
> .
> *** Error code 1
> 
> Stop in /usr/ports/www/mod_auth_pgsql2.
> #
> 
> I can give you an account on this box, if that would help....

Ok, the MD5_DIGEST constant now has a APR_ in front of it, and the
Apr_compat.h header is gone.

I think there is a later version of mod_auth_pgsql2 that is release, not
beta.

This one compiled with the changes implied above:

1) don't try to include apr_compat.h
2) s/MD5_DIGESTSIZE/APR_MD5_DIGESTSIZE/g

We still get the warning on line 634 about pointer<->int, but....

LER


-- 
Larry Rosenman                     http://www.lerctr.org/~ler
Phone: +1 512-248-2683                 E-Mail: ler@lerctr.org
US Mail: 430 Valona Loop, Round Rock, TX 78681-3683 US




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?016301c6386f$f034b830$66c8a8c0>