Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Dist-Zilla CPAN distribution.

Report information
The Basics
Id: 56425
Status: open
Priority: 0/
Queue: Dist-Zilla
People
Owner: Nobody in particular
Requestors: avar [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 2.100960
Fixed in: (no value)

History Show all quoted text
  Fri Apr 09 08:44:32 2010 avar [...] cpan.org - Ticket created
Subject: Dist::Zilla should detect & die if it's using old plugins
MakeMaker::Awesome was broken because it used FixedPrereqs, but I didn't notice because Dist::Zilla happily used FixedPrereqs.pm from an older DZ version that I had in my @INC. There should be a sanity check in Dist::Zilla that checks that all the plugins in core (or known to have been in core) have the same $VERSION value. Alteratively FixedPrereqs.pm could still be included with C<die "Use $this instead">.
  Wed Jun 09 07:57:09 2010 rjbs [...] cpan.org - Correspondence added
On 2010-04-09 08:44:32, AVAR wrote: Show quoted text
> There should be a sanity check in Dist::Zilla that checks that all the > plugins in core (or known to have been in core) have the same $VERSION > value.
With the global config system overhauled, I should now be able to make a CorePlugin role that asserts that $self->VERSION must be eq Dist::Zilla->VERSION. So should anybody else. -- rjbs
  Wed Jun 09 07:57:10 2010 The RT System itself - Status changed from 'new' to 'open'
  Tue Jul 23 13:47:20 2013 rjbs [...] cpan.org - Correspondence added
I have added a branch on github, coreplugin, which adds the CorePlugin role. It's just Plugin, but also asserts that $self->VERSION eq $zilla->VERSION I'm not sure it's worth doing this, though. It will help with plugin version mismatches, but not roles. What if, for example, Dist::Zilla::Role::PPI left core. The same problem would arise. Once we're trying to inspect all roles, I think we're going to enter the realm of overcomplexity. Thoughts? -- rjbs