Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 434286 - dev-perl/MooseX-ChainedAccessors-0.20.0 file collision with dev-perl/MooseX-Attribute-Chained-1.0.1
Summary: dev-perl/MooseX-ChainedAccessors-0.20.0 file collision with dev-perl/MooseX-A...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All All
: Normal normal
Assignee: Gentoo Perl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-08 07:46 UTC by Brad Macpherson
Modified: 2012-09-10 15:36 UTC (History)
1 user (show)

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


Attachments
/var/tmp/portage/dev-perl/MooseX-ChainedAccessors-0.20.0/temp/perl-diagnostics.log (perl-diagnostics.log,25.74 KB, text/plain)
2012-09-08 08:57 UTC, Brad Macpherson
Details
Git patch to add the -r1 ebuild. (0001-Added-r1-ebuild-to-remove-dependency-on-deprecated-p.patch,7.06 KB, patch)
2012-09-08 19:27 UTC, Brad Macpherson
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brad Macpherson 2012-09-08 07:46:57 UTC
Running perl-cleaner --all, using the perl-experimental overlay; the emerge of dev-perl/MooseX-ChainedAccessors-0.20.0 collides with already-merged files from dev-perl/MooseX-Attribute-Chained-1.0.1:

 * dev-perl/MooseX-Attribute-Chained-1.0.1
 *      /usr/lib/perl5/vendor_perl/5.16.1/Moose/Meta/Attribute/Custom/Trait/Chained.pm
 *      /usr/lib/perl5/vendor_perl/5.16.1/MooseX/ChainedAccessors.pm
 *      /usr/lib/perl5/vendor_perl/5.16.1/MooseX/ChainedAccessors/Accessor.pm
 *      /usr/lib/perl5/vendor_perl/5.16.1/MooseX/Traits/Attribute/Chained.pm

The MooseX::ChainedAccessors CPAN package has been marked DEPRECATED by its maintainer, and its files appear to have moved to the MooseX::Attribute::Chained CPAN package. dev-perl/MooseX-ChainedAccessors-0.20.0 is being pulled in by the ebuild for dev-perl/Catalyst-Controller-HTML-FormFu-0.90.30 (all three of the upstream packages are maintained by Moritz Onken). I think the dependencies for dev-perl/Catalyst-Controller-HTML-FormFu-0.90.30 should probably be updated to dev-perl/MooseX-Attribute-Chained-1.0.1 and the ebuild for dev-perl/MooseX-ChainedAccessors retired; or preferably the dev-perl/MooseX-ChainedAccessors ebuild made into a virtual package depending on >=dev-perl/MooseX-Attribute-Chained-1.0.1 .

Reproducible: Always

Steps to Reproduce:
1. perl-cleaner --all
Actual Results:  
File collision.

Expected Results:  
Perl modules (specifically Task::Catalyst and the HTML::FormFu controller) should be merged.
Comment 1 Brad Macpherson 2012-09-08 08:57:47 UTC
Created attachment 323192 [details]
/var/tmp/portage/dev-perl/MooseX-ChainedAccessors-0.20.0/temp/perl-diagnostics.log
Comment 2 Brad Macpherson 2012-09-08 08:58:42 UTC
Bug https://github.com/fireartist/Catalyst-Controller-HTML-FormFu/issues/2 raised with upstream to remove the dependency on the deprecated package.
Comment 3 Brad Macpherson 2012-09-08 17:07:19 UTC
Hi - here's a diff against the perl-experimental ebuild for Catalyst-Controller-HTML-FormFu, with a revision-jumped copy in my local overlay; perl-cleaner is running through fine now.

--- /var/lib/layman/perl-experimental/dev-perl/Catalyst-Controller-HTML-FormFu/Catalyst-Controller-HTML-FormFu-0.90.30.ebuild   2012-09-04 18:09:34.359269396 +0100
+++ /usr/local/portage/dev-perl/Catalyst-Controller-HTML-FormFu/Catalyst-Controller-HTML-FormFu-0.90.30-r1.ebuild       2012-09-08 17:28:30.000000000 +0100
@@ -16,7 +16,7 @@
 
 RDEPEND="
        >=dev-perl/Catalyst-Runtime-5.710.10
-       >=dev-perl/MooseX-ChainedAccessors-0.20.0
+       >=dev-perl/MooseX-Attribute-Chained-1.0.1
        dev-perl/Catalyst-Component-InstancePerContext
        dev-perl/Config-Any
        >=dev-perl/HTML-FormFu-0.90.0
Comment 4 Torsten Veller (RETIRED) gentoo-dev 2012-09-08 17:53:47 UTC
Brad, please fix it yourself :)

The repo is available at:
https://github.com/gentoo-perl/perl-experimental

(Fix the dependencies and remove the deprecated distribution.)

If you have questions, join us in #gentoo-perl
Comment 5 Brad Macpherson 2012-09-08 18:27:22 UTC
Hi Torsten - gladly, unfortunately I get a 403 when I try to push. I'll grab an IRC client and jump on #gentoo-perl later this evening.
Comment 6 Brad Macpherson 2012-09-08 19:27:55 UTC
Created attachment 323224 [details, diff]
Git patch to add the -r1 ebuild.
Comment 7 Torsten Veller (RETIRED) gentoo-dev 2012-09-10 15:36:59 UTC
Fixed. Thanks.

Please add a ChangeLog entry next time (see echangelog from app-portage/gentoolkit-dev).
Follow https://help.github.com/articles/using-pull-requests until you get access to the repo. (I am not sure how we should handle this, or how other teams do it.)