[Dancer-users] Fwd: Recent Build requirement breaks 5.8.x installs

[Lee Carmichael]

Gmail removed dancer users from email list :(


---------- Forwarded message ----------
From: Lee Carmichael <leecarmichael123 at gmail.com>
Date: Tue, May 31, 2011 at 1:18 PM
Subject: Re: [Dancer-users] Recent Build requirement breaks 5.8.x installs
To: Alberto Simoes <ambs at perl-hackers.net>


Hello,

Pulling out the BUILD_REQUIRES entry with captures works just fine:

Magic-box:Dancer-1.3051 root# diff  orig.Makefile.PL Makefile.PL
39,40c39
<         'Test::More'              => '0.94',
<         'Tie::Hash::NamedCapture' => '0',
---
>         'Test::More'              => '0.94'

Also, I noticed in the cpan shell, if I answer 'No' to prepending
'Tie::Hash::NamedCapture' the install works.

Thanks,

Lee



On Tue, May 31, 2011 at 11:42 AM, Alberto Simoes <ambs at perl-hackers.net> wrote:
>
> 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.
>>
>>
>
> _______________________________________________
> Dancer-users mailing list
> Dancer-users at perldancer.org
> http://www.backup-manager.org/cgi-bin/listinfo/dancer-users
>