PerlModule, PerlRequire, Perl and is now supported in
.htaccess. They will run for each request. [Gozer]
Philippe, can you please add the corresponding notes to the respective
docs sections (config.pod, PerlSections.pod)? It's great that you fix
all those things (and others), but if they aren't
Geoffrey Young wrote:
Have you by chance noticed:
% cat xs/tables/current/Apache/ConstantsTable.pm
package Apache::ConstantsTable;
# !!
# ! WARNING: generated by Apache::ParseSource/0.02
# ! Mon May 3 13:30:11 2004
# ! do NOT edit,
> Have you by chance noticed:
>
> % cat xs/tables/current/Apache/ConstantsTable.pm
> package Apache::ConstantsTable;
>
> # !!
> # ! WARNING: generated by Apache::ParseSource/0.02
> # ! Mon May 3 13:30:11 2004
> # ! do NOT edit, a
[EMAIL PROTECTED] wrote:
gozer 2004/08/11 15:49:15
Modified:.Changes
xs/tables/current/Apache ConstantsTable.pm
Log:
Add :context group of constants for check_cmd_context().
NOT_IN_VIRTUALHOST, NOT_IN_LIMIT, NOT_IN_DIRECTORY, NOT_IN_LOCATION,
NOT_IN_FILES,
Geoffrey Young wrote:
Stas Bekman wrote:
Some of the autoconfig magic starts to create more overhead than it used
to safe. e.g. all this magic of t/hooks/TestHooks is no longer useful,
as we add more tests and the test names aren't mapping 1:1 to the phase
name.
I think we should start stripping
Stas Bekman wrote:
Philippe M. Chiasson wrote:
Currently, soak_end_container resides under Apache::Command, while
expecting an
Apache::CmdParam as first argument. It was apparently misplaced to start
with.
I suggest either moving it to Apache::CmdParams or dropping it alltogether.
+1 to move
d
Stas Bekman wrote:
Philippe M. Chiasson wrote:
For now, things in Apache::Directive, Apache::Command and Apache::Module
having to
do with apache's configuration tree and modules have been made mostly
read-only.
After thinking about this a bit, I believe they might be changed to
writable before
Stas Bekman wrote:
Philippe M. Chiasson wrote:
it's a duplication of Apache::Module->name
$mod->name and $mod->find_module_name are identical
I'd like to remove the longer and stranger sounding find_module_name
method for clarity.
+1
Gone!
--
--
Stas Bekman wrote:
Philippe M. Chiasson wrote:
We are currently exposing dynamic_load_handle (i.e. the dlopen handle)
of a module
to the Perl API.
I don't think there is any usefullness to this and I'd like to see it
removed...
+1
Gone!
--
-
Stas Bekman wrote:
[EMAIL PROTECTED] wrote:
gozer 2004/08/10 11:18:08
Modified:todo api_status
Log:
Took care of Apache::Module API doc
Revision ChangesPath
1.24 +1 -1 modperl-2.0/todo/api_status
Index: api_status
Philippe M. Chiasson wrote:
Currently, soak_end_container resides under Apache::Command, while
expecting an
Apache::CmdParam as first argument. It was apparently misplaced to start
with.
I suggest either moving it to Apache::CmdParams or dropping it alltogether.
+1 to move
drop it if you can't s
Philippe M. Chiasson wrote:
For now, things in Apache::Directive, Apache::Command and Apache::Module
having to
do with apache's configuration tree and modules have been made mostly
read-only.
After thinking about this a bit, I believe they might be changed to
writable before
child-init, read-on
[EMAIL PROTECTED] wrote:
gozer 2004/08/10 11:18:08
Modified:todo api_status
Log:
Took care of Apache::Module API doc
Revision ChangesPath
1.24 +1 -1 modperl-2.0/todo/api_status
Index: api_status
===
Philippe M. Chiasson wrote:
it's a duplication of Apache::Module->name
$mod->name and $mod->find_module_name are identical
I'd like to remove the longer and stranger sounding find_module_name
method for clarity.
+1
--
__
Stas Bekman
Philippe M. Chiasson wrote:
We are currently exposing dynamic_load_handle (i.e. the dlopen handle)
of a module
to the Perl API.
I don't think there is any usefullness to this and I'd like to see it
removed...
+1
--
__
Stas Bekman
Geoffrey Young wrote:
Stas Bekman wrote:
+1 to remove all the code that isn't helping to test
$r->note_digest_auth_failure.
is this ok?
Excellent, +1
--
__
Stas BekmanJAm_pH --> Just Another mod_perl Hacker
http://stas
16 matches
Mail list logo