develooper Front page | perl.perl5.porters | Postings from September 2013

Re: [perl #119889] Bleadperl v5.19.3-653-ga49b10d breaks ILMARI/DBIx-Class-Schema-Loader-0.07036.tar.gz

Thread Previous | Thread Next
From:
Brian Fraser
Date:
September 27, 2013 14:21
Subject:
Re: [perl #119889] Bleadperl v5.19.3-653-ga49b10d breaks ILMARI/DBIx-Class-Schema-Loader-0.07036.tar.gz
Message ID:
CA+nL+nYrMwSR+swU7++7kEVWXkbTCU2rSORfasGmxsbUUKCrVg@mail.gmail.com
On Thu, Sep 26, 2013 at 10:12 AM, Father Chrysostomos via RT <
perlbug-followup@perl.org> wrote:

> On Sun Sep 22 20:54:24 2013, Hugmeir wrote:
> > On Fri, Sep 20, 2013 at 12:40 PM, Father Chrysostomos via RT <
> > perlbug-followup@perl.org> wrote:
> >
> > > On Fri Sep 20 02:24:04 2013, Hugmeir wrote:
> > > > For FREW/DBIx-Class-DeploymentHandler, the problem comes down to
> > > >
> > > > my %foo = (stuff => '%hash');
> > > > sub foo (&){["sub(&)"]}
> > > > warn @{foo { "stuff" }};
> > > >
> > > > Which changes behavior depending on whenever there's a newline after
> > > > the
> > > > @{, but consistently stays as @{foo{"stuff"}} in blead. Both suck,
> but
> > > > I
> > > > think consistency trumps.
> > > > Ideally, we should be detecting if a bareword followed by {...} is a
> &
> > > > sub,
> > > > but that's likely not doable at that point of the parsing, and
> > > > wouldn't
> > > > play well with plugged in keywords and/or parsers.
> > >
> > > I think the module should change.
> > >
> >
> > I've submitted a patch for the module on github[*] and pushed tests for
> > this in 7bb20a1.
> > Uh, I've never really followed this process through, so I have to ask,
> what
> > now? Does the ticket remain open until the affected module is patched?
>
> What I’ve been doing is adding it to Porting/perl5200delta.pod’s Known
> Issues section and then closing the ticket.  If it has fewer than 5
> dependents, though, I just close the ticket.
>
> The idea is that high-profile modules will be noted in 5.20.0’s delta.
> Or if the pumpking decides they are too high-profile, the release may be
> delayed.  But at least they all get listed in one place for the
> pumpking’s convenience.
>

Swell, thanks. DBIx::Class::DeploymentHandler has been patched to work on
>5.19.4, so this ticket can be closed, but I'll keep this in mind for the
next one.

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About