Bug #86609 for ExtUtils-MakeMaker: MakeMaker Fails to Recognize v-string Versions

Mon Jul 01 07:49:44 2013 dwheeler [...] cpan.org - Ticket created

Subject:	MakeMaker Fails to Recognize v-string Versions
Date:	Mon, 1 Jul 2013 13:49:12 +0200
To:	bug-extutils-makemaker [...] rt.cpan.org
From:	"David E. Wheeler" <dwheeler [...] cpan.org>

Given this Makefile.PL: use ExtUtils::MakeMaker; WriteMakefile( 'NAME' => 'Foo', 'VERSION_FROM' => 'Foo.pm', ); And this Foo.pm: package Foo; our $VERSION = v0.10.3; The output of `perl Makefile.PL` is: Can't parse version ' ' Writing Makefile for Foo Can't parse version ' ' Writing MYMETA.yml and MYMETA.json And the resulting Makefile has lines like this (binary bytes replaced with visible characters as presented in Emacs): VERSION = ^@ ^C Module::Build is able to parse vstring versions okay, FWIW. The workaround for EU::MM is to quote the version, instead: package Foo; our $VERSION = 'v0.10.3'; But I think EU::MM probably ought to be able to recognize vstrings.

Tue Jul 02 03:17:54 2013 BINGOS [...] cpan.org - Correspondence added 45 min

I have made a preliminary patch to EUMM to deal with this: https://github.com/Perl-Toolchain-Gang/ExtUtils-MakeMaker/commit/d3143e9f6246259d785e1d6367a2cfc9e726aada I hope to have a development release on CPAN later today.

Tue Jul 02 03:17:54 2013 The RT System itself - Status changed from 'new' to 'open'

Tue Jul 02 03:17:55 2013 BINGOS [...] cpan.org - Status changed from 'open' to 'patched'

Tue Jul 02 04:58:19 2013 dwheeler [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #86609] MakeMaker Fails to Recognize v-string Versions
Date:	Tue, 2 Jul 2013 10:58:00 +0200
To:	bug-ExtUtils-MakeMaker [...] rt.cpan.org
From:	"David E. Wheeler" <dwheeler [...] cpan.org>

On Jul 2, 2013, at 9:17 AM, BINGOS via RT <bug-ExtUtils-MakeMaker@rt.cpan.org> wrote: Show quoted text

> https://github.com/Perl-Toolchain-Gang/ExtUtils-MakeMaker/commit/d3143e9f6246259d785e1d6367a2cfc9e726aada > > I hope to have a development release on CPAN later today.

Does EU::MM still work with Perls without vstrings? Should there be a test or to to check the coverage? Best, David

Tue Jul 02 07:05:34 2013 BINGOS [...] cpan.org - Correspondence added 15 min

On Tue Jul 02 04:58:19 2013, DWHEELER wrote: Show quoted text

> On Jul 2, 2013, at 9:17 AM, BINGOS via RT <bug-ExtUtils- > MakeMaker@rt.cpan.org> wrote: >

> > https://github.com/Perl-Toolchain-Gang/ExtUtils-

> MakeMaker/commit/d3143e9f6246259d785e1d6367a2cfc9e726aada

> > > > I hope to have a development release on CPAN later today.

> > Does EU::MM still work with Perls without vstrings? > > Should there be a test or to to check the coverage? >

EUMM requires 5.6 or greater. I did update the applicable tests whilst fixing this.

Tue Jul 23 18:01:23 2013 BINGOS [...] cpan.org - Correspondence added 2 min

This issue is now resolved with the release of 6.70 Many thanks.

Tue Jul 23 18:01:24 2013 BINGOS [...] cpan.org - Status changed from 'patched' to 'resolved'