Bug#722332: libcommon-sense-perl: common::sense runs into trouble with Perl 5.18, eg. no say().

[Axel Beckert]

Hi Hagen,

Hagen Fuchs wrote:
> First of all: Nice response time!  I'm always in awe of Debian maintainers'
> work ethic.

You're welcome.

> > I've committed the changes for the first point; reviews welcome!
> 
> I'm not sure how I can review those changes?

Our package tracking system at
http://packages.qa.debian.org/libcommon-sense-perl (linked from the
package search at http://packages.debian.org/libcommon-sense-perl,
too) shows where our packaging git repository for that package is
located:

http://anonscm.debian.org/gitweb/?p=pkg-perl/packages/libcommon-sense-perl.git

> Meanwhile, here's the results of a little investigation of my own:
> 
> Executive summary.
> This is probably a case for upstream: they'll have to adapt their use of
> the '$^H' variable.

Thanks for this investigation.

I've just uploaded Gregor's fix on the packaging side (as there were
also multiple people posting there review results on our IRC channel
(#debian-perl on irc.debian.org), but I suspect that this isn't the
end of this issue. So we're happy about your review even though the
bug itself is considered fixed for Debian Unstable.

Wondering if we should clone it with lowered severity for a more
elegant fix. Comments?

> And, of course, a few more manglings happen.  Now, simply saying
> 
>     sed -i 's|0x820f00|0x1c000000|' /usr/lib/perl5/common/sense.pm
> 
> does the trick:
> 
>     $ perl -Mcommon::sense -e 'say $common::sense::VERSION; say $^V; say "Hi!";'
>     3.72
>     v5.18.1
>     Hi!
> 
> (Mind, this naïve substitution is all kinds of wrong -- this is a bitmask with
> definitive meaning and will quite probably still not work for all other
> use cases of common::sense (utf8, switch, ...) or even introduce more
> subtle errors!)

*nod* That definitely needs more investigation. Could have quite some
tripping hazards in that solution.

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe at debian.org>, http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE
  `-    |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5