Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread jean-frederic clere
[EMAIL PROTECTED] wrote:
mturk   2004/07/29 23:32:54
  Modified:ajp/ajplib/test ajp_header.c ajp.h
  Log:
  Change the casting from void* to msg*. We don't need that, cause
  the ajp_msg_t is public.
The ideas was to prevent including of ajp.h in proxy_ajp.c
Now ajp_header.c doesn't compile:
+++
ajp_header.c:738: error: conflicting types for `ajp_read_header'
ajp.h:360: error: previous declaration of `ajp_read_header'
make: *** [ajp_header.lo] Error 1
+++
Cheers
Jean-Frederic
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread Mladen Turk
 

 -Original Message-
 From: jean-frederic clere 
 
 The ideas was to prevent including of ajp.h in proxy_ajp.c 
 Now ajp_header.c doesn't compile:


But the ajp.h is a public API for libajp, so, it should be included in any
project using it.
It has defined 

MT.


smime.p7s
Description: S/MIME cryptographic signature


Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread jean-frederic clere
Mladen Turk wrote:
 


-Original Message-
From: jean-frederic clere 

The ideas was to prevent including of ajp.h in proxy_ajp.c 
Now ajp_header.c doesn't compile:

But the ajp.h is a public API for libajp, so, it should be included in any
project using it.
It has defined 
OK but I want to keep the AJP things as headen as possible in the proxy_ajp.c so 
 we need another include.

MT.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c ajp.h

2004-07-30 Thread Mladen Turk
 

jean-frederic clere wrote: 
  
  But the ajp.h is a public API for libajp, so, it should be 
 included in 
  any project using it.
  It has defined
 
 OK but I want to keep the AJP things as headen as possible in 
 the proxy_ajp.c so
   we need another include.


Sure thing.
Only the ajp.h will be needed.

MT.


smime.p7s
Description: S/MIME cryptographic signature


RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-30 Thread Mladen Turk
 

[EMAIL PROTECTED] commited:
   Log:
   Arrange small things and temporaly modify ajp_read_header().


I really don't understand why is unacceptable to include ajp.h in mod_proxy.
For example my compiler is complaining about missing prototypes in
proxy_ajp.

So either we'll make another header or just use the ajp.h
BTW there is nothing in the ajp.h that would brake the proxy_ajp thought.
Also I don't think that we need to hide msg using abstraction since we will
not use the .so for libajp inside proxy or we will?

MT.


smime.p7s
Description: S/MIME cryptographic signature


RE: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread Mladen Turk
 

[EMAIL PROTECTED]
 jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

   +rc = ajp_msg_create(r-pool,msg);

Can we use Apache coding convention having one space after comma in function
params?
It won't IMO have a chance to get into httpd tree otherwise.

I wanted to do that for JK2, but was always shut up, with the explanation
that it doesn't matter at the moment :)


MT.



smime.p7s
Description: S/MIME cryptographic signature


Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread jean-frederic clere
[EMAIL PROTECTED] wrote:
mturk   2004/07/28 09:34:22
  Modified:ajp/ajplib/test ajp_header.c
  Log:
  Modify sc_for_req_method. It is exactly the same as Apache's
  lookup_builtin_method with added SC_M_SEARCH
Thanks. Mine was telling:
+++
[Wed Jul 28 18:43:56 2004] [debug] ajp_header.c(239): Into 
ajp_marshal_into_msgb[Wed Jul 28 18:43:56 2004] [error] Error 
ajp_marshal_into_msgb - No such method GET
[Wed Jul 28 18:43:56 2004] [error] ajp_send_header: ajp_marshal_into_msgb failed
+++

It goes little futher but:
+++
[Wed Jul 28 18:47:38 2004] [debug] ajp_header.c(342): Into 
ajp_marshal_into_msgb[Wed Jul 28 18:47:39 2004] [notice] child pid 3072 exit 
signal Segmentation fault (11), possible coredump in /home/apache20/apache20
+++
Something wrong in ajp_msg.c?
+++
#0  0x08074bfd in ap_log_rerror (
file=0x402bcb00 ajp_msg_append_uint8(): BufferOverflowException %d %d,
line=1076611840, level=3, status=1076611840, r=0x0,
fmt=0x402bcb00 ajp_msg_append_uint8(): BufferOverflowException %d %d)
at log.c:576
#1  0x402bb047 in ajp_msg_append_uint8 (msg=0x831cbf8, value=2 '\002')
at ajp_msg.c:223
#2  0x402b9ebc in ajp_marshal_into_msgb (msg=0x831cbf8, r=0x83224a8)
at ajp_header.c:361
#3  0x402ba73f in ajp_send_header (sock=0x831cd78, r=0x83224a8)
at ajp_header.c:671
#4  0x402b82ef in ap_proxy_ajp_request (p=0x831c458, r=0x83224a8,
p_conn=0x831cbf8, origin=0x831cf10, conf=0x8286598, uri=0x831cbc8,
url=0x831cd68 /examples/, server_portstr=0xb4e0 :7779)
at proxy_ajp.c:489
#5  0x402b943b in ap_proxy_ajp_handler (r=0x83224a8, conf=0x8286598,
url=0x831cd68 /examples/, proxyname=0x0, proxyport=0) at proxy_ajp.c:1101
#6  0x4029d7e4 in proxy_run_scheme_handler (r=0x83224a8, conf=0x8286598,
url=0x83237c6 ajp://pgtr0327.mch.fsc.net:8009/examples/, proxyhost=0x0,
proxyport=0) at mod_proxy.c:1239
+++

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread jean-frederic clere
Mladen Turk wrote:
 

[EMAIL PROTECTED]
jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c
 +rc = ajp_msg_create(r-pool,msg);

Can we use Apache coding convention having one space after comma in function
params?
Oops, I have done lot of those... I will fix them tomorrow.
It won't IMO have a chance to get into httpd tree otherwise.
I wanted to do that for JK2, but was always shut up, with the explanation
that it doesn't matter at the moment :)
MT.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: cvs commit: jakarta-tomcat-connectors/ajp/ajplib/test ajp_header.c

2004-07-28 Thread jean-frederic clere
jean-frederic clere wrote:
[EMAIL PROTECTED] wrote:
mturk   2004/07/28 09:34:22
  Modified:ajp/ajplib/test ajp_header.c
  Log:
  Modify sc_for_req_method. It is exactly the same as Apache's
  lookup_builtin_method with added SC_M_SEARCH

Thanks. Mine was telling:
+++
[Wed Jul 28 18:43:56 2004] [debug] ajp_header.c(239): Into 
ajp_marshal_into_msgb[Wed Jul 28 18:43:56 2004] [error] Error 
ajp_marshal_into_msgb - No such method GET
[Wed Jul 28 18:43:56 2004] [error] ajp_send_header: 
ajp_marshal_into_msgb failed
+++

It goes little futher but:
+++
[Wed Jul 28 18:47:38 2004] [debug] ajp_header.c(342): Into 
ajp_marshal_into_msgb[Wed Jul 28 18:47:39 2004] [notice] child pid 3072 
exit signal Segmentation fault (11), possible coredump in 
/home/apache20/apache20
+++
Something wrong in ajp_msg.c?
Yes and now  I think I have fixed it ;-)
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]