Bug #101273 for Parse-PMFile: does not support { our $VERSION = '...' }

Sat Jan 03 20:02:28 2015 ether [...] cpan.org - Ticket created

Subject:

does not support { our $VERSION = '...' }

Parse::PMFile is not properly parsing version declarations that look like: { our $VERSION = '0.001'; } Note that this is a syntax that newer versions of Dist::Zilla::Plugin::PkgVersion (in core dzil) use: https://metacpan.org/source/RJBS/Dist-Zilla-5.029/lib/Dist/Zilla/Plugin/PkgVersion.pm#L169 Also see this ticket for an overview of some toolchain issues it has caused: https://rt.cpan.org/Ticket/Display.html?id=101095 I am preparing a PR that contains a TODO test for this.

Sat Jan 03 20:03:02 2015 ether [...] cpan.org - Reference to ticket #101095 added

Sat Jan 03 20:04:05 2015 ether [...] cpan.org - Correspondence added

Show quoted text

> I am preparing a PR that contains a TODO test for this.

Here it is: https://github.com/charsbar/Parse-PMFile/pull/7

Sat Jan 03 23:01:08 2015 ether [...] cpan.org - Correspondence added

On 2015-01-03 17:04:05, ETHER wrote: Show quoted text

>

> > I am preparing a PR that contains a TODO test for this.

> > Here it is: https://github.com/charsbar/Parse-PMFile/pull/7 >

The change I made here may be illuminating: https://github.com/Perl-Toolchain-Gang/Module-Metadata/commit/2e9ed0c20f09085b4884c9140569664d38eccdb3 (as well as #toolchain discussion in the last day or so)

Sun Jan 04 01:51:32 2015 ishigaki [...] cpan.org - Correspondence added

On Sun Jan 04 13:01:08 2015, ETHER wrote: Show quoted text

> On 2015-01-03 17:04:05, ETHER wrote:

> >

> > > I am preparing a PR that contains a TODO test for this.

> > > > Here it is: https://github.com/charsbar/Parse-PMFile/pull/7 > >

> > The change I made here may be illuminating: https://github.com/Perl- > Toolchain-Gang/Module- > Metadata/commit/2e9ed0c20f09085b4884c9140569664d38eccdb3 > > (as well as #toolchain discussion in the last day or so)

Thanks. Applied __clean_eval fix and released 0.34.

Sun Jan 04 01:51:32 2015 The RT System itself - Status changed from 'new' to 'open'

Sun Jan 04 01:51:34 2015 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'