Recent Build requirement breaks 5.8.x installs
Hello, I was in the process of upgrading to Dancer 1.3051 on one machine that runs Perl 5.8 and it failed due to build requirement of 'Tie::Hash::NamedCapture'. This was added to Perl in 5.10. The main headache for me is that it broke the 'cpan install Dancer' command. But it didn't stop me from updating dancer by hand. I hate to be the one asking for support on an dated language version but i guess i am :) Should I open a bug for this or is the forward direction of Dance away from 5.8? Thanks, Lee
On Tuesday 31 May 2011 17:25:24 Lee Carmichael wrote:
Hello,
I was in the process of upgrading to Dancer 1.3051 on one machine that runs Perl 5.8 and it failed due to build requirement of 'Tie::Hash::NamedCapture'. This was added to Perl in 5.10. The main headache for me is that it broke the 'cpan install Dancer' command. But it didn't stop me from updating dancer by hand.
I hate to be the one asking for support on an dated language version but i guess i am :) Should I open a bug for this or is the forward direction of Dance away from 5.8?
We don't want to break compatibility with 5.8.x if we can possibly help it; we realise that there are plenty of users still stuck on 5.8.x, for whatever reason. I'll take a look in to that and see where the requirement comes from, and see if we can work round it. If we depend on it for certain features, such as being able to use captures() to get params by name, we should just throw a warning at runtime if we're on an older Perl, but let everything else still work. -- David Precious ("bigpresh") http://www.preshweb.co.uk/ "Programming is like sex. One mistake and you have to support it for the rest of your life". (Michael Sinz)
Lee, can I ask you to remove the line requiring that module from Makefile.PL and testing the module? I think it is there by mistake, but I do not have a 5.8.x perl to test here. Thank you On 31/05/2011 17:32, David Precious wrote:
On Tuesday 31 May 2011 17:25:24 Lee Carmichael wrote:
Hello,
I was in the process of upgrading to Dancer 1.3051 on one machine that runs Perl 5.8 and it failed due to build requirement of 'Tie::Hash::NamedCapture'. This was added to Perl in 5.10. The main headache for me is that it broke the 'cpan install Dancer' command. But it didn't stop me from updating dancer by hand.
I hate to be the one asking for support on an dated language version but i guess i am :) Should I open a bug for this or is the forward direction of Dance away from 5.8?
We don't want to break compatibility with 5.8.x if we can possibly help it; we realise that there are plenty of users still stuck on 5.8.x, for whatever reason.
I'll take a look in to that and see where the requirement comes from, and see if we can work round it.
If we depend on it for certain features, such as being able to use captures() to get params by name, we should just throw a warning at runtime if we're on an older Perl, but let everything else still work.
On Tue, May 31, 2011 at 7:25 PM, Lee Carmichael <leecarmichael123@gmail.com> wrote:
Hello,
I was in the process of upgrading to Dancer 1.3051 on one machine that runs Perl 5.8 and it failed due to build requirement of 'Tie::Hash::NamedCapture'. This was added to Perl in 5.10. The main headache for me is that it broke the 'cpan install Dancer' command. But it didn't stop me from updating dancer by hand.
I hate to be the one asking for support on an dated language version but i guess i am :) Should I open a bug for this or is the forward direction of Dance away from 5.8?
Yeah, that was the issue I mentioned yesterday on the IRC channel. I upgraded Dancer with 'notest install Dancer'. I am surprised it has not been fixed yet as this now makes it difficult for people to install the security fix. Gabor
On 31/05/2011 19:09, Gabor Szabo wrote:
On Tue, May 31, 2011 at 7:25 PM, Lee Carmichael <leecarmichael123@gmail.com> wrote:
Hello,
I was in the process of upgrading to Dancer 1.3051 on one machine that runs Perl 5.8 and it failed due to build requirement of 'Tie::Hash::NamedCapture'. This was added to Perl in 5.10. The main headache for me is that it broke the 'cpan install Dancer' command. But it didn't stop me from updating dancer by hand.
I hate to be the one asking for support on an dated language version but i guess i am :) Should I open a bug for this or is the forward direction of Dance away from 5.8?
Yeah, that was the issue I mentioned yesterday on the IRC channel. I upgraded Dancer with 'notest install Dancer'.
I am surprised it has not been fixed yet as this now makes it difficult for people to install the security fix.
Fixed on github for 10 minutes :) Cheers
participants (5)
-
Alberto Simoes -
ambs -
David Precious -
Gabor Szabo -
Lee Carmichael