[Bug 166063] Review Request: perl-HTML-Mason

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-HTML-Mason


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=166063





------- Additional Comments From chris at chrisgrau.com  2005-08-21 02:00 EST -------
(In reply to comment #3)
[snip]
> - /var/www/mason and /var/www/comp
> I am not sure if these are clever choices. /var/www/mason seems reasonable to
> me, but /var/www/comp seems a bit too unspecific to me. However I don't have
> strong opinion on this.

My (unrequested) opinion would be to do what conf.d/perl.conf does and leave
every option commented as examples of what could be done.  For example, I use
/var/cache/mason (like mod_proxy) for MasonDataDir and /var/www/mason as
MasonCompRoot.  Also, since libapreq2 is available in extras, an example of
using the apache args method would be nice in addition to the cgi args method.

[snip]
> I am not as much an apache expert to be able to tell what is wrong with it.
> AFAIS, it definitely is related to these lines in perl-HTML-Mason.conf, which
> would indicate a mod_perl/mod_perl2 incompatiblity issue:
> PerlModule Apache2
> PerlModule Apache::compat

Neither of these lines are used with current versions of mod_perl and
HTML::Mason.  A physical Apache2 module was removed in mod_perl 2.0.0rc5, and
Apache::compat isn't needed in HTML::Mason >= 1.30.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.