Skip Menu |

This queue is for tickets about the ExtUtils-MakeMaker CPAN distribution.

Report information
The Basics
Id: 86007
Status: resolved
Priority: 0/
Queue: ExtUtils-MakeMaker

People
Owner: Nobody in particular
Requestors: 'spro^^*%*^6ut# [...] &$%*c
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 6.67_05



Subject: [PATCH] Move ‘Putting my or local on the preceding line’
This sentence was divorced from what it describes some years ago. Please consider the attached patch.
On Sun Jun 09 02:17:21 2013, SPROUT wrote: Show quoted text
> This sentence was divorced from what it describes some years ago. > Please consider the attached patch.
Erm, here is the patch.
Subject: open_LK0QTTz5.txt
diff -rup ExtUtils-MakeMaker-6.66-OJyGMl/lib/ExtUtils/MakeMaker.pm ExtUtils-MakeMaker-6.66-E37t3a/lib/ExtUtils/MakeMaker.pm --- ExtUtils-MakeMaker-6.66-OJyGMl/lib/ExtUtils/MakeMaker.pm 2013-04-19 09:51:48.000000000 -0700 +++ ExtUtils-MakeMaker-6.66-E37t3a/lib/ExtUtils/MakeMaker.pm 2013-06-08 23:11:00.000000000 -0700 @@ -2470,6 +2470,8 @@ but these will fail: local $VERSION = '1.02'; local $FOO::VERSION = '1.30'; +(Putting C<my> or C<local> on the preceding line will work o.k.) + "Version strings" are incompatible and should not be used. # Bad @@ -2484,8 +2486,6 @@ as $VERSION is declared. # All on one line use version; our $VERSION = qv(1.2.3); -(Putting C<my> or C<local> on the preceding line will work o.k.) - The file named in VERSION_FROM is not added as a dependency to Makefile. This is not really correct, but it would be a major pain during development to have to rewrite the Makefile for any smallish
This has been resolved by the 6.68 release. Many thanks.