Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 329965 - dev-lang/perl-5.12.1-r1: "Switch will be removed from the Perl code distribution..."
Summary: dev-lang/perl-5.12.1-r1: "Switch will be removed from the Perl code distribut...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Perl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-26 17:56 UTC by Samuli Suominen (RETIRED)
Modified: 2010-08-01 12:55 UTC (History)
1 user (show)

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 Samuli Suominen (RETIRED) gentoo-dev 2010-07-26 17:56:01 UTC
From media-sound/volwheel & dev-lang/perl-5.12.1-r1

$ volwheel 
Switch will be removed from the Perl core distribution in the next major release. Please install it from CPAN. It is being used at /usr/bin/volwheel, line 21.

I couldn't find Switch from dev-perl or perl-core categories...
Comment 1 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2010-07-26 18:20:08 UTC
http://search.cpan.org/~rgarcia/Switch-2.16/

I think this is a candidate for a virtual?

Ideally, I think nuking anything that needs Switch.pm would be advised because its just so bad, but I'm just that way inclined =).
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2010-07-26 18:23:28 UTC
virtual/perl-CGI (perl-core/CGI):

>>> /usr/lib64/perl5/vendor_perl/5.12.1/CGI/Switch.pm

It still complains about that missing module... is that a bug in volwheel instead?
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2010-07-26 18:24:28 UTC
Nevermind; missed your comment :)
Comment 4 Torsten Veller (RETIRED) gentoo-dev 2010-07-29 19:19:29 UTC
So should we add Switch or should we patch volwheel?
We could patch it to use "given" but then we would depend on >=dev-lang/perl-5.10.
Comment 5 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2010-07-29 19:31:27 UTC
(In reply to comment #4)
> So should we add Switch or should we patch volwheel?
> We could patch it to use "given" but then we would depend on
> >=dev-lang/perl-5.10.
> 

I'm thinking virtual/perl-Switch + dev-perl/Switch ( not in perl-core cos its no longer in perl's core =) )     

|| ( <dev/lang-perl-5.12 dev-perl/Switch )

then dep volwheel on virtual/perl-Switch

in dev-perl/Switch put warnings about its deprecation and now its bad to use, file bugs to upstream on volwheel and anything else that uses Switch suggesting "Can we have a copy without this crap?", and when nothings left in tree that needs it, hardmask it ( and the virtual ),

I don't see depping on >perl-5.10 a /bad/ thing, but it shouldn't be forced until 5.10 is stable. If upstream get solve our problems for us, then we can stick the new versions in tree as ~arch, and this problem can slip away =).

Alternatively, we can just hack it ourself, make it a -r1 and mark it as ~arch depping >=5.10
Comment 6 Torsten Veller (RETIRED) gentoo-dev 2010-07-29 19:43:05 UTC
(In reply to comment #5)
> I don't see depping on >perl-5.10 a /bad/ thing, but it shouldn't be forced
> until 5.10 is stable.

I hope we can stabilize perl-5.12.1 in August.

So if we ignore the warnings for now, we have another half a year to fix volwheel before 5.14 is released.
Comment 7 Olivier Duclos 2010-07-30 21:06:03 UTC
Hi there! I'm the author of VolWheel. I'm planning to release a new version this weekend, and it will fix this bug. I will not use 'given' because I need VolWheel to run on some old distributions.
Comment 8 Torsten Veller (RETIRED) gentoo-dev 2010-07-30 21:57:09 UTC
Olivier, that sounds great. Thanks
Comment 9 Olivier Duclos 2010-08-01 12:29:46 UTC
v0.2.8 is available!
Comment 10 Samuli Suominen (RETIRED) gentoo-dev 2010-08-01 12:55:38 UTC
(In reply to comment #9)
> v0.2.8 is available!
> 

excellent :)

+*volwheel-0.2.8 (01 Aug 2010)
+
+  01 Aug 2010; Samuli Suominen <ssuominen@gentoo.org>
+  +volwheel-0.2.8.ebuild:
+  Version bump wrt #329965, thanks to Torsten Veller and Kent Fredric for
+  analysis and Olivier Duclos (upstream) for the fix.