Renaming proxy backends

[Sander Striker]

Hi,

Anyone object to me renaming proxy_[ajp|balancer|connect|ftp|http].c to
mod_proxy_[ajp|balancer|connect|ftp|http].c?

Reason:

IfModule mod_proxy_http.c
  ...
/IfModule

I had to look a few times to figure out I had to leave out the
mod_ part.  Since all the other modules start with mod_, I'd
like to make this consistent.


Sander

Re: Renaming proxy backends

[Paul Querna]

Sander Striker wrote:
Hi,
Anyone object to me renaming proxy_[ajp|balancer|connect|ftp|http].c to
mod_proxy_[ajp|balancer|connect|ftp|http].c?
Reason:
IfModule mod_proxy_http.c
  ...
/IfModule
I had to look a few times to figure out I had to leave out the
mod_ part.  Since all the other modules start with mod_, I'd
like to make this consistent.
Sander

An alternative:
IfModule proxy_http_module
IfModule proxy_ajp_module
IfModule proxy_ftp_module
etc
These should all work, and are not dependent on the name of the source 
file.

-Paul

Re: Renaming proxy backends

[Jim Jagielski]

On Feb 3, 2005, at 12:33 PM, Sander Striker wrote:
Hi,
Anyone object to me renaming proxy_[ajp|balancer|connect|ftp|http].c to
mod_proxy_[ajp|balancer|connect|ftp|http].c?
Reason:
IfModule mod_proxy_http.c
  ...
/IfModule
I had to look a few times to figure out I had to leave out the
mod_ part.  Since all the other modules start with mod_, I'd
like to make this consistent.
+1

Re: Renaming proxy backends

[Mladen Turk]

Sander Striker wrote:
Hi,
Anyone object to me renaming proxy_[ajp|balancer|connect|ftp|http].c to
mod_proxy_[ajp|balancer|connect|ftp|http].c?
Reason:
IfModule mod_proxy_http.c
  ...
/IfModule
I had to look a few times to figure out I had to leave out the
mod_ part.  Since all the other modules start with mod_, I'd
like to make this consistent.
+1
Mladen.

Re: Renaming proxy backends

[Graham Leggett]

William A. Rowe, Jr. said:

 While we are at it... util_ldap.c?  This wasn't the only odd
 case we have out there.

+1.

Regards,
Graham
--