Re: Extra functionality for Build dist

On Fri, February 29, 2008 2:07 am, Eric Wilhelm wrote:
> # from Christopher J. Madsen
> # on Thursday 28 February 2008 23:34:
>
>>Module::Build::Dist:
>>  Making the trivial changes outside of version control
>>  that I've described here (VERSION sections & README)
>
> Ok, but why is that not also a separate developer tool?

Because I don't see how a separate tool could modify the files in distdir
without modifying the master copies in version control.

> Note:  one trouble with the "eval {require My::Build::Class}" is that if
> you have create_makefile_pl set to passthru, you get a compatibility
> Makefile.PL which (unconditionally) wants My::Build::Class (which is
> okay if you don't mind 99% of cpantesters sending you FAIL reports.)

Thanks, I hadn't thought of that.  But most problems are solvable if you
hit them with a big enough hammer.  How about this?

sub do_create_makefile_pl
{
  my $self = shift;

  if (ref($self) ne __PACKAGE__) {
    $self->SUPER::do_create_makefile_pl(@_);
  } else {
    bless $self, 'Module::Build';
    $self->do_create_makefile_pl(@_);
    bless $self, __PACKAGE__;
  }
} # end do_create_makefile_pl

I think I'll also have it read the generated Makefile.PL and make sure it
doesn't mention the package.

-- 
Chris Madsen                                          [email protected]
   --------------------  http://www.cjmweb.net  --------------------