Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-02 Thread Henri Gomez
May be at some time in the future we should think about a mod_jk 2.x requiring and using APR to remove all platform complexity :-) Even on Apache 1.3 we could have jk bundled with APR statically 2007/4/2, Mladen Turk [EMAIL PROTECTED]: Rainer Jung wrote: I used 2003 .net, but this includes

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-02 Thread Mladen Turk
Henri Gomez wrote: May be at some time in the future we should think about a mod_jk 2.x requiring and using APR to remove all platform complexity :-) Think we tried that with jk2 and failed. Even on Apache 1.3 we could have jk bundled with APR statically Don't think so. Regards, Mladen.

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-02 Thread Henri Gomez
jk2 was more than just APRised mod_jk. 2007/4/2, Mladen Turk [EMAIL PROTECTED]: Henri Gomez wrote: May be at some time in the future we should think about a mod_jk 2.x requiring and using APR to remove all platform complexity :-) Think we tried that with jk2 and failed. Even on Apache

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-02 Thread Mladen Turk
Rainer Jung wrote: Hi Mladen, I checked out jk trunk today and built mod_jk Can you double check the trunk? Think I've fixed the CRLF issue. Regards, Mladen. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-01 Thread Rainer Jung
Hi Mladen, I checked out jk trunk today and built mod_jk on XP pro using Visual Studio .net against httpd 2.0.59, 2.2.4 and 1.3.37. The build worked without problems. I then made a little test using LogLevel debug without rotatelogs, so logging directly to file and the resulting log file

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-01 Thread Mladen Turk
Rainer Jung wrote: Hi Mladen, I think we don't use stdio. APR has apr_file_open() which uses CreateFile() and apr_file_write() which goes back to WriteFile() on Windows. We are using stdio for other web servers, so if you add CRLF then the IIS will be broken. Perhaps opening the files as

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-01 Thread Mladen Turk
Rainer Jung wrote: Hi Mladen, I built against an installed form of the official download apache_1.3.37-win32-x86-src.msi. ... and you are using VS 2005 I suppose? That won't work cause you have different MSVCRT's Regards, Mladen.

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-01 Thread Rainer Jung
I used 2003 .net, but this includes a 7.0 version of msvcrt.dll. I build apache httpd 1.3 by myself and then mod_jk, and now everything works as expected. So you are right, the binary httpd download is not compatible with module compilation by vc++ 7.0. Any idea how one can make vc++ 7 use the

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-04-01 Thread Mladen Turk
Rainer Jung wrote: I used 2003 .net, but this includes a 7.0 version of msvcrt.dll. I build apache httpd 1.3 by myself and then mod_jk, and now everything works as expected. So you are right, the binary httpd download is not compatible with module compilation by vc++ 7.0. Any idea how one can

Re: svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-03-26 Thread Mladen Turk
Rainer Jung wrote: I think I added those after I was frustrated by a customer whose mod_jk logs had all non-windows line endings. Sorry, I didn't really test your change, but my experience questions this patch. With you patch the line endings were \r,\r,\n Do we need to set text mode on

svn commit: r521417 - /tomcat/connectors/trunk/jk/native/common/jk_util.c

2007-03-22 Thread mturk
Author: mturk Date: Thu Mar 22 12:35:34 2007 New Revision: 521417 URL: http://svn.apache.org/viewvc?view=revrev=521417 Log: Fix windows logging. There is no need to add the extra \r, because windows by default adds them for any text files. Modified: