On Tuesday 03 March 2009 16:15:28 Shlomi Fish wrote:
> On Saturday 28 February 2009 03:52:50 Michael G Schwern wrote:
> > Shlomi Fish wrote:
> > > $builder->add_build_element('extradata');
> > > $builder->install_path->{'extradata'} = $builder->prefix() .
> > > "/data/modules/XML-Grammar-Products-Syndication/data";
> >
> > Don't use prefix(), it's there for MakeMaker compatibility. Use
> > install_base().
>
> So should I do:
>
> {{{
> $builder->install_path->{'extra_data'} =
> File::Spec->catdir(
> $builder->install_base(),
> qw(data modules XML-Grammar-ProductsSyndication data)
> );
> }}}
>
> ?
>
> Because Eric didn't mention the install_path mangling in his response and
> placed everything under "lib/" (where it seems to be installed), so I
> assumed it was no longer needed.
>
> install_base is mentioned here:
>
> http://search.cpan.org/~ewilhelm/Module-Build-0.32/lib/Module/Build/API.pod
>
OK, converting to use install_base() in that script caused it to return
undef(). I didn't specify it in a command line argument, but expect it to work
nevertheless.
In any case, I found out that $builder->install_destination("lib") mostly did
the right thing, and I ended up with the following re-usable sub-class of
Module::Build : http://xrl.us/beh69o .
Any further comments on it will be welcome.
Regards,
Shlomi Fish
--
-----------------------------------------------------------------
Shlomi Fish http://www.shlomifish.org/
Stop Using MSIE - http://www.shlomifish.org/no-ie/
<mauke> I'm not interested in what you're doing; what are you trying to
achieve?
<PerlJam> mauke: I'm trying to achieve world peace and this regex is
the last thing standing in my way! ;)
