On Sat, Apr 21, 2012 at 12:39:26AM +1200, Kent Fredric wrote:
> If you really want to support Perl Modules, ( which theres not much
> need for at present, looks like the team have gone through already for
> the most part and added remote-id's where possible already ), anything
> that inherits 'perl-module.eclass' has a bit of magic, in that neither
> SRC_URI or HOMEPAGE is required in the ebuild, and it just gets the
> package name from what gentoo is using. We've tried to be as close to
> upstream as possible for the ease of maintenance.
Actually the eclass seems to fill SRC_URI and HOMEPAGE, so when you source
the ebuild they are available.
> However, there are still exception cases, for instance, BioPerl has to
> define 'MY_PN' to tell the perl-module eclass to use a different token
> ( and when this is present, it should be sufficient to say that that
> should be the remote-id instead of the package name:
> see dev-perl/Moose # an example with neither src_uri or homepage
> see sci-biology/bioperl # an example where the package name has
> been forced overridden as its changed upstream
>
> But resolving module names is much trickier, its easy-ish to map a
> module name to a package using the service, but doing it the other way
> round is not so straight forward, as one package can have many
> modules, and its common in perl to state dependencies in terms of the
> module to require, not the package its in, but there's also often a
> defacto "main module".
>
> But I'm myself still working out how to best do that with the service
> , so auto-populating a "cpan-module" identifier can be left to later,
> its just something I considered "useful" to have metadata wise because
> that value is more useful to users.
Yeah, not very important, but seems to work with this patch:
https://github.com/iksaif/portage-janitor/commit/972aff94744741e34e99f917337430d245883c48
Example:
$ python remoteids.py --diff WWW-Bugzilla Moose bioperl
--- a/dev-perl/WWW-Bugzilla/metadata.xml
+++ b/dev-perl/WWW-Bugzilla/metadata.xml
@@ -3,6 +3,7 @@
<pkgmetadata>
<herd>perl</herd>
<upstream>
+ <remote-id type="cpan-module">WWW::Bugzilla</remote-id>
<remote-id type="cpan">WWW-Bugzilla</remote-id>
</upstream>
</pkgmetadata>
--- a/dev-perl/Moose/metadata.xml
+++ b/dev-perl/Moose/metadata.xml
@@ -3,6 +3,7 @@
<pkgmetadata>
<herd>perl</herd>
<upstream>
+ <remote-id type="cpan-module">Moose</remote-id>
<remote-id type="cpan">Moose</remote-id>
</upstream>
</pkgmetadata>
--- a/sci-biology/bioperl/metadata.xml
+++ b/sci-biology/bioperl/metadata.xml
@@ -8,6 +8,7 @@
<flag name="db">Install sci-biology/bioperl-run</flag>
</use>
<upstream>
+ <remote-id type="cpan-module">BioPerl</remote-id>
<remote-id type="cpan">BioPerl</remote-id>
</upstream>
</pkgmetadata>
--
Corentin Chary
http://xf.iksaif.net/
|