Re: Apache, Solaris, AcceptMutex and EDEADLK

2008-06-12 Thread Michael Durket
(I sent this originally to Joe Orton who suggested I post it to this list instead): I've been recently debugging an issue with Solaris, Apache and EDEADLK. Turning to Google, I ran across several posts, but found this fairly recent post:

Call for 1.3.1 already

2008-06-12 Thread William A. Rowe, Jr.
Folks, we've found lots of nice small fixes to several platforms, and it would seem good to release a 1.3.1 bugfix update fairly promptly. Here are some options [ ] tag it friday (13th) [ ] tag it next wednesday (18th) [ ] tag it two weeks from friday (27th) wondering what folks want, and

Re: Call for 1.3.1 already

2008-06-12 Thread Jim Jagielski
On Jun 12, 2008, at 1:09 PM, William A. Rowe, Jr. wrote: Folks, we've found lots of nice small fixes to several platforms, and it would seem good to release a 1.3.1 bugfix update fairly promptly. Here are some options [ ] tag it friday (13th) That way, if people want to, they can

Re: Call for 1.3.1 already

2008-06-12 Thread Bojan Smojver
On Thu, 2008-06-12 at 12:09 -0500, William A. Rowe, Jr. wrote: we've found lots of nice small fixes to several platforms, and it would seem good to release a 1.3.1 bugfix update fairly promptly. Here are some options [X] tag it friday (13th) [ ] tag it next wednesday (18th) [ ] tag

Re: svn commit: r667182 - in /apr/apr-util/trunk: Makefile.win aprutil.dsw dbd/apr_dbd.c dbd/apr_dbd_odbc.c dbd/apr_dbd_odbc.dsp include/apu.hw include/private/apr_dbd_odbc_v2.h

2008-06-12 Thread Bojan Smojver
On Thu, 2008-06-12 at 17:54 +, [EMAIL PROTECTED] wrote: +SQL_C_CHAR, /*SQL_C_TYPE_TIME, /* APR_DBD_TYPE_TIME, \%pDi */ +SQL_C_CHAR, /*SQL_C_TYPE_DATE, /* APR_DBD_TYPE_DATE, \%pDd */ +SQL_C_CHAR, /*SQL_C_TYPE_TIMESTAMP, /* APR_DBD_TYPE_DATETIME, \%pDa */

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread William A. Rowe, Jr.
[EMAIL PROTECTED] wrote: Author: bojan Date: Thu Jun 12 17:11:13 2008 New Revision: 667300 URL: http://svn.apache.org/viewvc?rev=667300view=rev Log: Also make it (ODBC DBD) link :-) :) This is looking great, do we feel it's ready for people to try out with apr 1.3.1? Bill

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread Bojan Smojver
On Thu, 2008-06-12 at 19:17 -0500, William A. Rowe, Jr. wrote: This is looking great, do we feel it's ready for people to try out with apr 1.3.1? I have no idea if it actually works (it does compile and link). But hey - that's how we'll find out :-) -- Bojan

Re: svn commit: r667182 - in /apr/apr-util/trunk: Makefile.win aprutil.dsw dbd/apr_dbd.c dbd/apr_dbd_odbc.c dbd/apr_dbd_odbc.dsp include/apu.hw include/private/apr_dbd_odbc_v2.h

2008-06-12 Thread Tom Donovan
Bojan Smojver wrote: On Thu, 2008-06-12 at 17:54 +, [EMAIL PROTECTED] wrote: +SQL_C_CHAR, /*SQL_C_TYPE_TIME, /* APR_DBD_TYPE_TIME, \%pDi */ +SQL_C_CHAR, /*SQL_C_TYPE_DATE, /* APR_DBD_TYPE_DATE, \%pDd */ +SQL_C_CHAR, /*SQL_C_TYPE_TIMESTAMP, /*

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread Bojan Smojver
On Fri, 2008-06-13 at 10:26 +1000, Bojan Smojver wrote: I have no idea if it actually works (it does compile and link). It would be good if Windows folks could check this change that I made: -- #ifdef HAVE_SQL_H #include sql.h #include sqlext.h #elif defined(HAVE_ODBC_SQL_H)

Re: svn commit: r667182 - in /apr/apr-util/trunk: Makefile.win aprutil.dsw dbd/apr_dbd.c dbd/apr_dbd_odbc.c dbd/apr_dbd_odbc.dsp include/apu.hw include/private/apr_dbd_odbc_v2.h

2008-06-12 Thread Bojan Smojver
On Thu, 2008-06-12 at 20:27 -0400, Tom Donovan wrote: I would be happy to take the /*'s out if it's confusing. I already took them out. Is there a reason not to have /* in a comment? They don't nest in C (but, curiously - they're suppose to nest in SQL - although few dbs actually do

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread Tom Donovan
Bojan Smojver wrote: On Fri, 2008-06-13 at 10:26 +1000, Bojan Smojver wrote: It would be good if Windows folks could check this change that I made: -- #ifdef HAVE_SQL_H All set on Windows now - r667309

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread Bojan Smojver
On Thu, 2008-06-12 at 21:14 -0400, Tom Donovan wrote: All set on Windows now - r667309 Thanks! -- Bojan

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread Bojan Smojver
On Thu, 2008-06-12 at 19:17 -0500, William A. Rowe, Jr. wrote: This is looking great, do we feel it's ready for people to try out with apr 1.3.1? Tom, Do you have an opinion on this? PS. I would personally prefer to see ODBC DBC in 1.3.1. -- Bojan

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread Tom Donovan
Bojan Smojver wrote: On Thu, 2008-06-12 at 19:17 -0500, William A. Rowe, Jr. wrote: This is looking great, do we feel it's ready for people to try out with apr 1.3.1? Tom, Do you have an opinion on this? PS. I would personally prefer to see ODBC DBC in 1.3.1. It's seen considerable use

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread Bojan Smojver
On Thu, 2008-06-12 at 23:14 -0400, Tom Donovan wrote: I changed APU_DECLARE_DATA to APU_MODULE_DECLARE_DATA when I checked it into apr (copying the mysql driver). Am I missing something? Sadly, nothing. The whole build thing is busted - even for FreeTDS driver. I'm fixing now. -- Bojan

Re: svn commit: r667300 - /apr/apr-util/trunk/build/dbd.m4

2008-06-12 Thread William A. Rowe, Jr.
Bojan Smojver wrote: On Thu, 2008-06-12 at 23:14 -0400, Tom Donovan wrote: I changed APU_DECLARE_DATA to APU_MODULE_DECLARE_DATA when I checked it into apr (copying the mysql driver). Am I missing something? Sadly, nothing. The whole build thing is busted - even for FreeTDS driver. I'm

Re: svn commit: r667328 - in /apr/apr-util/trunk: dbd/apr_dbd.c dbd/apr_dbd_freetds.c dbd/apr_dbd_odbc.c include/apu.h.in

2008-06-12 Thread Bojan Smojver
On Fri, 2008-06-13 at 04:10 +, [EMAIL PROTECTED] wrote: @@ -646,7 +649,7 @@ if (SQL_SUCCEEDED(rc) || rc == SQL_NO_DATA) { -if (rc = SQL_SUCCESS_WITH_INFO +if (rc == SQL_SUCCESS_WITH_INFO ( len_indicator == SQL_NO_TOTAL || len_indicator = bufsize)

Re: svn commit: r667328 - in /apr/apr-util/trunk: dbd/apr_dbd.c dbd/apr_dbd_freetds.c dbd/apr_dbd_odbc.c include/apu.h.in

2008-06-12 Thread Tom Donovan
Bojan Smojver wrote: On Fri, 2008-06-13 at 04:10 +, [EMAIL PROTECTED] wrote: @@ -646,7 +649,7 @@ if (SQL_SUCCEEDED(rc) || rc == SQL_NO_DATA) { -if (rc = SQL_SUCCESS_WITH_INFO +if (rc == SQL_SUCCESS_WITH_INFO ( len_indicator == SQL_NO_TOTAL ||

Re: svn commit: r667328 - in /apr/apr-util/trunk: dbd/apr_dbd.c dbd/apr_dbd_freetds.c dbd/apr_dbd_odbc.c include/apu.h.in

2008-06-12 Thread Bojan Smojver
On Fri, 2008-06-13 at 00:32 -0400, Tom Donovan wrote: The second one doesn't change the behavior, just cleaning up -Wall warnings, right? That's right. Thanks for reviewing. If nobody objects, I'm going to backport ODBC driver to 1.3.x then. -- Bojan

Re: svn commit: r667328 - in /apr/apr-util/trunk: dbd/apr_dbd.c dbd/apr_dbd_freetds.c dbd/apr_dbd_odbc.c include/apu.h.in

2008-06-12 Thread Tom Donovan
Bojan Smojver wrote: On Fri, 2008-06-13 at 00:32 -0400, Tom Donovan wrote: The second one doesn't change the behavior, just cleaning up -Wall warnings, right? That's right. Thanks for reviewing. If nobody objects, I'm going to backport ODBC driver to 1.3.x then. Builds runs OK now on

Re: svn commit: r667328 - in /apr/apr-util/trunk: dbd/apr_dbd.c dbd/apr_dbd_freetds.c dbd/apr_dbd_odbc.c include/apu.h.in

2008-06-12 Thread William A. Rowe, Jr.
Bojan Smojver wrote: On Fri, 2008-06-13 at 01:14 -0400, Tom Donovan wrote: maybe ship it with httpd 2.2.9? Whew! Nah, you were just (un)lucky to hit the right spot :-) :) Unfortunately, not in the 2.2.9 package, since they adopted apr 1.3.0, but anyone will be free to grab on. I hope we