Sun Jun 15 17:55:25 2014: Request 95809 was acted upon.
Transaction: Correspondence added by s...@parasite.cc
Queue: Inline
Subject: Re: [rt.cpan.org #95809] [PATCH] Using Inline in a distribution
with multiple modules
Broken in: (no value)
Severity: (no value)
Owner: Nobody
Requestors: s...@parasite.cc
Status: open
Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=95809 >
On Sun, 15 Jun 2014 00:25:16 -0400
"Sisyphus via RT" <bug-inl...@rt.cpan.org> wrote:
> <URL: https://rt.cpan.org/Ticket/Display.html?id=95809 >
>
> Sub satisfy_makefile_dep() would perhaps be better placed in MakeMaker.pm but
> I don't see any imperative to move it there - especially since it's doing
> stuff that was already being done in Inline.pm (as you noted).
I hadn't really thought about that. I guess I could see it in either place,
but if it's moved to MakeMaker, then MakeMaker will have to be included in the
command run by the generated Makefile which right now only includes Inline. I
guess in that sense it made sense for it to remain in the Inline module since
it's code that's being using when Inline is in _INSTALL_ mode, not while it's
generating the Makefile when perl Makefile.PL is run. Or something. :)
> I'll get Inline-0.55_01 out in the next week, and we'll see how it goes.
>
> Thanks for your work !!
Sounds great! I enjoyed working with you on this, thanks for all the help and
input...
Jason
