Re: [PATCH] ./native2/common/jk_channel_apr_socket.c - use apr_port_t instead of short for port

2004-02-02 Thread Henri Gomez
Commited, thanks - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]

Re: [PATCH] ./native2/common/jk_channel_apr_socket.c - use apr_port_t instead of short for port

2004-02-02 Thread Günter Knauf
Commited, thanks thanks, Henri! Do you know what's about the ./native2/common/jk_channel_socket.c file? Does soemone still use it since APR is now mandatory, or can we remove it? If it stays then we should also patch the port there to 'unsigned short'... Guenter.

Re: [PATCH] ./native2/common/jk_channel_apr_socket.c - use apr_port_t instead of short for port

2004-02-02 Thread Henri Gomez
Günter Knauf a écrit : Commited, thanks thanks, Henri! Do you know what's about the ./native2/common/jk_channel_socket.c file? Does soemone still use it since APR is now mandatory, or can we remove it? If it stays then we should also patch the port there to 'unsigned short'... Yes, it should be

Re: [PATCH] ./native2/common/jk_channel_apr_socket.c - use apr_port_t instead of short for port

2004-02-02 Thread Günter Knauf
Hi Henri, Do you know what's about the ./native2/common/jk_channel_socket.c file? Does soemone still use it since APR is now mandatory, or can we remove it? If it stays then we should also patch the port there to 'unsigned short'... Yes, it should be removed but the conservative approach will

Re: [PATCH] ./native2/common/jk_channel_apr_socket.c - use apr_port_t instead of short for port

2004-02-02 Thread Mike Anderson
Committed. Mike Anderson [EMAIL PROTECTED] 2/2/2004 10:29:26 AM Hi Henri, Do you know what's about the ./native2/common/jk_channel_socket.c file? Does soemone still use it since APR is now mandatory, or can we remove it? If it stays then we should also patch the port there to 'unsigned

Re: [PATCH] ./native2/common/jk_channel_apr_socket.c - use apr_port_t instead of short for port

2004-02-01 Thread NormW
Good morning All. This would also allow closing of Bug 17579 since the recent changeover to APR for mod_jk2 (now using jk_channel_apr_socket.c) Norm Hi, a user just found that the port number is limited to less than 32768 with mod_jk2; the patch below solves this:

Re: [PATCH] ./native2/common/jk_channel_apr_socket.c - use apr_port_t instead of short for port

2004-02-01 Thread Günter Knauf
Hi, This would also allow closing of Bug 17579 since the recent changeover to APR for mod_jk2 (now using jk_channel_apr_socket.c) thanks for the pointer! Showed that my recent patch was incomplete...; so next try: http://www.gknw.com/test/jk_channel_apr_socket.c.diff