Alvaro Lopez Ortega escribió:
>   There is no need to say that this code should never be used in a
>   production environment. These pre-releases are meant to be used only
>   by brave and intrepid beta testers. ;-)

I'll be glad to test it in a production environment yet no-so-visited as
sites from the other fellow users. However, currently I've spotted these
issues:

1) The configure script lacks an option for disabling LDAP support. I'm
a total noob to autoconf so I read about it, Googled for an
AX_LIB_MYSQL-like macro for LDAP, read again and finally wrote a small
patch for configure.in which hopefully provides the --with-ldap option.

2) As ADF said, there's a lack of documentation and information about
the current configuration file changes, yet sample conffiles are shipped
within the tarball. I could help rewriting the documentation, but since
I might procrastinate with that (not enough time until October), I would
like to help alo writing the "migration" script until more guidelines on
documentation are offered.

3) I used the plain (and bad and ugly and all that stuff) "phpcgi"
handler for a couple of PHP sites. As the new sample conffiles were
"fcgi" by default, I gave it a shot, but I got this error while trying
to fire Cherokee: "ERROR: fcgi handler needs a balancer" which is on
handler_fcgi.c:249, so:

  * I double-checked the paths and all seems correct.
  * I tried phpcgi and Cherokee booted, but PHP pages weren't served
  * I went back to 0.5.4 and tried fastcgi, and it worked.

4) On the Debian-related side, I built and tried this snapshot using
checkinstall in Debian Etch, with no problems.

Just my two cents by now. Thanks again to everyone who has invested time
working in Cherokee.

Jose

-- 
José M. Parrella -> Debian Sid, k2.6.17.4
Escuela de Ingenieria Electrica
Universidad Central de Venezuela -> ucvlug.info
--- configure.in        2006-09-06 12:47:14.000000000 -0400
+++ ../../cherokee-0.6.0b395/configure.in       2006-09-06 17:28:30.000000000 
-0400
@@ -893,16 +893,39 @@
 
 dnl
 dnl LDAP
+dnl This should offer a new --with-ldap option at the configure script
 dnl
-AC_CHECK_LIB(ldap, ldap_init, have_ldap_lib=yes, have_ldap_lib=no)
-AC_CHECK_HEADER(ldap.h, have_ldap_include=yes, have_ldap_include=no)
-if test "$have_ldap_lib $have_ldap_include" = "yes yes"; then
-   have_ldap="yes"
+
+AC_ARG_WITH([ldap],
+        AC_HELP_STRING([--with-ldap=@<:@ARG@:>@],
+            [use OpenLDAP development library @<:@default=yes@:>@, optionally 
specify path to dev libs]
+        ),
+        [
+        if test "$withval" = "no"; then
+            want_ldap="no"
+        elif test "$withval" = "yes"; then
+            want_ldap="yes"
+        else
+            want_ldap="yes"
+            LDAP_CONFIG="$withval"
+        fi
+        ],
+        [want_ldap="yes"]
+)
+
+if test "$want_ldap" = "yes"; then
+       AC_CHECK_LIB(ldap, ldap_init, have_ldap_lib=yes, have_ldap_lib=no)
+       AC_CHECK_HEADER(ldap.h, have_ldap_include=yes, have_ldap_include=no)
+       if test "$have_ldap_lib $have_ldap_include" = "yes yes"; then
+          have_ldap="yes"
+       else
+          have_ldap="no"
+       fi
 else
-   have_ldap="no"
+       have_ldap="no"
 fi
-AM_CONDITIONAL(HAVE_LDAP, test $have_ldap = "yes")
 
+       AM_CONDITIONAL(HAVE_LDAP, test $have_ldap = "yes")
 
 dnl
 dnl Check for MySQL 4.0.0 support

_______________________________________________
Cherokee mailing list
Cherokee@0x50.org
http://www.0x50.org/cgi-bin/mailman/listinfo/cherokee

Reply via email to