Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 310513 - Please update perl-module.eclass for EAPI=3
Summary: Please update perl-module.eclass for EAPI=3
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Gentoo Perl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: perl-eclass prefix-gx86
  Show dependency tree
 
Reported: 2010-03-21 10:48 UTC by Fabian Groffen
Modified: 2010-04-17 23:00 UTC (History)
1 user (show)

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


Attachments
perl-module EAPI=3 patch (perl-module-eapi3.patch,3.21 KB, patch)
2010-03-21 10:49 UTC, Fabian Groffen
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fabian Groffen gentoo-dev 2010-03-21 10:48:08 UTC
perl-module.eclass is not yet ready for EAPI=3.  The attached patch is a suggestion on what could be done to make it so.
Comment 1 Fabian Groffen gentoo-dev 2010-03-21 10:49:09 UTC
Created attachment 224479 [details, diff]
perl-module EAPI=3 patch
Comment 2 Torsten Veller (RETIRED) gentoo-dev 2010-03-21 11:51:00 UTC
It's already in <http://git.overlays.gentoo.org/gitweb/?p=proj/perl-overlay.git;a=tree;f=eclass;hb=HEAD>. Testing will reveal if i missed anything.

The eclass function names might change if we can come up with something better.

I'll ask for review/comments in -dev soon.
Comment 3 Fabian Groffen gentoo-dev 2010-03-21 11:56:46 UTC
Ok.

Looks like your EAPI case statement in perl-module.eclass doesn't handle EAPI=3, is that correct?
Comment 4 Torsten Veller (RETIRED) gentoo-dev 2010-03-21 21:49:01 UTC
(In reply to comment #3)
> Looks like your EAPI case statement in perl-module.eclass doesn't handle
> EAPI=3, is that correct?

I fixed it. Thanks
Comment 5 Torsten Veller (RETIRED) gentoo-dev 2010-03-30 07:44:46 UTC
07:35 <@ABCD> the hunks where `find "${D}" -type f ...` is changed to 
              `find "${ED}" -type f ...` are not needed, as 
              they will end up finding the exact same files
07:37 <@ABCD> so the fixlocalpod() function can be left entirely alone 
              (everything in it works just fine with $D instead of $ED,
              as there are never any files in $D that are not in $ED 
              (if there are, the build will fail anyway))

Changed.