[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-09-15 14:24 EST -------
(In reply to comment #14)
> +# Requires: filter perl(MasonX::Request::PlusApacheSession)
> +cat <<__EOF__ > %{name}-perlreq
> +#!/bin/sh
> +/usr/lib/rpm/perl.req \$* | grep -v 'perl(MasonX::Request::PlusApacheSession)'
> +__EOF__
> +%define __perl_requires %{_builddir}/HTML-Mason-%{version}/%{name}-perlreq
> +chmod +x %{__perl_requires}
> 
> Is this really necessary?  I don't see this dependency.

I seem to recall that perl(MasonX::Request::PlusApacheSession) gets pulled in by
one of the examples.  I had to do something like this with an older custom
package of HTML::Mason.

> -mkdir -p $RPM_BUILD_ROOT%{_localstatedir}/www/mason/html
> +mkdir -p $RPM_BUILD_ROOT%{_localstatedir}/www/mason
> 
> Shouldn't DocumentRoot (/var/www/mason/html) be under MasonCompRoot
> (/var/www/mason)?  I see that's not what the config file I'm including does now.

When I set up a Mason site, I usually use /var/www/html as the document root and
/var/www/mason as the private component root.  YMMV, of course.


-- 
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.