Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 33229 - New ebuild needed for mod_perl 1.99_11
Summary: New ebuild needed for mod_perl 1.99_11
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Highest normal (vote)
Assignee: Gentoo Perl team
URL: http://marc.theaimsgroup.com/?l=apach...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-11 07:26 UTC by Haroon Rafique
Modified: 2004-08-05 18:07 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Haroon Rafique 2003-11-11 07:26:10 UTC
mod_perl 1.99_11 was released yesterday, hence a new ebuild is needed.
The reason why this is marked as major is because as seen on the top of this
email:
   http://marc.theaimsgroup.com/?l=apache-modperl&m=106823905012088&w=2
it states that 1.99_11 will be required for users running perl 5.8.2 which was
released in portage as "~x86 ~amd blah blah...." just yesterday. BTW, I didn't
see this requirement in the Changes file.

In any case, the mod_perl ebuild should be released as soon as possible as I
am assuming that perl 5.8.2 will not be able to compile mod_perl 1.99_10 (and
lower) causing problems for user (and unnecessary bug reports).

I have successfully built (100% pass for tests) 1.99_11 and used it on my
development site without any hiccups so far. As mentioned in bug 30087 comment 5 the
sandbox fixes are already in the upstream sources, so the following fix is
unnecessary and should be taken out from the ebuild.

--- /usr/portage/dev-perl/mod_perl/mod_perl-1.99.10.ebuild      2003-10-30 20:01:18.000000000 -0500
+++ mod_perl-1.99.11.ebuild     2003-11-11 10:18:47.000000000 -0500
@@ -86,13 +86,6 @@
                chown nobody:nobody ${T}
        fi
 
-       # these next two stanzas avoid sandbox problems with make test
-       # tell mod_cgid to make the socket here instead of /var/run/cgisock
-       echo -e "\nScriptSock logs/cgisock" >> ${S}/t/conf/extra.conf.in
-       echo -e "\nScriptSock logs/cgisock" >> ${S}/ModPerl-Registry/t/conf/extra.conf.in
-       # tell CGI.pm to create new tmpfiles in this directory
-       echo -e "\nSetEnv TMPDIR ${T}" >> ${S}/t/conf/extra.conf.in
-
        # this does not || die because of bug 21325.
 
        make test

The only thing that I did not do is to create a "new" files/mod_perl-1.99.11-nonukes.patch
file since I have no clue what it does. Due to changes in the source code
for this version, this patch will have to be regenerated by whomever generated
it in the first place (and knows its purpose).
Comment 1 Tim Hobbs 2003-11-18 10:27:16 UTC
please set MAKEOPTS="-j1" (See bug 30257,comment 9)
Comment 2 Haroon Rafique 2003-11-18 10:32:05 UTC
Say what, Tim? This bug entry is about pushing up the mod_perl version to 11 since mod_perl 1.99_11 has some special doodads for perl 5.8.2.

In case you are wondering, I have no problems compiling 1.99_11 with perl 5.8.2.

Some people might also be inclined to try 1.99_10 with perl 5.8.2 which IMO might not be "the right thing to do" since the code might not behave properly even though it compiled properly.
Comment 3 Robert Coie (RETIRED) gentoo-dev 2003-12-19 16:26:56 UTC
1.99.11 went in to the tree a few days ago.  Enjoy.
Comment 4 Haroon Rafique 2003-12-22 11:01:22 UTC
Thanks Robert, however, as mentioned in the description and in bug 30087 comment 5, the sandbox "echo fixes" have made it upstream and should be taken out of the ebuild (as they are no longer necessary). What can I say, I like to take the minimalist approach (least amount of work possible :-) to get satisfactory results). Means more work for you tho ;-)

And on an aside, only 1.99.10 ebuild has a reference to nonukes.patch. Is that cause for concern? I don't know what nonukes is or what its purpose is.
Comment 5 Michael Cummings (RETIRED) gentoo-dev 2004-08-05 18:07:48 UTC
Lost bug - closing because its in the tree now (sorry Haroon)